[v5,0/3] Add RZ/G2L Sound support

Message

Biju Das Aug. 13, 2021, 9:11 a.m. UTC

This patch series aims to add ASoC support on RZ/G2L SoC's.

It is based on the work done by Chris Brandt for RZ/A ASoC driver.

v4->v5
 * Moved validation of sample bits in hw_params
 * Removed validation of frame bits as it is redundant
 * split the rz_ssi_start_stop function into rz_ssi_start and rz_ssi_stop.
 * remove the spin_lock around rz_ssi_stream_init.
 * Updated dmas description and removed fixes as it is an enhancement
   now.
 * updated ssi_start functions with setting fifo thresholds
   and ssi_stop function with cancel all dma txn.
v3->v4:
 * Updated the subject line as per style for the subsystem.
 * Removed select SND_SIMPLE_CARD from Kconfig
 * Added C++ comments for copyright and driver description.
 * Moved validation of channels in hw_params
 * removed asm issue reported by bot as well as Mark
 * replaced master/slave macros with provider/consumer macros
 * Improved locking and added more null pointer checks.
v2->v3:
 * Fixed the dependency on KCONFIG
 * Merged the binding patch with dma feature added
 * Updated dt binding example with encoded #dma-cells value.
 * Improved Error handling in probe function
 * Removed the passing legacy channel configuration parameters from
   dmaengine_slave_config function
 * started using dma_request_chan instead of deprecated
   dma_request_slave_channel
 * Removed SoC dtsi and config patches from this series. Will send it later.
v1->v2:
 * Rebased to latest rc kernel

Biju Das (3):
  ASoC: sh: Add RZ/G2L SSIF-2 driver
  ASoC: dt-bindings: renesas,rz-ssi: Update slave dma channel
    configuration parameters
  ASoC: sh: rz-ssi: Add SSI DMAC support

 .../bindings/sound/renesas,rz-ssi.yaml        |   22 +-
 sound/soc/sh/Kconfig                          |    6 +
 sound/soc/sh/Makefile                         |    4 +
 sound/soc/sh/rz-ssi.c                         | 1063 +++++++++++++++++
 4 files changed, 1093 insertions(+), 2 deletions(-)
 create mode 100644 sound/soc/sh/rz-ssi.c

Comments

Mark Brown Aug. 13, 2021, 5:26 p.m. UTC | #1

On Fri, 13 Aug 2021 10:11:53 +0100, Biju Das wrote:
> This patch series aims to add ASoC support on RZ/G2L SoC's.
> 
> It is based on the work done by Chris Brandt for RZ/A ASoC driver.
> 
> v4->v5
>  * Moved validation of sample bits in hw_params
>  * Removed validation of frame bits as it is redundant
>  * split the rz_ssi_start_stop function into rz_ssi_start and rz_ssi_stop.
>  * remove the spin_lock around rz_ssi_stream_init.
>  * Updated dmas description and removed fixes as it is an enhancement
>    now.
>  * updated ssi_start functions with setting fifo thresholds
>    and ssi_stop function with cancel all dma txn.
> v3->v4:
>  * Updated the subject line as per style for the subsystem.
>  * Removed select SND_SIMPLE_CARD from Kconfig
>  * Added C++ comments for copyright and driver description.
>  * Moved validation of channels in hw_params
>  * removed asm issue reported by bot as well as Mark
>  * replaced master/slave macros with provider/consumer macros
>  * Improved locking and added more null pointer checks.
> v2->v3:
>  * Fixed the dependency on KCONFIG
>  * Merged the binding patch with dma feature added
>  * Updated dt binding example with encoded #dma-cells value.
>  * Improved Error handling in probe function
>  * Removed the passing legacy channel configuration parameters from
>    dmaengine_slave_config function
>  * started using dma_request_chan instead of deprecated
>    dma_request_slave_channel
>  * Removed SoC dtsi and config patches from this series. Will send it later.
> v1->v2:
>  * Rebased to latest rc kernel
> 
> [...]

Applied to

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

Thanks!

[1/3] ASoC: sh: Add RZ/G2L SSIF-2 driver
      commit: 03e786bd43410fa93e5d2459f7a43e90ff0ae801
[2/3] ASoC: dt-bindings: renesas,rz-ssi: Update slave dma channel configuration parameter
      commit: bed0b1c1e88a27b76c74584128cadebc6fa58622
[3/3] ASoC: sh: rz-ssi: Add SSI DMAC support
      commit: 26ac471c5354583cf4fe0e42537a2c6b84d6d74e

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