On Thu, Feb 1, 2024 at 3:53 AM Jiri Slaby (SUSE) <jirislaby@xxxxxxxxxx> wrote: > > The pressure member in struct hid_data is unused. It was > added in commit 5ae6e89f7409 (HID: wacom: implement the finger part of > the HID generic handling), but never used. > > As this is not a struct to communicate with the HW, remove that member. > > Found by https://github.com/jirislaby/clang-struct. > > Signed-off-by: Jiri Slaby (SUSE) <jirislaby@xxxxxxxxxx> Wacom current touch devices do not support pressure. We could add pressure back when it is necessary in the future. Thank you Jiri. Acked-by: Ping Cheng <ping.cheng@xxxxxxxxx> Cheers, Ping > Cc: Ping Cheng <ping.cheng@xxxxxxxxx> > Cc: Jason Gerecke <jason.gerecke@xxxxxxxxx> > Cc: Jiri Kosina <jikos@xxxxxxxxxx> > Cc: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx> > --- > drivers/hid/wacom_wac.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/hid/wacom_wac.h b/drivers/hid/wacom_wac.h > index e63b1e806e34..6ec499841f70 100644 > --- a/drivers/hid/wacom_wac.h > +++ b/drivers/hid/wacom_wac.h > @@ -309,7 +309,6 @@ struct hid_data { > bool confidence; > int x; > int y; > - int pressure; > int width; > int height; > int id; > -- > 2.43.0 > >
