Thread (3 messages) 3 messages, 2 authors, 18d ago

Re: [PATCH v1 4/4] ASOC: Unify code style for platform_device_id arrays

From: Cezary Rojewski <cezary.rojewski@intel.com>
Date: 2026-05-28 09:28:12
Also in: imx, linux-arm-kernel, linux-arm-msm, linux-sound, lkml

On 5/28/2026 11:06 AM, Uwe Kleine-König (The Capable Hub) wrote:
  - Add a trailing comma for initializers unless the closing brace is on
    the same line and for the list terminator;
  - Use a single space in the list terminator;
  - Use compact one-line style for small entries;
  - s/\t=/ =/ were the tab is only one char wide anyhow;

Signed-off-by: Uwe Kleine-König (The Capable Hub) <redacted>
---
  sound/soc/amd/acp/acp-sof-mach.c       | 14 +++++++-------
  sound/soc/codecs/adau7118-hw.c         |  2 +-
  sound/soc/codecs/bt-sco.c              | 10 +++-------
  sound/soc/codecs/wcd934x.c             |  6 ++----
  sound/soc/fsl/imx-pcm-rpmsg.c          |  6 +++---
...
  sound/soc/intel/avs/boards/da7219.c    |  6 ++----
  sound/soc/intel/avs/boards/dmic.c      |  6 ++----
  sound/soc/intel/avs/boards/es8336.c    |  6 ++----
  sound/soc/intel/avs/boards/hdaudio.c   |  6 ++----
  sound/soc/intel/avs/boards/i2s_test.c  |  6 ++----
  sound/soc/intel/avs/boards/max98357a.c |  6 ++----
  sound/soc/intel/avs/boards/max98373.c  |  6 ++----
  sound/soc/intel/avs/boards/max98927.c  |  6 ++----
  sound/soc/intel/avs/boards/nau8825.c   |  6 ++----
  sound/soc/intel/avs/boards/pcm3168a.c  |  6 ++----
Please separate all the avs-driver changes into a separate patch with 
the following prefix for the title: "ASoC: Intel: avs:".

The code itself looks good. Provided the separation is done, feel free 
to add my tag:

Acked-by: Cezary Rojewski <cezary.rojewski@intel.com>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help