From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Murphy Date: Thu, 11 Jul 2013 08:48:29 -0500 Subject: [U-Boot] [[PATCH v2 3/6] ARM: OMAP5: USB: Add OMAP5 common USB EHCI information In-Reply-To: <51DE7B6A.1070708@ti.com> References: <1373486709-13015-1-git-send-email-dmurphy@ti.com> <1373486709-13015-4-git-send-email-dmurphy@ti.com> <51DE2BDF.5010509@ti.com> <51DE658D.6040806@ti.com> <51DE6E47.20207@ti.com> <51DE731F.6030706@ti.com> <51DE7B6A.1070708@ti.com> Message-ID: <51DEB7AD.2050301@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 07/11/2013 04:31 AM, Sricharan R wrote: > On Thursday 11 July 2013 02:25 PM, Roger Quadros wrote: >> On 07/11/2013 11:35 AM, Sricharan R wrote: >>> On Thursday 11 July 2013 01:28 PM, Roger Quadros wrote: >>>> On 07/11/2013 06:51 AM, Lokesh Vutla wrote: >>>>> On Thursday 11 July 2013 01:35 AM, Dan Murphy wrote: >>>>>> * Enable the OMAP5 EHCI host clocks >>>>>> * Add OMAP5 EHCI register definitions >>>>>> * Add OMAP5 ES2 host revision >>>>>> >>>>>> Signed-off-by: Dan Murphy >>>>>> --- >>>>>> arch/arm/cpu/armv7/omap5/hw_data.c | 13 ++++++++++ >>>>>> arch/arm/include/asm/arch-omap5/clock.h | 6 +++++ >>>>>> arch/arm/include/asm/arch-omap5/ehci.h | 43 +++++++++++++++++++++++++++++++ >>>>>> arch/arm/include/asm/ehci-omap.h | 1 + >>>>>> drivers/usb/host/ehci-omap.c | 2 +- >>>>>> 5 files changed, 64 insertions(+), 1 deletion(-) >>>>>> create mode 100644 arch/arm/include/asm/arch-omap5/ehci.h >>>>>> >>>>>> diff --git a/arch/arm/cpu/armv7/omap5/hw_data.c b/arch/arm/cpu/armv7/omap5/hw_data.c >>>>>> index 56cf1f8..055f058 100644 >>>>>> --- a/arch/arm/cpu/armv7/omap5/hw_data.c >>>>>> +++ b/arch/arm/cpu/armv7/omap5/hw_data.c >>>>>> @@ -412,6 +412,8 @@ void enable_basic_clocks(void) >>>>>> (*prcm)->cm_l4per_gpio4_clkctrl, >>>>>> (*prcm)->cm_l4per_gpio5_clkctrl, >>>>>> (*prcm)->cm_l4per_gpio6_clkctrl, >>>>>> + (*prcm)->cm_clksel_usb_60mhz, >>>>>> + (*prcm)->cm_l3init_hsusbtll_clkctrl, >>>>> guard this with CONFIG_USB_EHCI please or it ll >>>>> throw an error for DRA7xx boards. >>>> why is DRA7xx using omap5/hw_data.c? >>>> >>>> doesn't it qualify for its own SoC directory? >>> We tried to keep common things for OMAP5/DRA intact and >>> added the difference. The above clocks list was same for both. >>> In fact there is no armv7/dra directory at all. >> If there is no directory, it could be created I suppose. >> IMHO it would become ugly soon if it doesn't have its own hw_data. > I am not much against it, it might look clean but would result in some code > duplication. I feel we should do it if we add another DRA variant. > > Regards, > Sricharan For now I will wrap it in CONFIG_USB_EHCI_OMAP until the dra7xx is figured out. +1 to a separate SoC directory for dra7xx Dan -- ------------------ Dan Murphy