Re: [PATCH] clk: ti: fix dual-registration of uart4_ick

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Tero Kristo <t-kristo@xxxxxx> [151005 05:39]:
> On 10/05/2015 01:17 PM, Tony Lindgren wrote:
> >* Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx> [150929 06:14]:
> >>On 25/09/15 06:42, Tero Kristo wrote:
> >>>On 09/23/2015 08:30 PM, Tony Lindgren wrote:
> >>>>* Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx> [150923 07:53]:
> >>>>>On the OMAP AM3517 platform the uart4_ick gets registered
> >>>>>twice, causing any power managment to /dev/ttyO3 to fail
> >>>>>when trying to wake the device up.
> >>>>>
> >>>>>This solves the following oops:
> >>>>>
> >>>>>[] Unhandled fault: external abort on non-linefetch (0x1028) at
> >>>>>0xfa09e008
> >>>>>[] PC is at serial_omap_pm+0x48/0x15c
> >>>>>[] LR is at _raw_spin_unlock_irqrestore+0x30/0x5c
> >>>>
> >>>>No uart4 on 34xx/35xx, that got introduced with 36xx so:
> >>>>
> >>>>Acked-by: Tony Lindgren <tony@xxxxxxxxxxx>
> >>>>
> >>>>>Signed-off-by: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx>
> >>>>>---
> >>>>>   drivers/clk/ti/clk-3xxx.c | 1 -
> >>>>>   1 file changed, 1 deletion(-)
> >>>>>
> >>>>>diff --git a/drivers/clk/ti/clk-3xxx.c b/drivers/clk/ti/clk-3xxx.c
> >>>>>index 5e12f32..2e7ef1f 100644
> >>>>>--- a/drivers/clk/ti/clk-3xxx.c
> >>>>>+++ b/drivers/clk/ti/clk-3xxx.c
> >>>>>@@ -374,7 +374,6 @@ static struct ti_dt_clk omap3xxx_clks[] = {
> >>>>>       DT_CLK(NULL, "gpio2_ick", "gpio2_ick"),
> >>>>>       DT_CLK(NULL, "wdt3_ick", "wdt3_ick"),
> >>>>>       DT_CLK(NULL, "uart3_ick", "uart3_ick"),
> >>>>>-    DT_CLK(NULL, "uart4_ick", "uart4_ick"),
> >>>>>       DT_CLK(NULL, "gpt9_ick", "gpt9_ick"),
> >>>>>       DT_CLK(NULL, "gpt8_ick", "gpt8_ick"),
> >>>>>       DT_CLK(NULL, "gpt7_ick", "gpt7_ick"),
> >>>
> >>>NAK.
> >>>
> >>>This patch is actually only partially correct, it removes the uart4_ick
> >>>alias for omap36xx also. It should be added back to the omap36xx clks
> >>>array. Care to post a v2?
> >>
> >>Thankyou for the feedback. I'll update, but do not have a device to test.
> >
> >Hmm but isn't that entry already there for 36xx? So this just removes
> >a duplicate for 36xx.. Tero, can you please check again, maybe I'm
> >confused.
> 
> uart4_ick wasn't there, we only had uart4_fck listed for omap36xx.

OK thanks for checking.

Tony
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]