mbox series

[v6,0/4] Add bridged amplifiers to cs42l43

Message ID 20240415140925.3518990-1-ckeepax@opensource.cirrus.com
Headers show
Series Add bridged amplifiers to cs42l43 | expand

Message

Charles Keepax April 15, 2024, 2:09 p.m. UTC 
In some cs42l43 systems a couple of cs35l56 amplifiers are attached
to the cs42l43's SPI and I2S. On Windows the cs42l43 is controlled
by a SDCA class driver and these two amplifiers are controlled by
firmware running on the cs42l43. However, under Linux the decision
was made to interact with the cs42l43 directly, affording the user
greater control over the audio system. However, this has resulted
in an issue where these two bridged cs35l56 amplifiers are not
populated in ACPI and must be added manually. There is at least an
SDCA extension unit DT entry we can key off.

The process of adding this is handled using a software node, firstly the
ability to add native chip selects to software nodes must be added.
Secondly, an additional flag for naming the SPI devices is added this
allows the machine driver to key to the correct amplifier. Then finally,
the cs42l43 SPI driver adds the two amplifiers directly onto its SPI
bus.

An additional series will follow soon to add the audio machine driver
parts (in the sof-sdw driver), however that is fairly orthogonal to
this part of the process, getting the actual amplifiers registered.

Thanks,
Charles

Series changes since v5:
 - Add back help for GPIO_SWNODE_UNDEFINED
 - Correct some typos
 - Only check for undefined swnode if the Kconfig is set
 - Add a pr_fmt
 - Move swnode_gpio_undefined to include/linux/gpio/property.h
 - Remove includes of fwnode.h
 - Use %pwfP to get firmware node name
 - Remove NULL check on acpi_handle

Series changes since v4:
 - Remove extraneous fwnode_handle_puts in driver/spi/spi-cs42l43.c
 - Make Kconfig for swnode undef gpios not user visible
 - Add some missing headers
 - Add patch to update handling in spi_dev_set_name
 - Remove stray blank line
 - Use ACPI_HANDLE_FWNODE

Series changes since v3:
 - Add Kconfig to make swnode conditionally built
 - Add define for swnode name
 - Add custom init and exit calls to register swnode
 - Use export namespaces
 - Always name swnode SPI devices after the node name
 - Correct some header includes
 - Use HZ_PER_MHZ
 - Use some swnode helper macros
 - Use acpi_get_local_address
 - Correct some handle puts
 - Add some dev_err_probes

Series changes since v2:
 - Add missing fwnode_handle_puts in driver/spi/spi-cs423l43.c

Series changes since v1:
 - Add missing statics in driver/spi/spi-cs42l43.c


Charles Keepax (3):
  gpio: swnode: Add ability to specify native chip selects for SPI
  spi: Switch to using is_acpi_device_node() in spi_dev_set_name()
  spi: Update swnode based SPI devices to use the fwnode name

Maciej Strozek (1):
  spi: cs42l43: Add bridged cs35l56 amplifiers

 drivers/gpio/Kconfig          |   9 +++
 drivers/gpio/gpiolib-swnode.c |  44 +++++++++++
 drivers/spi/Kconfig           |   1 +
 drivers/spi/spi-cs42l43.c     | 135 +++++++++++++++++++++++++++++++++-
 drivers/spi/spi.c             |  12 ++-
 include/linux/gpio/property.h |   4 +
 6 files changed, 200 insertions(+), 5 deletions(-)

Comments

Andy Shevchenko April 15, 2024, 4:06 p.m. UTC | #1 
On Mon, Apr 15, 2024 at 5:09 PM Charles Keepax
<ckeepax@opensource.cirrus.com> wrote:
>
> Use is_acpi_device_node() rather than checking ACPI_COMPANION(), such
> that when checking for other types of firmware node the code can

a firmware node

> consistently do checks against the fwnode.

Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Charles Keepax April 16, 2024, 9:53 a.m. UTC | #2 
On Mon, Apr 15, 2024 at 07:06:56PM +0300, Andy Shevchenko wrote:
> On Mon, Apr 15, 2024 at 5:09 PM Charles Keepax
> <ckeepax@opensource.cirrus.com> wrote:
> >
> > Use is_acpi_device_node() rather than checking ACPI_COMPANION(), such
> > that when checking for other types of firmware node the code can
> 
> a firmware node
> 

Pretty sure this parses better without the 'a'.

Thanks,
Charles