[v6,00/13] Add support for IIO devices in ASoC

Message ID 20230623085830.749991-1-herve.codina@bootlin.com
Headers
Series Add support for IIO devices in ASoC |

Message

Herve Codina June 23, 2023, 8:58 a.m. UTC
  Several weeks ago, I sent a series [1] for adding a potentiometer as an
auxiliary device in ASoC. The feedback was that the potentiometer should
be directly handled in IIO (as other potentiometers) and something more
generic should be present in ASoC in order to have a binding to import
some IIO devices into sound cards.

The series related to the IIO potentiometer device is already applied.

This series introduces audio-iio-aux. Its goal is to offer the binding
between IIO and ASoC.
It exposes attached IIO devices as ASoC auxiliary devices and allows to
control them through mixer controls.

On my system, the IIO device is a potentiometer and it is present in an
amplifier design present in the audio path.

Compare to the previous iteration
  https://lore.kernel.org/linux-kernel/20230615152631.224529-1-herve.codina@bootlin.com/
This v6 series mainly:
 - Fixes comments
 - Does some minor code modification

Best regards,
Hervé

[1] https://lore.kernel.org/linux-kernel/20230203111422.142479-1-herve.codina@bootlin.com/
[2] https://lore.kernel.org/linux-kernel/20230421085245.302169-1-herve.codina@bootlin.com/

Changes v5 -> v6
  - Patches 3, 4, 11, 13
    No changes

  - Patch 1
    Add 'Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>'
    Add 'Reviewed-by: Rob Herring <robh@kernel.org>'

  - Patch 2
    Add 'Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>'

  - Patches 5, 6
    Add 'Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>'

  - Patch 7
    Fix comments:
      'buf' -> 'buff',
       '__unconst_typeof' -> '__unconst_integer_typeof'

  - Patch 8, 9, 10
    Add 'Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>'

  - Patch 12
    Set struct snd_kcontrol_new fields for the constrol variable at
    variable initalization.
    Add a missing blank line
    Add a comment related to min/max swapping
    Add 'Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>'

Changes v4 -> v5
  - Patches 1, 2, 3, 4, 5, 9, 10, 11, 12, 13
    No changes.

  - Patch 6
    Fix commit log.
    Add 'Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>'

  - Patch 7
    Fix the macros to be able to use them with:
     - an array defined as int *buff;
     - an array defined as int buff[N];
     - Rework the way to "unconstify" the temporary variable to avoid
       issues due to integer promotion.
    Add 'Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>'

  - Patch 8
    Add 'Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>'

Changes v3 -> v4
  - Patches 1, 2
    No changes.

  - Patches 3, 4, 5
    Add 'Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>'.

  - Patch 6 (new in v4)
    Fix headers inclusion order.

  - Patch 7 (patch 6 in v3)
    Add a comment related to __must_be_array()
    Use __array[0] of *__array

  - Patch 8 (patch 7 in v3)
    Fix minmax.h inclusion order.
    Add 'Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>'.

  - Patch 9 (patch 8 in v3)
    Add 'Suggested-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>'.
    Add 'Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>'.

  - Patch 10 (patch 9 in v3)
    Add 'Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>'.

  - Patch 11 (patch 10 in v3)
    Fix a typo.
    Add	'Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>'.
    Add	'Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>'.

  - Patch 12 (patch 11 in v3)
    Fix typos in the commit log.
    Fix headers inclusion order.
    Removed unneeded variable initialization.
    Replace {0} by {}.
    Use struct device *dev in probe().
    Check an error on the snd-control-invert-range property read.

  - Patch 13 (patch12 in v3)
    No changes.

Changes v2 -> v3
  - Patches 1, 2
    No changes.

  - Patch 3, 4
    Add 'Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>'.

  - Patch 5 (new in v3)
    Removed the 'unused' variable and check the null pointer when used.

  - Patch 6 (new in v3)
    Introduce {min,max}_array().

  - Patch 7 (new in v3)
    Use max_array() in iio_channel_read_max().

  - Patch 8 (new in v3)
    Replace a FIXME comment by a TODO one.

  - Patch 9 (patch 5 in v2)
    Removed the 'unused' variable and check the null pointer when used.
    Use min_array().
    Remplace a FIXME comment by a TODO one.

  - Patch 10 (patch 6 in v2)
    Convert existing macros to return a compound litteral instead of
    adding a new helper.

  - Patch 11 (patch 7 in v2)
    Remove the file name from the C file header.
    Use directly converted DAPM macros.
    Replace <linux/module.h> by <linux/mod_devicetable.h>.
    Add <linux/platform_device.h>.
    Be sure that min <= max. Swap values if it is not the case.
    Move the bool structure member after the int ones.
    Remove unneeded assignements.
    Use dev_err_probe() when relevant.
    Use str_on_off().
    Use static_assert() instead of BUILD_BUG_ON().
    Remove unneeded comma and blank line.
    Use device_property_*() instead of the OF API.

  - patch 8 available in v2 removed as already applied

  - Patch 12 (patch 9 in v2)
    Use devm_add_action_or_reset().
    Call simple_populate_aux() from simple_parse_of().

