Hi Bingbu, On Mon, Nov 26, 2018 at 7:56 PM <bingbu.cao@xxxxxxxxx> wrote: > > From: Bingbu Cao <bingbu.cao@xxxxxxxxx> > > Some Sony camera sensors have same test pattern > definitions, this patch unify the pattern naming > to make it more clear to the userspace. > > Suggested-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > Signed-off-by: Bingbu Cao <bingbu.cao@xxxxxxxxx> > --- > drivers/media/i2c/imx258.c | 8 ++++---- > drivers/media/i2c/imx319.c | 8 ++++---- > drivers/media/i2c/imx355.c | 8 ++++---- > 3 files changed, 12 insertions(+), 12 deletions(-) > Thanks for the patch! One comment inline. > diff --git a/drivers/media/i2c/imx258.c b/drivers/media/i2c/imx258.c > index 31a1e2294843..a8a2880c6b4e 100644 > --- a/drivers/media/i2c/imx258.c > +++ b/drivers/media/i2c/imx258.c > @@ -504,10 +504,10 @@ struct imx258_mode { > > static const char * const imx258_test_pattern_menu[] = { > "Disabled", > - "Color Bars", > - "Solid Color", > - "Grey Color Bars", > - "PN9" > + "Solid Colour", > + "Eight Vertical Colour Bars", Is it just me or "solid color" and "color bars" are being swapped here? Did the driver had the names mixed up before or the order of modes is different between these sensors? Best regards, Tomasz