mbox series

[v8,0/7] iio: add new backend framework

Message ID 20240202-iio-backend-v8-0-f65ee8c8203d@analog.com (mailing list archive)
Headers show
Series iio: add new backend framework | expand

Message

Nuno Sa via B4 Relay Feb. 2, 2024, 3:08 p.m. UTC
v1:
 https://lore.kernel.org/linux-iio/20231204144925.4fe9922f@jic23-huawei/T/#m222f5175273b81dbfe40b7f0daffcdc67d6cb8ff

v2:
 https://lore.kernel.org/r/20231208-dev-iio-backend-v2-0-5450951895e1@analog.com

v3:
 https://lore.kernel.org/linux-iio/20231213-dev-iio-backend-v3-0-bb9f12a5c6dc@analog.com/

v4:
 https://lore.kernel.org/r/20231220-iio-backend-v4-0-998e9148b692@analog.com

v5:
 https://lore.kernel.org/r/20240112-iio-backend-v5-0-bdecad041ab4@analog.com

v6:
 https://lore.kernel.org/r/20240119-iio-backend-v6-0-189536c35a05@analog.com

v7
 https://lore.kernel.org/r/20240123-iio-backend-v7-0-1bff236b8693@analog.com

Changes in v8:
 - Dropped commit ("of: property: fix typo in io-channels") - applied
   via DT tree. Also dropped commit
   ("driver: core: allow modifying device_links flags") - did not made
   sense.
 - Patch 7
  * Do not change the version string format during probe.

Jonathan, the series is based on next-20240202 since it already includes
the io-channels fix Rob applied in his tree. I guess it should land in rc3 so
after you rebase, all patches should apply cleanly (if applying them of course
:)). Let me know if anything fails...

(also dropped the devlink Reviewers from the Cc list as that patch was
dropped).

Keeping the block diagram  so we don't have to follow links
to check one of the typical setups.

                                           -------------------------------------------------------
 ------------------                        | -----------         ------------      -------  FPGA |
 |     ADC        |------------------------| | AXI ADC |---------| DMA CORE |------| RAM |       |
 | (Frontend/IIO) | Serial Data (eg: LVDS) | |(backend)|---------|          |------|     |       |
 |                |------------------------| -----------         ------------      -------       |
 ------------------                        -------------------------------------------------------

---
Nuno Sa (6):
      dt-bindings: adc: ad9467: add new io-backend property
      dt-bindings: adc: axi-adc: update bindings for backend framework
      iio: buffer-dmaengine: export buffer alloc and free functions
      iio: add the IIO backend framework
      iio: adc: ad9467: convert to backend framework
      iio: adc: adi-axi-adc: move to backend framework

Olivier Moysan (1):
      of: property: add device link support for io-backends

 .../devicetree/bindings/iio/adc/adi,ad9467.yaml    |   4 +
 .../devicetree/bindings/iio/adc/adi,axi-adc.yaml   |   8 +-
 MAINTAINERS                                        |   8 +
 drivers/iio/Kconfig                                |   9 +
 drivers/iio/Makefile                               |   1 +
 drivers/iio/adc/Kconfig                            |   4 +-
 drivers/iio/adc/ad9467.c                           | 268 +++++++++-----
 drivers/iio/adc/adi-axi-adc.c                      | 379 +++++--------------
 drivers/iio/buffer/industrialio-buffer-dmaengine.c |   8 +-
 drivers/iio/industrialio-backend.c                 | 412 +++++++++++++++++++++
 drivers/of/property.c                              |   2 +
 include/linux/iio/adc/adi-axi-adc.h                |  68 ----
 include/linux/iio/backend.h                        |  72 ++++
 include/linux/iio/buffer-dmaengine.h               |   3 +
 14 files changed, 793 insertions(+), 453 deletions(-)
---
base-commit: 076d56d74f17e625b3d63cf4743b3d7d02180379
change-id: 20231219-iio-backend-a3dc1a6a7a58
--

Thanks!
- Nuno Sá

Comments

Jonathan Cameron Feb. 4, 2024, 2:59 p.m. UTC | #1
On Fri, 02 Feb 2024 16:08:31 +0100
Nuno Sa via B4 Relay <devnull+nuno.sa.analog.com@kernel.org> wrote:

> v1:
>  https://lore.kernel.org/linux-iio/20231204144925.4fe9922f@jic23-huawei/T/#m222f5175273b81dbfe40b7f0daffcdc67d6cb8ff
> 
> v2:
>  https://lore.kernel.org/r/20231208-dev-iio-backend-v2-0-5450951895e1@analog.com
> 
> v3:
>  https://lore.kernel.org/linux-iio/20231213-dev-iio-backend-v3-0-bb9f12a5c6dc@analog.com/
> 
> v4:
>  https://lore.kernel.org/r/20231220-iio-backend-v4-0-998e9148b692@analog.com
> 
> v5:
>  https://lore.kernel.org/r/20240112-iio-backend-v5-0-bdecad041ab4@analog.com
> 
> v6:
>  https://lore.kernel.org/r/20240119-iio-backend-v6-0-189536c35a05@analog.com
> 
> v7
>  https://lore.kernel.org/r/20240123-iio-backend-v7-0-1bff236b8693@analog.com
> 
> Changes in v8:
>  - Dropped commit ("of: property: fix typo in io-channels") - applied
>    via DT tree. Also dropped commit
>    ("driver: core: allow modifying device_links flags") - did not made
>    sense.
>  - Patch 7
>   * Do not change the version string format during probe.
> 
> Jonathan, the series is based on next-20240202 since it already includes
> the io-channels fix Rob applied in his tree. I guess it should land in rc3 so
> after you rebase, all patches should apply cleanly (if applying them of course
> :)). Let me know if anything fails...

