RE: [PATCHv3 0/3] OMAP UART: Adding support for omap-serial driver.

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

 



> -----Original Message-----
> From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Olof Johansson
> Sent: Tuesday, November 24, 2009 10:52 PM
> To: G, Manjunath Kondaiah
> Cc: Govindraj; g@xxxxxxxxx; Tony Lindgren; linux-
> omap@xxxxxxxxxxxxxxx; 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:olof@xxxxxxxxx]
> > > Sent: Monday, November 23, 2009 10:36 PM
> > > To: Govindraj; g@xxxxxxxxx
> > > Cc: Tony Lindgren; linux-omap@xxxxxxxxxxxxxxx; 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 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