Re: [PATCH] omap2/3/4: serial: Half revert multiboot changes

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

 



* Aguirre, Sergio <saaguirre@xxxxxx> [100224 19:04]:
> 
> 
> > -----Original Message-----
> > From: Tony Lindgren [mailto:tony@xxxxxxxxxxx]
> > Sent: Wednesday, February 24, 2010 10:21 AM
> > To: Aguirre, Sergio
> > Cc: linux-omap@xxxxxxxxxxxxxxx; Sonasath, Moiz; Pandita, Vikram
> > Subject: Re: [PATCH] omap2/3/4: serial: Half revert multiboot changes
> > 
> > * Aguirre, Sergio <saaguirre@xxxxxx> [100223 23:37]:
> > > Hmm..
> > >
> > > I think I sent this patch too soon...
> > >
> > > Please ignore it, this is not a proper solution I feel..
> > >
> > > But what it is true... is that, patch "omap2/3/4: Fix mach-
> > omap2/serial.c for multiboot" is definitely breaking Zoom3 boot, and needs
> > to be fixed.
> > 
> > Yes sounds like only zoom has the external uart, other 3630
> > boards may be using the internal uart4.
> 
> Actually, I don't see any patch to handle uart4 interface/functional clocks, so I assume the addition of uart4 for 3630 chip is incomplete.
> 
> Shall we apply my 2nd patch, until proper clock handling is in place?
> 
> I don't have an omap4 to test, but I think my patch shouldn't hurt the initial intention to make multiomap builds possible... agree?

See also the recent patch from Thomas. I'd say let's rather fix this
properly with your clock related patchset once it's been tested.

We already have all these temporary hacks still in place for omap4,
let's try not to add more.

Regards,

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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux