On Fri, 2022-11-25 at 00:38 +0100, Philipp Jungkamp wrote: > Hello, > > I'm sorry for the confusion caused by me sending revisions as replies. > I've now addresses the points raised. As I'm unfamiliar with mailing > lists > any recommendation on making the process more seamless is welcome. > > Thank you for your time! > For the series Acked-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx> > v2: > - Patch 1/4: > - Remove unnecessary comments. > - Change return type of functions to bool. (introduced an error, > fixed in v4) > - Swallows return value of a function. (fixed in v4) > - Patch 2/4: > - Create this 'noop' patch that just adds the LISS sensors to the > custom > sensor match table. > - Patch 3/4: > - Remove some unnecessary newlines in function calls. > - Patch 4/4: > - Allow for 1 byte human presence reports. > > v3: > - Patch 2/4 > - Add missing 'Signed-Off-By' > > v4: > - Patch 1/4: > - Fix return value and error codes on property query. > - Document and ignore return values for queries of optionally matched > properties. > - Don't modify *known in _get_known, return error when not matched. > - Remove comment on newly added sensors from commit message. > - Patch 4/4: > - Clean up switch statement. > > Regards, > Philipp Jungkamp > > Philipp Jungkamp (4): > HID: hid-sensor-custom: Allow more custom iio sensors > HID: hid-sensor-custom: Add LISS custom sensors > IIO: hid-sensor-als: Use generic usage > IIO: hid-sensor-prox: Use generic usage > > drivers/hid/hid-sensor-custom.c | 233 +++++++++++++++++++--------- > drivers/iio/light/hid-sensor-als.c | 27 ++-- > drivers/iio/light/hid-sensor-prox.c | 37 +++-- > include/linux/hid-sensor-ids.h | 1 + > 4 files changed, 199 insertions(+), 99 deletions(-) > > -- > 2.38.1 >