Messages in this thread | | | Date | Thu, 3 Nov 2022 01:40:54 +0100 | From | Greg KH <> | Subject | Re: [PATCH] staging: media: atomisp: make hive_int8 explictly signed |
| |
On Thu, Nov 03, 2022 at 01:19:14AM +0100, Jason A. Donenfeld wrote: > The current definition of hive_int8 is a naked char, without any sign > specifier. This is incorrect on platforms such as arm, where char is > unsigned. Fortunately nothing in the kernel actually uses a hive_int8 > type, but in case it gets used later rather than removed, this makes it > explicitly signed. > > Cc: Mauro Carvalho Chehab <mchehab@kernel.org> > Cc: Sakari Ailus <sakari.ailus@linux.intel.com> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> > --- > Greg - if you're going to take this for 6.1, that's fine with me. > Otherwise, if it's for 6.2, I'll take this in my unsigned-char tree to > keep all of these fixups together. -Jason
I don't take drivers/staging/media/* patches, they go through Mauro's tree, so I'll let him and you fight it out here :)
thanks,
greg k-h
| |