[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[Pedro Jardim]

Function init_usb_clk() is not used anywhere, so it can simply be deleted.
This was detected by the following sparse warning:

arch/arm/mach-imx/imx8m/clock_imx8mq.c:377:6: warning: no previous prototype
 for ‘init_usb_clk’ [-Wmissing-prototypes]
 void init_usb_clk(void)
      ^~~~~~~~~~~~

Signed-off-by: Pedro Jardim <jardim.c.pedro@gmail.com>
---
Chages since v1:

- Fixed patch corruption
 arch/arm/mach-imx/imx8m/clock_imx8mq.c | 22 ----------------------
 1 file changed, 22 deletions(-)

diff --git a/arch/arm/mach-imx/imx8m/clock_imx8mq.c b/arch/arm/mach-imx/imx8m/clock_imx8mq.c
index 878f2be166..663fe4d3ca 100644
--- a/arch/arm/mach-imx/imx8m/clock_imx8mq.c
+++ b/arch/arm/mach-imx/imx8m/clock_imx8mq.c
@@ -374,28 +374,6 @@ void init_wdog_clk(void)
 	clock_enable(CCGR_WDOG3, 1);
 }
 
-void init_usb_clk(void)
-{
-	if (!is_usb_boot()) {
-		clock_enable(CCGR_USB_CTRL1, 0);
-		clock_enable(CCGR_USB_CTRL2, 0);
-		clock_enable(CCGR_USB_PHY1, 0);
-		clock_enable(CCGR_USB_PHY2, 0);
-		/* 500MHz */
-		clock_set_target_val(USB_BUS_CLK_ROOT, CLK_ROOT_ON |
-				     CLK_ROOT_SOURCE_SEL(1));
-		/* 100MHz */
-		clock_set_target_val(USB_CORE_REF_CLK_ROOT, CLK_ROOT_ON |
-				     CLK_ROOT_SOURCE_SEL(1));
-		/* 100MHz */
-		clock_set_target_val(USB_PHY_REF_CLK_ROOT, CLK_ROOT_ON |
-				     CLK_ROOT_SOURCE_SEL(1));
-		clock_enable(CCGR_USB_CTRL1, 1);
-		clock_enable(CCGR_USB_CTRL2, 1);
-		clock_enable(CCGR_USB_PHY1, 1);
-		clock_enable(CCGR_USB_PHY2, 1);
-	}
-}
 
 void init_nand_clk(void)
 {
-- 
2.17.1

[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[Anatolij Gustschin]

On Thu, 23 Jan 2020 10:20:14 -0300
Pedro Jardim jardim.c.pedro at gmail.com wrote:

> Function init_usb_clk() is not used anywhere, so it can simply be deleted.
> This was detected by the following sparse warning:
> 
> arch/arm/mach-imx/imx8m/clock_imx8mq.c:377:6: warning: no previous prototype
>  for ‘init_usb_clk’ [-Wmissing-prototypes]
>  void init_usb_clk(void)

Cc'ing Peng. Maybe this was in preparation for adding USB support ?

--
Anatolij

[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[Fabio Estevam]

Hi Anatolij,

On Thu, Jan 23, 2020 at 11:43 AM Anatolij Gustschin <agust@denx.de> wrote:
>
> On Thu, 23 Jan 2020 10:20:14 -0300
> Pedro Jardim jardim.c.pedro at gmail.com wrote:
>
> > Function init_usb_clk() is not used anywhere, so it can simply be deleted.
> > This was detected by the following sparse warning:
> >
> > arch/arm/mach-imx/imx8m/clock_imx8mq.c:377:6: warning: no previous prototype
> >  for ‘init_usb_clk’ [-Wmissing-prototypes]
> >  void init_usb_clk(void)
>
> Cc'ing Peng. Maybe this was in preparation for adding USB support ?

Unfortunately i.MX8M USB support has been pending for a long time.

I prefer if we could get rid of this unused code and then bring this
back when USB support comes in.

[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[Peng Fan]

> Subject: Re: [PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()
> 
> Hi Anatolij,
> 
> On Thu, Jan 23, 2020 at 11:43 AM Anatolij Gustschin <agust@denx.de> wrote:
> >
> > On Thu, 23 Jan 2020 10:20:14 -0300
> > Pedro Jardim jardim.c.pedro at gmail.com wrote:
> >
> > > Function init_usb_clk() is not used anywhere, so it can simply be deleted.
> > > This was detected by the following sparse warning:
> > >
> > > arch/arm/mach-imx/imx8m/clock_imx8mq.c:377:6: warning: no previous
> > > prototype  for ‘init_usb_clk’ [-Wmissing-prototypes]  void
> > > init_usb_clk(void)
> >
> > Cc'ing Peng. Maybe this was in preparation for adding USB support ?
> 
> Unfortunately i.MX8M USB support has been pending for a long time.
> 
> I prefer if we could get rid of this unused code and then bring this back when
> USB support comes in.

Not sure whether TI guys have done the cadence USB support.

Previously Sherry did some usb work, but TI guys take over that.

After the usb IP support ready, I or someone could start support usb for i.MX8MQ.

For now, the function could be dropped.

Thanks,
Peng.

[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[Jon Nettleton]

On Mon, Jan 27, 2020 at 2:16 PM Peng Fan <peng.fan@nxp.com> wrote:
>
> > Subject: Re: [PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()
> >
> > Hi Anatolij,
> >
> > On Thu, Jan 23, 2020 at 11:43 AM Anatolij Gustschin <agust@denx.de> wrote:
> > >
> > > On Thu, 23 Jan 2020 10:20:14 -0300
> > > Pedro Jardim jardim.c.pedro at gmail.com wrote:
> > >
> > > > Function init_usb_clk() is not used anywhere, so it can simply be deleted.
> > > > This was detected by the following sparse warning:
> > > >
> > > > arch/arm/mach-imx/imx8m/clock_imx8mq.c:377:6: warning: no previous
> > > > prototype  for ‘init_usb_clk’ [-Wmissing-prototypes]  void
> > > > init_usb_clk(void)
> > >
> > > Cc'ing Peng. Maybe this was in preparation for adding USB support ?
> >
> > Unfortunately i.MX8M USB support has been pending for a long time.
> >
> > I prefer if we could get rid of this unused code and then bring this back when
> > USB support comes in.
>
> Not sure whether TI guys have done the cadence USB support.
>
> Previously Sherry did some usb work, but TI guys take over that.
>
> After the usb IP support ready, I or someone could start support usb for i.MX8MQ.
>
> For now, the function could be dropped.
>
> Thanks,
> Peng.

I have a working port of the Linux usb phy driver for the iMX8MQ
already.  I think the only piece that is missing to use dwc3-generic
for iMX8MQ is the common clk driver.  Has anyone started work on this?

-Jon

[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[Lokesh Vutla]

On 27/01/20 6:46 PM, Peng Fan wrote:
>> Subject: Re: [PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()
>>
>> Hi Anatolij,
>>
>> On Thu, Jan 23, 2020 at 11:43 AM Anatolij Gustschin <agust@denx.de> wrote:
>>>
>>> On Thu, 23 Jan 2020 10:20:14 -0300
>>> Pedro Jardim jardim.c.pedro at gmail.com wrote:
>>>
>>>> Function init_usb_clk() is not used anywhere, so it can simply be deleted.
>>>> This was detected by the following sparse warning:
>>>>
>>>> arch/arm/mach-imx/imx8m/clock_imx8mq.c:377:6: warning: no previous
>>>> prototype  for ‘init_usb_clk’ [-Wmissing-prototypes]  void
>>>> init_usb_clk(void)
>>>
>>> Cc'ing Peng. Maybe this was in preparation for adding USB support ?
>>
>> Unfortunately i.MX8M USB support has been pending for a long time.
>>
>> I prefer if we could get rid of this unused code and then bring this back when
>> USB support comes in.
> 
> Not sure whether TI guys have done the cadence USB support.

I guess you are talking about Cadence USB3 driver. Marek already merged the
support. Please find the corresponding commits below:

f69257baa8 usb: composite: add BOS descriptor support to composite framework
d80effb184 usb: gadget: Add gadget_is_cdns3() macro
a9ca4193bd usb: cdns3: Add TI wrapper driver for CDNS USB3 controller
927c22b0da usb: cdns3: gadget: Implement udc_set_speed() callback
7e91f6ccdc usb: Add Cadence USB3 host and gadget driver
8d94e184ff usb: udc: Introduce ->udc_set_speed() method
77dcbdf3c1 usb: gadget: Add match_ep() op to usb_gadget_ops
c93e305af7 bitmaps: import for_each_set_bit() macro
c37f594280 list: import list_first_entry_or_null()

Thanks and regards,
Lokesh

> 
> Previously Sherry did some usb work, but TI guys take over that.
> 
> After the usb IP support ready, I or someone could start support usb for i.MX8MQ.
> 
> For now, the function could be dropped.
> 
> Thanks,
> Peng.
> 

[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[Peng Fan]

> Subject: Re: [PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()
> 
> 
> 
> On 27/01/20 6:46 PM, Peng Fan wrote:
> >> Subject: Re: [PATCH v2 1/4] clock_imx8mq: Delete not used
> >> init_usb_clk()
> >>
> >> Hi Anatolij,
> >>
> >> On Thu, Jan 23, 2020 at 11:43 AM Anatolij Gustschin <agust@denx.de>
> wrote:
> >>>
> >>> On Thu, 23 Jan 2020 10:20:14 -0300
> >>> Pedro Jardim jardim.c.pedro at gmail.com wrote:
> >>>
> >>>> Function init_usb_clk() is not used anywhere, so it can simply be
> deleted.
> >>>> This was detected by the following sparse warning:
> >>>>
> >>>> arch/arm/mach-imx/imx8m/clock_imx8mq.c:377:6: warning: no
> previous
> >>>> prototype  for ‘init_usb_clk’ [-Wmissing-prototypes]  void
> >>>> init_usb_clk(void)
> >>>
> >>> Cc'ing Peng. Maybe this was in preparation for adding USB support ?
> >>
> >> Unfortunately i.MX8M USB support has been pending for a long time.
> >>
> >> I prefer if we could get rid of this unused code and then bring this
> >> back when USB support comes in.
> >
> > Not sure whether TI guys have done the cadence USB support.
> 
> I guess you are talking about Cadence USB3 driver. Marek already merged the
> support. Please find the corresponding commits below:
> 
> f69257baa8 usb: composite: add BOS descriptor support to composite
> framework
> d80effb184 usb: gadget: Add gadget_is_cdns3() macro a9ca4193bd usb:
> cdns3: Add TI wrapper driver for CDNS USB3 controller 927c22b0da usb:
> cdns3: gadget: Implement udc_set_speed() callback 7e91f6ccdc usb: Add
> Cadence USB3 host and gadget driver 8d94e184ff usb: udc: Introduce
> ->udc_set_speed() method
> 77dcbdf3c1 usb: gadget: Add match_ep() op to usb_gadget_ops
> c93e305af7 bitmaps: import for_each_set_bit() macro
> c37f594280 list: import list_first_entry_or_null()

Great.

Then I think better keep the function.

Thanks,
Peng.

> 
> Thanks and regards,
> Lokesh
> 
> >
> > Previously Sherry did some usb work, but TI guys take over that.
> >
> > After the usb IP support ready, I or someone could start support usb for
> i.MX8MQ.
> >
> > For now, the function could be dropped.
> >
> > Thanks,
> > Peng.
> >

[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[Fabio Estevam]

On Mon, Feb 3, 2020 at 5:48 AM Jon Nettleton <jon@solid-run.com> wrote:

> I have a working port of the Linux usb phy driver for the iMX8MQ
> already.  I think the only piece that is missing to use dwc3-generic
> for iMX8MQ is the common clk driver.  Has anyone started work on this?

Not that I am aware of.

Peng, do you know?

[PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()

[Peng Fan]

> Subject: Re: [PATCH v2 1/4] clock_imx8mq: Delete not used init_usb_clk()
> 
> On Mon, Feb 3, 2020 at 5:48 AM Jon Nettleton <jon@solid-run.com> wrote:
> 
> > I have a working port of the Linux usb phy driver for the iMX8MQ
> > already.  I think the only piece that is missing to use dwc3-generic
> > for iMX8MQ is the common clk driver.  Has anyone started work on this?
> 
> Not that I am aware of.
> 
> Peng, do you know?

I have not start to do that.

Jon, great to know that you did that. Look forward your patches in community.

Thanks,
Peng.