[v4] iio: accel: mc3230: add mount matrix, of match and mc3510c support

[PATCH RESEND v4 0/5] iio: accel: mc3230: add mount matrix, of match and mc3510c support

Posted by Vasiliy Doylov via B4 Relay 1 year ago

Changes includes:
- Add mount matrix handling
- Add match table to work with DT
- Add MC3510C support

MC3510C use same registors as MC3230, but different value scale.

Signed-off-by: Vasiliy Doylov <nekodevelopper@gmail.com>
---
Changes in v4:
- Fixed commit messages
- Added const modificator to chip_infos
- Return name->chip_id->product_code->scale order
- `{}` replaced by `{ }` (with space)
- Link to v3: https://lore.kernel.org/r/20250112-mainlining-mc3510c-v3-0-9ee6520ab69d@gmail.com

Changes in v2:
- Ordered commits
- Fixed comment style
- Ordered struct members
- Fixed device table
- MC5310C commit splitted
- Link to v1: https://lore.kernel.org/r/20250111-mainlining-mc3510c-v1-0-57be503addf8@gmail.com

Changes in v3:
- Fixed patch (poped stash)
- Link to v2: https://lore.kernel.org/all/20250112-mainlining-mc3510c-v2-0-322804a545cf@gmail.com

---
Vasiliy Doylov (5):
      dt-bindings: iio: accel: mc3230: document mc3510c
      iio: accel: mc3230: add mount matrix support
      iio: accel: mc3230: add OF match table
      iio: accel: mc3230: add multiple devices support
      iio: accel: mc3230: add mc3510c support

 .../devicetree/bindings/trivial-devices.yaml       |  2 +
 drivers/iio/accel/mc3230.c                         | 95 ++++++++++++++++++----
 2 files changed, 80 insertions(+), 17 deletions(-)
---
base-commit: 2b88851f583d3c4e40bcd40cfe1965241ec229dd
change-id: 20250111-mainlining-mc3510c-564702fba487

Best regards,
-- 
Vasiliy Doylov <nekodevelopper@gmail.com>

Re: [PATCH RESEND v4 0/5] iio: accel: mc3230: add mount matrix, of match and mc3510c support

Posted by Jonathan Cameron 1 year ago

On Thu, 16 Jan 2025 16:52:41 +0300
Vasiliy Doylov via B4 Relay <devnull+nekodevelopper.gmail.com@kernel.org> wrote:

Always say why you are resending.

Jonathan

> Changes includes:
> - Add mount matrix handling
> - Add match table to work with DT
> - Add MC3510C support
> 
> MC3510C use same registors as MC3230, but different value scale.
> 
> Signed-off-by: Vasiliy Doylov <nekodevelopper@gmail.com>
> ---
> Changes in v4:
> - Fixed commit messages
> - Added const modificator to chip_infos
> - Return name->chip_id->product_code->scale order
> - `{}` replaced by `{ }` (with space)
> - Link to v3: https://lore.kernel.org/r/20250112-mainlining-mc3510c-v3-0-9ee6520ab69d@gmail.com
> 
> Changes in v2:
> - Ordered commits
> - Fixed comment style
> - Ordered struct members
> - Fixed device table
> - MC5310C commit splitted
> - Link to v1: https://lore.kernel.org/r/20250111-mainlining-mc3510c-v1-0-57be503addf8@gmail.com
> 
> Changes in v3:
> - Fixed patch (poped stash)
> - Link to v2: https://lore.kernel.org/all/20250112-mainlining-mc3510c-v2-0-322804a545cf@gmail.com
> 
> ---
> Vasiliy Doylov (5):
>       dt-bindings: iio: accel: mc3230: document mc3510c
>       iio: accel: mc3230: add mount matrix support
>       iio: accel: mc3230: add OF match table
>       iio: accel: mc3230: add multiple devices support
>       iio: accel: mc3230: add mc3510c support
> 
>  .../devicetree/bindings/trivial-devices.yaml       |  2 +
>  drivers/iio/accel/mc3230.c                         | 95 ++++++++++++++++++----
>  2 files changed, 80 insertions(+), 17 deletions(-)
> ---
> base-commit: 2b88851f583d3c4e40bcd40cfe1965241ec229dd
> change-id: 20250111-mainlining-mc3510c-564702fba487
> 
> Best regards,

Re: [PATCH RESEND v4 0/5] iio: accel: mc3230: add mount matrix, of match and mc3510c support

Posted by Jonathan Cameron 1 year ago

On Thu, 16 Jan 2025 16:42:11 +0000
Jonathan Cameron <Jonathan.Cameron@huawei.com> wrote:

> On Thu, 16 Jan 2025 16:52:41 +0300
> Vasiliy Doylov via B4 Relay <devnull+nekodevelopper.gmail.com@kernel.org> wrote:
> 
> Always say why you are resending.

Other than that, looks good and you seem to have addressed all
comments on v3, so applied to the testing branch of iio.git.
Note I'll be rebasing that on rc1 once available and only then pushing
it out as togreg.  In the meantime it will get some build bot coverage.

Thanks,

Jonathan

> 
> Jonathan
> 
> > Changes includes:
> > - Add mount matrix handling
> > - Add match table to work with DT
> > - Add MC3510C support
> > 
> > MC3510C use same registors as MC3230, but different value scale.
> > 
> > Signed-off-by: Vasiliy Doylov <nekodevelopper@gmail.com>
> > ---
> > Changes in v4:
> > - Fixed commit messages
> > - Added const modificator to chip_infos
> > - Return name->chip_id->product_code->scale order
> > - `{}` replaced by `{ }` (with space)
> > - Link to v3: https://lore.kernel.org/r/20250112-mainlining-mc3510c-v3-0-9ee6520ab69d@gmail.com
> > 
> > Changes in v2:
> > - Ordered commits
> > - Fixed comment style
> > - Ordered struct members
> > - Fixed device table
> > - MC5310C commit splitted
> > - Link to v1: https://lore.kernel.org/r/20250111-mainlining-mc3510c-v1-0-57be503addf8@gmail.com
> > 
> > Changes in v3:
> > - Fixed patch (poped stash)
> > - Link to v2: https://lore.kernel.org/all/20250112-mainlining-mc3510c-v2-0-322804a545cf@gmail.com
> > 
> > ---
> > Vasiliy Doylov (5):
> >       dt-bindings: iio: accel: mc3230: document mc3510c
> >       iio: accel: mc3230: add mount matrix support
> >       iio: accel: mc3230: add OF match table
> >       iio: accel: mc3230: add multiple devices support
> >       iio: accel: mc3230: add mc3510c support
> > 
> >  .../devicetree/bindings/trivial-devices.yaml       |  2 +
> >  drivers/iio/accel/mc3230.c                         | 95 ++++++++++++++++++----
> >  2 files changed, 80 insertions(+), 17 deletions(-)
> > ---
> > base-commit: 2b88851f583d3c4e40bcd40cfe1965241ec229dd
> > change-id: 20250111-mainlining-mc3510c-564702fba487
> > 
> > Best regards,  
> 
>