Hi Sakari & Jason, > -----Original Message----- > From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media- > owner@xxxxxxxxxxxxxxx] On Behalf Of Sakari Ailus > Sent: Wednesday, May 9, 2018 5:43 PM > To: Chen, JasonX Z <jasonx.z.chen@xxxxxxxxx> > Cc: Tomasz Figa <tfiga@xxxxxxxxxxxx>; Yeh, Andy <andy.yeh@xxxxxxxxx>; > Linux Media Mailing List <linux-media@xxxxxxxxxxxxxxx>; Chiang, AlanX > <alanx.chiang@xxxxxxxxx> > Subject: Re: [PATCH v11] media: imx258: Add imx258 camera sensor driver > > Hi Jason, > > On Wed, May 09, 2018 at 09:28:30AM +0000, Chen, JasonX Z wrote: > > Hello Tomasz > > > > >> +/* Test Pattern Control */ > > >> +#define IMX258_REG_TEST_PATTERN 0x0600 > > >> +#define IMX258_TEST_PATTERN_DISABLE 0 > > >> +#define IMX258_TEST_PATTERN_SOLID_COLOR 1 > > >> +#define IMX258_TEST_PATTERN_COLOR_BARS 2 #define > > >> +IMX258_TEST_PATTERN_GREY_COLOR 3 > > >> +#define IMX258_TEST_PATTERN_PN9 4 I suppose we only use IMX258_TEST_PATTERN_COLOR_BARS. I heard that we'd better remove the functions/code no one would use. Is that true? e.g. remove all h_flip and v_flip ioctls because it's not used by anyone. Best Regards, Jianxu(Clive) Zheng