Given that merge (between my tree and Rob's) is about as trivial as they come
I'll take the series now (rather than delaying) and rely on Stephen (for next)
and Greg (once I send a pull request) to deal with it.

It's the sort of merge that makes Linus grumpy if people rebase to avoid it!

On that note, applied with those tweaks to the broken binding patch.

Initially pushed out as testing to see if 0-day moans at us a lot about anything.

Jonathan

> 
> (also dropped the devlink Reviewers from the Cc list as that patch was
> dropped).
> 
> Keeping the block diagram  so we don't have to follow links
> to check one of the typical setups.
> 
>                                            -------------------------------------------------------
>  ------------------                        | -----------         ------------      -------  FPGA |
>  |     ADC        |------------------------| | AXI ADC |---------| DMA CORE |------| RAM |       |
>  | (Frontend/IIO) | Serial Data (eg: LVDS) | |(backend)|---------|          |------|     |       |
>  |                |------------------------| -----------         ------------      -------       |
>  ------------------                        -------------------------------------------------------
> 
> ---
> Nuno Sa (6):
>       dt-bindings: adc: ad9467: add new io-backend property
>       dt-bindings: adc: axi-adc: update bindings for backend framework
>       iio: buffer-dmaengine: export buffer alloc and free functions
>       iio: add the IIO backend framework
>       iio: adc: ad9467: convert to backend framework
>       iio: adc: adi-axi-adc: move to backend framework
> 
> Olivier Moysan (1):
>       of: property: add device link support for io-backends
> 
>  .../devicetree/bindings/iio/adc/adi,ad9467.yaml    |   4 +
>  .../devicetree/bindings/iio/adc/adi,axi-adc.yaml   |   8 +-
>  MAINTAINERS                                        |   8 +
>  drivers/iio/Kconfig                                |   9 +
>  drivers/iio/Makefile                               |   1 +
>  drivers/iio/adc/Kconfig                            |   4 +-
>  drivers/iio/adc/ad9467.c                           | 268 +++++++++-----
>  drivers/iio/adc/adi-axi-adc.c                      | 379 +++++--------------
>  drivers/iio/buffer/industrialio-buffer-dmaengine.c |   8 +-
>  drivers/iio/industrialio-backend.c                 | 412 +++++++++++++++++++++
>  drivers/of/property.c                              |   2 +
>  include/linux/iio/adc/adi-axi-adc.h                |  68 ----
>  include/linux/iio/backend.h                        |  72 ++++
>  include/linux/iio/buffer-dmaengine.h               |   3 +
>  14 files changed, 793 insertions(+), 453 deletions(-)
> ---
> base-commit: 076d56d74f17e625b3d63cf4743b3d7d02180379
> change-id: 20231219-iio-backend-a3dc1a6a7a58
> --
> 
> Thanks!
> - Nuno Sá
>
Jonathan Cameron Feb. 4, 2024, 4:01 p.m. UTC | #2
On Sun, 4 Feb 2024 14:59:33 +0000
Jonathan Cameron <jic23@kernel.org> wrote:

> On Fri, 02 Feb 2024 16:08:31 +0100
> Nuno Sa via B4 Relay <devnull+nuno.sa.analog.com@kernel.org> wrote:
> 
> > v1:
> >  https://lore.kernel.org/linux-iio/20231204144925.4fe9922f@jic23-huawei/T/#m222f5175273b81dbfe40b7f0daffcdc67d6cb8ff
> > 
> > v2:
> >  https://lore.kernel.org/r/20231208-dev-iio-backend-v2-0-5450951895e1@analog.com
> > 
> > v3:
> >  https://lore.kernel.org/linux-iio/20231213-dev-iio-backend-v3-0-bb9f12a5c6dc@analog.com/
> > 
> > v4:
> >  https://lore.kernel.org/r/20231220-iio-backend-v4-0-998e9148b692@analog.com
> > 
> > v5:
> >  https://lore.kernel.org/r/20240112-iio-backend-v5-0-bdecad041ab4@analog.com
> > 
> > v6:
> >  https://lore.kernel.org/r/20240119-iio-backend-v6-0-189536c35a05@analog.com
> > 
> > v7
> >  https://lore.kernel.org/r/20240123-iio-backend-v7-0-1bff236b8693@analog.com
> > 
> > Changes in v8:
> >  - Dropped commit ("of: property: fix typo in io-channels") - applied
> >    via DT tree. Also dropped commit
> >    ("driver: core: allow modifying device_links flags") - did not made
> >    sense.
> >  - Patch 7
> >   * Do not change the version string format during probe.
> > 
> > Jonathan, the series is based on next-20240202 since it already includes
> > the io-channels fix Rob applied in his tree. I guess it should land in rc3 so
> > after you rebase, all patches should apply cleanly (if applying them of course
> > :)). Let me know if anything fails...  
> 
> Given that merge (between my tree and Rob's) is about as trivial as they come
> I'll take the series now (rather than delaying) and rely on Stephen (for next)
> and Greg (once I send a pull request) to deal with it.
> 
> It's the sort of merge that makes Linus grumpy if people rebase to avoid it!
> 
> On that note, applied with those tweaks to the broken binding patch.
> 
> Initially pushed out as testing to see if 0-day moans at us a lot about anything.
Dropped again as comments in from Andy crossed with this.

Jonathan

> 
> Jonathan
> 
> > 
> > (also dropped the devlink Reviewers from the Cc list as that patch was
> > dropped).
> > 
> > Keeping the block diagram  so we don't have to follow links
> > to check one of the typical setups.
> > 
> >                                            -------------------------------------------------------
> >  ------------------                        | -----------         ------------      -------  FPGA |
> >  |     ADC        |------------------------| | AXI ADC |---------| DMA CORE |------| RAM |       |
> >  | (Frontend/IIO) | Serial Data (eg: LVDS) | |(backend)|---------|          |------|     |       |
> >  |                |------------------------| -----------         ------------      -------       |
> >  ------------------                        -------------------------------------------------------
> > 
> > ---
> > Nuno Sa (6):
> >       dt-bindings: adc: ad9467: add new io-backend property
> >       dt-bindings: adc: axi-adc: update bindings for backend framework
> >       iio: buffer-dmaengine: export buffer alloc and free functions
> >       iio: add the IIO backend framework
> >       iio: adc: ad9467: convert to backend framework
> >       iio: adc: adi-axi-adc: move to backend framework
> > 
> > Olivier Moysan (1):
> >       of: property: add device link support for io-backends
> > 
> >  .../devicetree/bindings/iio/adc/adi,ad9467.yaml    |   4 +
> >  .../devicetree/bindings/iio/adc/adi,axi-adc.yaml   |   8 +-
> >  MAINTAINERS                                        |   8 +
> >  drivers/iio/Kconfig                                |   9 +
> >  drivers/iio/Makefile                               |   1 +
> >  drivers/iio/adc/Kconfig                            |   4 +-
> >  drivers/iio/adc/ad9467.c                           | 268 +++++++++-----
> >  drivers/iio/adc/adi-axi-adc.c                      | 379 +++++--------------
> >  drivers/iio/buffer/industrialio-buffer-dmaengine.c |   8 +-
> >  drivers/iio/industrialio-backend.c                 | 412 +++++++++++++++++++++
> >  drivers/of/property.c                              |   2 +
> >  include/linux/iio/adc/adi-axi-adc.h                |  68 ----
> >  include/linux/iio/backend.h                        |  72 ++++
> >  include/linux/iio/buffer-dmaengine.h               |   3 +
> >  14 files changed, 793 insertions(+), 453 deletions(-)
> > ---
> > base-commit: 076d56d74f17e625b3d63cf4743b3d7d02180379
> > change-id: 20231219-iio-backend-a3dc1a6a7a58
> > --
> > 
> > Thanks!
> > - Nuno Sá
> >   
> 
>