Message ID | 20250109205232.92336-1-kettenis@openbsd.org (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | arm64: dts: qcom: x1e80100: Mark usb_2 as dma-coherent | expand |
On 9.01.2025 9:52 PM, Mark Kettenis wrote: > Make this USB controller consistent with the others on this platform. > > Fixes: 4af46b7bd66f ("arm64: dts: qcom: x1e80100: Add USB nodes") > Signed-off-by: Mark Kettenis <kettenis@openbsd.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Konrad
On Thu, Jan 09, 2025 at 09:52:31PM +0100, Mark Kettenis wrote: > Make this USB controller consistent with the others on this platform. > > Fixes: 4af46b7bd66f ("arm64: dts: qcom: x1e80100: Add USB nodes") > Signed-off-by: Mark Kettenis <kettenis@openbsd.org> > --- > arch/arm64/boot/dts/qcom/x1e80100.dtsi | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/x1e80100.dtsi b/arch/arm64/boot/dts/qcom/x1e80100.dtsi > index 4936fa5b98ff..aad1153a443d 100644 > --- a/arch/arm64/boot/dts/qcom/x1e80100.dtsi > +++ b/arch/arm64/boot/dts/qcom/x1e80100.dtsi > @@ -4814,6 +4814,8 @@ usb_2_dwc3: usb@a200000 { > snps,dis-u1-entry-quirk; > snps,dis-u2-entry-quirk; > > + dma-coherent; > + Can someone from Qualcomm please confirm that this is correct, and that it's not the other way round and this property should be removed from the other controllers (e.g. if this was just some copy-pasta from sm8550 which is the only other Qualcomm platform that claims to have dma-coherent USB controllers). Johan
diff --git a/arch/arm64/boot/dts/qcom/x1e80100.dtsi b/arch/arm64/boot/dts/qcom/x1e80100.dtsi index 4936fa5b98ff..aad1153a443d 100644 --- a/arch/arm64/boot/dts/qcom/x1e80100.dtsi +++ b/arch/arm64/boot/dts/qcom/x1e80100.dtsi @@ -4814,6 +4814,8 @@ usb_2_dwc3: usb@a200000 { snps,dis-u1-entry-quirk; snps,dis-u2-entry-quirk; + dma-coherent; + ports { #address-cells = <1>; #size-cells = <0>;
Make this USB controller consistent with the others on this platform. Fixes: 4af46b7bd66f ("arm64: dts: qcom: x1e80100: Add USB nodes") Signed-off-by: Mark Kettenis <kettenis@openbsd.org> --- arch/arm64/boot/dts/qcom/x1e80100.dtsi | 2 ++ 1 file changed, 2 insertions(+)