[linux-yocto] [PATCH 03/18] arch/powerpc: Checking in correct USB entries to ACP3421 DTS

[Darren Hart]

On 2/26/14, 13:40, "Charlie Paul" <cpaul.windriver at gmail.com> wrote:

>From: SangeethaRao <sangeetha.rao at lsi.com>
>
>These changes were made to fix the failure in the USB.

What is the failure? How does it manifest?

>This change only affects the ACP34321 DTS
>
>Signed-off-by: SangeethaRao <sangeetha.rao at lsi.com>
>---
> arch/powerpc/boot/dts/acp342x.dts |   33
>+++++++++++++++++----------------
> 1 file changed, 17 insertions(+), 16 deletions(-)
>
>diff --git a/arch/powerpc/boot/dts/acp342x.dts
>b/arch/powerpc/boot/dts/acp342x.dts
>index b851788..f5cf3a6 100644
>--- a/arch/powerpc/boot/dts/acp342x.dts
>+++ b/arch/powerpc/boot/dts/acp342x.dts
>@@ -90,64 +90,65 @@
>                         clock-frequency = <0>; // Filled in by zImage
>                         UART0: serial at 00404000 {
>                                 device_type = "serial";
>-                                compatible = "acp-uart0";
>+                                compatible = "lsi,acp-uart0";

So you are adding "lsi" as a driver that can consume this DT. That should
be in the commit message for sure.

>
>                         USB0: usb at 004a4000 {
>                                 device_type = "usb";
>-                                compatible = "acp-usb";
>-                                enabled = <0>;
>-                                reg = <0x004a4000 0x00020000>;
>+                                compatible = "lsi,acp-usb";
>+                                enabled = <1>;
>+                                reg = <0x20 0x004A0000 0x0 0020000>,
>+					<0x20 0x0040C000 0x0 0001000>;

Does this impact the existing use cases for this dts? acp-usb for example?


-- 
Darren Hart
Yocto Project - Linux Kernel
Intel Open Source Technology Center