Re: [PATCH V4] tty/serial: add support for Xilinx PS UART

[Greg KH <greg@kroah.com>]

On Fri, Apr 29, 2011 at 10:02:39PM +0000, johnlinn@comcast.net wrote:
> 
> > On Wed, Apr 27, 2011 at 12:17:36PM -0600, johnlinn@comcast.net wrote:
> > > From: John Linn <john.linn@xilinx.com>
> > > 
> > > The Xilinx PS Uart is used on the new ARM based SoC. This
> > > UART is not compatible with others such that a seperate
> > > driver is required.
> > > 
> > > Signed-off-by: John Linn <john.linn@xilinx.com>
> > 
> > I get build failures with this patch:
> > 
> >   CC [M]  drivers/tty/serial/xilinx_uartps.o
> > drivers/tty/serial/xilinx_uartps.c: In function ‘xuartps_get_port’:
> > drivers/tty/serial/xilinx_uartps.c:785:14: error: ‘NO_IRQ’ undeclared
> > (first use in this function)
> > drivers/tty/serial/xilinx_uartps.c:785:14: note: each undeclared
> > identifier is reported only once for each function it appears in
> > make[1]: *** [drivers/tty/serial/xilinx_uartps.o] Error 1
> > 
> > Care to fix this up so it's in a state I can properly apply it to the
> > tree?
> > 
> > thanks,
> > 
> > greg k-h
> 
> Done and sent out as V5 of patch.
> 
> Just for future reference (and maybe I should know this).  How should I 
> have prevented this problem? I guess I would need to build the kernel 
> for other architectures and I don't normally do that.

Yes, you should at the very least, build the code for x86 as that's what
we use for development and applying the patches on.

You can use cross-compliers to build for other architectures as well if
you want.

thanks,

greg k-h