From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Jacques Hiblot Date: Tue, 15 May 2018 18:10:20 +0200 Subject: [U-Boot] [PATCH v5 4/9] board: ti: dra7xx-evm: turn on USB clocks in late init stage In-Reply-To: References: <1525444491-24324-1-git-send-email-jjhiblot@ti.com> <1525444491-24324-5-git-send-email-jjhiblot@ti.com> <94cee3cd-3e4a-9e7b-174d-0eba1f52b9c2@ti.com> <63cc1a70-416f-1198-b69a-8ba6da966077@denx.de> <170cd2a6-b143-aa78-872a-18f729855be4@ti.com> Message-ID: <5482bf79-ea10-9489-e987-35d7185420db@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de Hi Marek, On 04/05/2018 21:06, Marek Vasut wrote: >>>>> >>>>> Shouldnt the driver turn this on? >>>> AFAIK there is no clock driver for the OMAP. So we have to do it before >>>> the driver is probed. >>> Maybe it's time to implement it instead of piling up those hacks ? >> I was thinking along the same lines at some point but then I realized >> that it will make the SPL really big. And we are already tight on some >> platforms like am335x. > But this is not only SPL , right ? And yes, for SPL you might need to > poke registers or something. IMHO having 2 different ways to enable the clocks in SPL and in u-boot=20 is an not improvement over the current situation. In any case, This is a subject that can be discussed. Tom, Lokesh,=C2=A0 do you have an opinion on this ? Meanwhile It would nice to have DM_USB support for the DRA7/AM57=20 platforms. It allows using a USB ether adapter with CONFIG_DM_ETHER=20 enabled (default on TI's platform). And this series allows using the USB=20 3.0 on AM57 evm. JJ