Re: [PATCH] staging: speakup: fix type mismatch warnings

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

 



Hello,

Greg KH, on Fri 11 Mar 2016 14:32:52 -0800, wrote:
> On Thu, Mar 10, 2016 at 08:21:35PM +0000, Okash Khawaja wrote:
> > Compiling speakup driver with sparse produces following warning:
> > 
> > drivers/staging/speakup/serialio.c:22:9: warning: incorrect type in
> > initializer (different base types)
> > drivers/staging/speakup/serialio.c:22:9:    expected unsigned int
> > [unsigned] flags
> > drivers/staging/speakup/serialio.c:22:9:    got restricted upf_t
> > 
> > This patch fixes it.
> > 
> > Signed-off-by: Okash Khawaja <okash.khawaja@xxxxxxxxx>

> > @@ -16,7 +17,7 @@ struct old_serial_port {
> >  	unsigned int baud_base;
> >  	unsigned int port;
> >  	unsigned int irq;
> > -	unsigned int flags; /* unused */
> > +	upf_t flags; /* unused */
> 
> I need some agreement from the speakup developers about this, I don't
> know if it's really needed or not...

It is not needed for functionality (upf_t is a typedef for unsigned
long), but in principle the field should be upf_t indeed (because it has
the __bitwise qualifier).

samuel
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux