2024-06-07 11:43:18

by Yasin Lee

[permalink] [raw]
Subject: [PATCH v4 0/2] iio:proximity:hx9023s: Add TYHX HX9023S sensor driver

From: Yasin Lee <[email protected]>

v4:
- Removed hardware-irrelevant properties from dt-bindings, retaining only channel configuration
related `channel-positive` and `channel-negative`. Grouped by channel.
Retained `channel-in-use` as it is hardware-related.
- Removed redundant register definitions.
- Reorganized `struct hx9023s_data`, extracting channel-related attributes
into a new `struct hx9023s_ch_data`.
- Optimized bit operation related code.
- Replaced `of_` versions with generic firmware parsing functions.
- Fixed other issues mentioned in the email feedback.

Link: https://lore.kernel.org/linux-iio/20240602152638.2c674930@jic23-huawei/

Yasin Lee (2):
dt-bindings:iio:proximity: Add hx9023s binding
iio:proximity:hx9023s: Add TYHX HX9023S sensor driver

.../bindings/iio/proximity/tyhx,hx9023s.yaml | 103 ++
.../devicetree/bindings/vendor-prefixes.yaml | 2 +
drivers/iio/proximity/Kconfig | 14 +
drivers/iio/proximity/Makefile | 1 +
drivers/iio/proximity/hx9023s.c | 1162 +++++++++++++++++
5 files changed, 1282 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/proximity/tyhx,hx9023s.yaml
create mode 100644 drivers/iio/proximity/hx9023s.c

--
2.25.1



2024-06-08 16:51:17

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH v4 0/2] iio:proximity:hx9023s: Add TYHX HX9023S sensor driver

On Fri, 7 Jun 2024 19:41:36 +0800
Yasin Lee <[email protected]> wrote:

> From: Yasin Lee <[email protected]>
>
> v4:
> - Removed hardware-irrelevant properties from dt-bindings, retaining only channel configuration
> related `channel-positive` and `channel-negative`. Grouped by channel.
> Retained `channel-in-use` as it is hardware-related.
> - Removed redundant register definitions.
> - Reorganized `struct hx9023s_data`, extracting channel-related attributes
> into a new `struct hx9023s_ch_data`.
> - Optimized bit operation related code.
> - Replaced `of_` versions with generic firmware parsing functions.
> - Fixed other issues mentioned in the email feedback.
>
> Link: https://lore.kernel.org/linux-iio/20240602152638.2c674930@jic23-huawei/

Hi Yasin,

Please fix your patch threading if possible. If it's corporate email getting in
the way, take a look at the b4 tool. It has a nice trick of being able to use
a web gateway and that tends to get out of almost all company networks.

Jonathan

>
> Yasin Lee (2):
> dt-bindings:iio:proximity: Add hx9023s binding
> iio:proximity:hx9023s: Add TYHX HX9023S sensor driver
>
> .../bindings/iio/proximity/tyhx,hx9023s.yaml | 103 ++
> .../devicetree/bindings/vendor-prefixes.yaml | 2 +
> drivers/iio/proximity/Kconfig | 14 +
> drivers/iio/proximity/Makefile | 1 +
> drivers/iio/proximity/hx9023s.c | 1162 +++++++++++++++++
> 5 files changed, 1282 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/iio/proximity/tyhx,hx9023s.yaml
> create mode 100644 drivers/iio/proximity/hx9023s.c
>