Message ID | CAK8P3a3UuLFVxZv1o1+gKSafsGgV=vgWRM_yjnT_E4fAUPZx8g@mail.gmail.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Hi Arnd, On ven., juin 23 2017, Arnd Bergmann <arnd@arndb.de> wrote: > > > Please send a revert for that, and remove the code that tries to > parse the dma-mask from DT from that driver! I see on IRC you agree with Antoine about it the driver part. For the DT part do you want a PR with the revert or just the patch alone? Thanks, Gregory > > Arnd
On Fri, Jun 23, 2017 at 4:14 PM, Gregory CLEMENT <gregory.clement@free-electrons.com> wrote: > Hi Arnd, > > On ven., juin 23 2017, Arnd Bergmann <arnd@arndb.de> wrote: > > >> >> >> Please send a revert for that, and remove the code that tries to >> parse the dma-mask from DT from that driver! > > I see on IRC you agree with Antoine about it the driver part. For the DT > part do you want a PR with the revert or just the patch alone? Either way is fine, or just let me know (here or on IRC) if I should just revert the patch and add an explanation myself. Arnd
Hi Arnd, On ven., juin 23 2017, Arnd Bergmann <arnd@arndb.de> wrote: > On Fri, Jun 23, 2017 at 4:14 PM, Gregory CLEMENT > <gregory.clement@free-electrons.com> wrote: >> Hi Arnd, >> >> On ven., juin 23 2017, Arnd Bergmann <arnd@arndb.de> wrote: >> >> >>> >>> >>> Please send a revert for that, and remove the code that tries to >>> parse the dma-mask from DT from that driver! >> >> I see on IRC you agree with Antoine about it the driver part. For the DT >> part do you want a PR with the revert or just the patch alone? > > Either way is fine, or just let me know (here or on IRC) if I should > just revert the patch and add an explanation myself. If you can do it, it's better. You know better than me why we should revert this patch! :) Gregory > > Arnd > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi index 576e825585c9..e02ac90f7fdf 100644 --- a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi +++ b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi @@ -242,6 +242,7 @@ interrupt-names = "mem", "ring0", "ring1", "ring2", "ring3", "eip"; clocks = <&cpm_syscon0 1 26>; + dma-mask = <0xff 0xffffffff>; status = "disabled"; }; }; diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi index 797208a11f9d..7468fe00fc37 100644 --- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi +++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi @@ -232,6 +232,7 @@ interrupt-names = "mem", "ring0", "ring1", "ring2", "ring3", "eip"; clocks = <&cps_syscon0 1 26>; + dma-mask = <0xff 0xffffffff>; status = "disabled"; }; };