On Wed, 4 Sep 2024 11:35:55 +0000 wangshuaijie@xxxxxxxxxx wrote: > From: shuaijie wang <wangshuaijie@xxxxxxxxxx> > > AW96103 is a low power consumption capacitive touch and proximity controller. > Each channel can be independently config as sensor input, shield output. > > Channel Information: > aw96103: 3-channel > aw96105: 5-channel > > Signed-off-by: shuaijie wang <wangshuaijie@xxxxxxxxxx> Once a series had been applied, send patches on top. That means if you want to adopt Andy's suggestions then break it up into a series where each patch is a particular type of change. That allows rapid review in comparison with sending the thing all over again. As such I've just fixed the early return issue directly. Some of the changes would be rejected. > + > +#define FAR 0x00 > +#define TRIGGER_TH0 BIT(24) > +#define TRIGGER_TH1 (BIT(24) | BIT(16)) > +#define TRIGGER_TH2 (BIT(24) | BIT(16) | BIT(8)) > +#define TRIGGER_TH3 (BIT(24) | BIT(16) | BIT(8) | BIT(0)) > +#define TRIGGER_TH_MASK (BIT(24) | BIT(16) | BIT(8) | BIT(0)) No. If you want to use BIT then each BIT needs a meaningful define. If the bit's on their own convey no useful meaning then define these differently. Jonathan
