On Saturday 31 August 2013 12:40 AM, Kumar Gala wrote: > On Aug 30, 2013, at 12:52 PM, Mugunthan V N wrote: > >> > DT node's unit address should be its own register offset address to make it a >> > unique across the system. This patch corrects the incorrect USB entries with >> > correct register offset for unit address. >> > >> > Cc: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> >> > Cc: Felipe Balbi <balbi@xxxxxx> >> > Signed-off-by: Mugunthan V N <mugunthanvnm@xxxxxx> >> > --- >> > arch/arm/boot/dts/am33xx.dtsi | 4 ++-- >> > 1 file changed, 2 insertions(+), 2 deletions(-) >> > >> > diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi >> > index f9c5da9..e9b6775 100644 >> > --- a/arch/arm/boot/dts/am33xx.dtsi >> > +++ b/arch/arm/boot/dts/am33xx.dtsi >> > @@ -346,7 +346,7 @@ >> > ti,hwmods = "usb_otg_hs"; >> > status = "disabled"; >> > >> > - ctrl_mod: control@44e10000 { >> > + ctrl_mod: control@44e10620 { >> > compatible = "ti,am335x-usb-ctrl-module"; >> > reg = <0x44e10620 0x10 >> > 0x44e10648 0x4>; >> > @@ -449,7 +449,7 @@ >> > "tx14", "tx15"; >> > }; >> > >> > - cppi41dma: dma-controller@07402000 { >> > + cppi41dma: dma-controller@47402000 { > Why isn't the unit address 47400000 ? > The above address is a shared address between CPPI dma and USB sub system. It has to be shared because CPPI DMA has to access Glue layer of the USB sub system for accessing IRQ registers. So 47400000 is already used in USB SS so if it cannot be used for CPPI DT node. Regards Mugunthan V N -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html