On 05/06/2024 11:11, wangshuaijie@xxxxxxxxxx wrote: > From: shuaijie wang <wangshuaijie@xxxxxxxxxx> > No commit msg, no proper subject prefix. > Signed-off-by: shuaijie wang <wangshuaijie@xxxxxxxxxx> > | Reported-by: kernel test robot <lkp@xxxxxxxxx> > | Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > | Reported-by: Dan Carpenter <error27@xxxxxxxxx> Drop all these, this is some bogus tags. Tag never starts with pipe, btw. > --- > drivers/input/misc/aw_sar/aw9610x/aw9610x.c | 884 ++++++++++++++++++++ > drivers/input/misc/aw_sar/aw9610x/aw9610x.h | 327 ++++++++ > 2 files changed, 1211 insertions(+) > create mode 100644 drivers/input/misc/aw_sar/aw9610x/aw9610x.c > create mode 100644 drivers/input/misc/aw_sar/aw9610x/aw9610x.h ... > +struct aw_reg_data { > + unsigned char rw; > + unsigned short reg; > +}; > +/******************************************** > + * Register Access > + *******************************************/ > +#define REG_NONE_ACCESS (0) > +#define REG_RD_ACCESS (1 << 0) > +#define REG_WR_ACCESS (1 << 1) > + > +static const struct aw_reg_data g_aw9610x_reg_access[] = { Ehh... so we are at basics of C. Data structures do not go to headers. Best regards, Krzysztof