Changes v1 -> v2
  - Patch 1
    Rename simple-iio-aux to audio-iio-aux
    Rename invert to snd-control-invert-range
    Remove the /schemas/iio/iio-consumer.yaml reference
    Remove the unneeded '|' after description

  - Patch 2 (new in v2)
    Introduce the simple-audio-card additional-devs subnode

  - Patch 3 (new in v2)
    Check err before switch() in iio_channel_read_max()

  - Patch 4 (new in v2)
    Fix raw reads and raw writes documentation

  - Patch 5 (patch 2 in v1)
    Check err before switch() in iio_channel_read_min()
    Fix documentation

  - Patch 6 (path 3 in v1)
    No changes

  - Patch 7 (patch 4 in v1)
    Rename simple-iio-aux to audio-iio-aux
    Rename invert to snd-control-invert-range
    Remove the mask usage from audio_iio_aux_{get,put}_volsw helpers
    Use directly PTR_ERR() in dev_err_probe() parameter
    Remove the '!!' construction
    Remove of_match_ptr()

  - Patch 8 (new in v2)
    Add a missing of_node_put() in the simple-card driver

  - Patch 9 (new in v2)
    Handle additional-devs in the simple-card driver

Herve Codina (13):
  ASoC: dt-bindings: Add audio-iio-aux
  ASoC: dt-bindings: simple-card: Add additional-devs subnode
  iio: inkern: Check error explicitly in iio_channel_read_max()
  iio: consumer.h: Fix raw values documentation notes
  iio: inkern: Remove the 'unused' variable usage in
    iio_channel_read_max()
  iio: inkern: Fix headers inclusion order
  minmax: Introduce {min,max}_array()
  iio: inkern: Use max_array() to get the maximum value from an array
  iio: inkern: Replace a FIXME comment by a TODO one
  iio: inkern: Add a helper to query an available minimum raw value
  ASoC: soc-dapm.h: Convert macros to return a compound literal
  ASoC: codecs: Add support for the generic IIO auxiliary devices
  ASoC: simple-card: Handle additional devices

 .../bindings/sound/audio-iio-aux.yaml         |  64 ++++
 .../bindings/sound/simple-card.yaml           |  53 +++
 drivers/iio/inkern.c                          |  86 ++++-
 include/linux/iio/consumer.h                  |  37 +-
 include/linux/minmax.h                        |  64 ++++
 include/sound/soc-dapm.h                      | 138 ++++---
 sound/soc/codecs/Kconfig                      |  12 +
 sound/soc/codecs/Makefile                     |   2 +
 sound/soc/codecs/audio-iio-aux.c              | 344 ++++++++++++++++++
 sound/soc/generic/simple-card.c               |  46 ++-
 10 files changed, 775 insertions(+), 71 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/sound/audio-iio-aux.yaml
 create mode 100644 sound/soc/codecs/audio-iio-aux.c
  

Comments

Mark Brown July 12, 2023, 11:46 a.m. UTC | #1
On Fri, 23 Jun 2023 10:58:17 +0200, Herve Codina wrote:
> Several weeks ago, I sent a series [1] for adding a potentiometer as an
> auxiliary device in ASoC. The feedback was that the potentiometer should
> be directly handled in IIO (as other potentiometers) and something more
> generic should be present in ASoC in order to have a binding to import
> some IIO devices into sound cards.
> 
> The series related to the IIO potentiometer device is already applied.
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[01/13] ASoC: dt-bindings: Add audio-iio-aux
        commit: cacff5e8a305b98595829ac4f5fe669e015ce19c
[02/13] ASoC: dt-bindings: simple-card: Add additional-devs subnode
        commit: b5f3484117b86cb128f500ff2d730c3cfcb9ddfc
[03/13] iio: inkern: Check error explicitly in iio_channel_read_max()
        commit: f75c544d74133278b004195220f540d8ab953e14
[04/13] iio: consumer.h: Fix raw values documentation notes
        commit: 1e1b4fbd6d0f8c54af14dcf18bd3136816153b12
[05/13] iio: inkern: Remove the 'unused' variable usage in iio_channel_read_max()
        commit: ad4e8480a1db8713ee7dfcc2770ea9f577750111
[06/13] iio: inkern: Fix headers inclusion order
        commit: 524cfdf6b88e2536f90f726b8c01ffe218f37d68
[07/13] minmax: Introduce {min,max}_array()
        commit: c952c748c7a983a8bda9112984e6f2c1f6e441a5
[08/13] iio: inkern: Use max_array() to get the maximum value from an array
        commit: 97aee7157eeadaf628e7f76da5e49cee91f19901
[09/13] iio: inkern: Replace a FIXME comment by a TODO one
        commit: 4dc603735826ec3963e30d6f25260255ca96d103
[10/13] iio: inkern: Add a helper to query an available minimum raw value
        commit: 7560418078b939e1e83f7dce502ec3c1ca8c152f
[11/13] ASoC: soc-dapm.h: Convert macros to return a compound literal
        commit: 12e58fec5b2aff3ae6fef4e6c278f295a666b9b6
[12/13] ASoC: codecs: Add support for the generic IIO auxiliary devices
        commit: 1d298ad822178d365b53eac298c1752730505306
[13/13] ASoC: simple-card: Handle additional devices
        commit: 6d8ad35d119ca4c9c6fdf83faa733102c4a63f4b

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark