> -----Original Message----- > From: [email protected] [mailto:linux-omap- > [email protected]] On Behalf Of Olof Johansson > Sent: Tuesday, November 24, 2009 10:52 PM > To: G, Manjunath Kondaiah > Cc: Govindraj; [email protected]; Tony Lindgren; linux- > [email protected]; Pandita, Vikram; Raja, Govindraj > Subject: Re: [PATCHv3 0/3] OMAP UART: Adding support for omap-serial > driver. > > On Tue, Nov 24, 2009 at 03:04:09PM +0530, G, Manjunath Kondaiah > wrote: > > > > > > > -----Original Message----- > > > From: Olof Johansson [mailto:[email protected]] > > > Sent: Monday, November 23, 2009 10:36 PM > > > To: Govindraj; [email protected] > > > Cc: Tony Lindgren; [email protected]; Pandita, > > > Vikram; Raja, Govindraj; G, Manjunath Kondaiah > > > Subject: Re: [PATCHv3 0/3] OMAP UART: Adding support for > > > omap-serial driver. > > > > > > On Fri, Nov 20, 2009 at 03:21:26PM +0530, Govindraj wrote: > > > > I have not seen any comments on this patch series yet. > > > > > > > > Tony, > > > > Is there something I need to modify? > > > > > > Govindaj, > > > > > > I'm having problems with these patches on Zoom2 with debug > > > board, since > > > they don't seem to coexist very well together with 8250. > Enabling both > > > options will result in build errors. > > > > Why do we need to enable both 8250 and omap-serial for console. > The console > > UART can be either external UART or any one of the omap UART > right? > > You should allow both of them to be enabled at the same time, so the > same > kernel can for example be booted on a ZOOM2 with debug board > attached > (8250 on GPMC), or on a beagle/overo board. > > Making them exclusive would be a step backwards.
More so, selecting both config option really shouldn't break the build. If one expect a build break because only one of the options can be enabled then clearly the dependency should be used so that user can select only one option at a time. Regards, Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
