linux-kernel - Re: [PATCH v2 5/6] Revert "ARM: OMAP2+: Provide alias to USB PHY clock"

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20131208232258.GA29921@atomide.com>
Date:	Sun, 8 Dec 2013 15:22:58 -0800
From:	Tony Lindgren <tony@...mide.com>
To:	Roger Quadros <rogerq@...com>
Cc:	bcousson@...libre.com, balbi@...com, linux-usb@...r.kernel.org,
	linux-omap@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 5/6] Revert "ARM: OMAP2+: Provide alias to USB PHY
 clock"

* Roger Quadros <rogerq@...com> [131023 03:14]:
> On 10/22/2013 06:17 PM, Tony Lindgren wrote:
> > * Roger Quadros <rogerq@...com> [131010 02:23]:
> >> On 10/09/2013 06:15 PM, Tony Lindgren wrote:
> >>> * Roger Quadros <rogerq@...com> [131009 00:19]:
> >>>> Hi Tony,
> >>>>
> >>>> On 10/08/2013 01:06 PM, Roger Quadros wrote:
> >>>>> This reverts commit 741532c4a995be11815cb72d4d7a48f442a22fea.
> >>>>>
> >>>>> The proper clock reference is provided in device tree so we
> >>>>> no longer need this.
> >>>>
> >>>> Could you please Ack this one? I think it is best if it goes through Benoit's tree.
> >>>
> >>> I could queue this into my board removal series that will be
> >>> based on Benoit's branch if that works for you. And this
> >>> also seems to depend on the omap-for-v3.13/quirk branch
> >>> that moves all the legacy pdata handling into pdata-quirks.c.
> >>
> >> OK Tony. Thanks.
> > 
> > Actually, can you please update this patch against branch
> > omap-for-v3.13/board-removal-take2 and check the
> > omap5_uevm_legacy_init() part as well?
> 
> Updated patch is below. Note that this must go in only after Benoit has picked
> up Tero's clock series and the dts patches in this series.
> If needed, you can wait and I can send you a reminder when that happens. Thanks.

FYI, I'll mark this as read for now, can you please resend one more
time when we can apply after Tero's patches. Otherwise I'll probably
mess up things and apply this by accident too soon..

Regards,

Tony
 
> From ab81199d5b9c487c8493e6aa1d8b4bf17c0c5110 Mon Sep 17 00:00:00 2001
> From: Roger Quadros <rogerq@...com>
> Date: Wed, 23 Oct 2013 12:58:59 +0300
> Subject: [PATCH] ARM: OMAP2+: Get rid of legacy_init_ehci_clk()
> 
> The proper clock reference is provided in device tree so we
> no longer need this.
> 
> Signed-off-by: Roger Quadros <rogerq@...com>
> ---
>  arch/arm/mach-omap2/pdata-quirks.c | 25 -------------------------
>  1 file changed, 25 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/pdata-quirks.c b/arch/arm/mach-omap2/pdata-quirks.c
> index 10c7145..22537ed 100644
> --- a/arch/arm/mach-omap2/pdata-quirks.c
> +++ b/arch/arm/mach-omap2/pdata-quirks.c
> @@ -26,20 +26,6 @@ struct pdata_init {
>  	void (*fn)(void);
>  };
>  
> -/*
> - * Create alias for USB host PHY clock.
> - * Remove this when clock phandle can be provided via DT
> - */
> -static void __init __used legacy_init_ehci_clk(char *clkname)
> -{
> -	int ret;
> -
> -	ret = clk_add_alias("main_clk", NULL, clkname, NULL);
> -	if (ret)
> -		pr_err("%s:Failed to add main_clk alias to %s :%d\n",
> -		       __func__, clkname, ret);
> -}
> -
>  #if IS_ENABLED(CONFIG_WL12XX)
>  
>  static struct wl12xx_platform_data wl12xx __initdata;
> @@ -105,18 +91,10 @@ static void __init omap4_sdp_legacy_init(void)
>  static void __init omap4_panda_legacy_init(void)
>  {
>  	omap4_panda_display_init_of();
> -	legacy_init_ehci_clk("auxclk3_ck");
>  	legacy_init_wl12xx(WL12XX_REFCLOCK_38, 0, 53);
>  }
>  #endif
>  
> -#ifdef CONFIG_SOC_OMAP5
> -static void __init omap5_uevm_legacy_init(void)
> -{
> -	legacy_init_ehci_clk("auxclk1_ck");
> -}
> -#endif
> -
>  static struct pcs_pdata pcs_pdata;
>  
>  void omap_pcs_legacy_init(int irq, void (*rearm)(void))
> @@ -149,9 +127,6 @@ static struct pdata_init pdata_quirks[] __initdata = {
>  	{ "ti,omap4-sdp", omap4_sdp_legacy_init, },
>  	{ "ti,omap4-panda", omap4_panda_legacy_init, },
>  #endif
> -#ifdef CONFIG_SOC_OMAP5
> -	{ "ti,omap5-uevm", omap5_uevm_legacy_init, },
> -#endif
>  	{ /* sentinel */ },
>  };
>  
> -- 
> 1.8.3.2
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/