1. 10 Aug, 2020 2 commits
  2. 07 Aug, 2020 1 commit
  3. 06 Aug, 2020 4 commits
  4. 05 Aug, 2020 4 commits
  5. 04 Aug, 2020 2 commits
  6. 03 Aug, 2020 7 commits
  7. 02 Aug, 2020 1 commit
  8. 01 Aug, 2020 2 commits
  9. 31 Jul, 2020 10 commits
  10. 30 Jul, 2020 7 commits
    • Mark Brown's avatar
      Merge series "drop unnecessary list_empty" from Julia Lawall <Julia.Lawall@inria.fr>: · 14e5ad7d
      Mark Brown authored
      The various list iterators are able to handle an empty list.
      The only effect of avoiding the loop is not initializing some
      index variables.
      Drop list_empty tests in cases where these variables are not
      used.
      
      The semantic patch that makes these changes is as follows:
      (http://coccinelle.lip6.fr/)
      
      <smpl>
      @@
      expression x,e;
      iterator name list_for_each_entry;
      statement S;
      identifier i;
      @@
      
      -if (!(list_empty(x))) {
         list_for_each_entry(i,x,...) S
      - }
       ... when != i
      ? i = e
      
      @@
      expression x,e;
      iterator name list_for_each_entry_safe;
      statement S;
      identifier i,j;
      @@
      
      -if (!(list_empty(x))) {
         list_for_each_entry_safe(i,j,x,...) S
      - }
       ... when != i
           when != j
      (
        i = e;
      |
      ? j = e;
      )
      
      @@
      expression x,e;
      iterator name list_for_each;
      statement S;
      identifier i;
      @@
      
      -if (!(list_empty(x))) {
         list_for_each(i,x) S
      - }
       ... when != i
      ? i = e
      
      @@
      expression x,e;
      iterator name list_for_each_safe;
      statement S;
      identifier i,j;
      @@
      
      -if (!(list_empty(x))) {
         list_for_each_safe(i,j,x) S
      - }
       ... when != i
           when != j
      (
        i = e;
      |
      ? j = e;
      )
      
      // -------------------
      
      @@
      expression x,e;
      statement S;
      identifier i;
      @@
      
      -if (!(list_empty(x)))
         list_for_each_entry(i,x,...) S
       ... when != i
      ? i = e
      
      @@
      expression x,e;
      statement S;
      identifier i,j;
      @@
      
      -if (!(list_empty(x)))
         list_for_each_entry_safe(i,j,x,...) S
       ... when != i
           when != j
      (
        i = e;
      |
      ? j = e;
      )
      
      @@
      expression x,e;
      statement S;
      identifier i;
      @@
      
      -if (!(list_empty(x)))
         list_for_each(i,x) S
       ... when != i
      ? i = e
      
      @@
      expression x,e;
      statement S;
      identifier i,j;
      @@
      
      -if (!(list_empty(x)))
         list_for_each_safe(i,j,x) S
       ... when != i
           when != j
      (
        i = e;
      |
      ? j = e;
      )
      </smpl>
      
      ---
      
       drivers/media/pci/saa7134/saa7134-core.c                      |   14 ++---
       drivers/media/usb/cx231xx/cx231xx-core.c                      |   16 ++----
       drivers/media/usb/tm6000/tm6000-core.c                        |   24 +++-------
       drivers/net/ethernet/mellanox/mlx5/core/steering/dr_matcher.c |   13 ++---
       drivers/net/ethernet/mellanox/mlx5/core/steering/dr_rule.c    |    5 --
       drivers/net/ethernet/sfc/ptp.c                                |   20 +++-----
       drivers/net/wireless/ath/dfs_pattern_detector.c               |   15 ++----
       sound/soc/intel/atom/sst/sst_loader.c                         |   10 +---
       sound/soc/intel/skylake/skl-pcm.c                             |    8 +--
       sound/soc/intel/skylake/skl-topology.c                        |    5 --
       10 files changed, 53 insertions(+), 77 deletions(-)
      14e5ad7d
    • Alper Nebi Yasak's avatar
      ASoC: rk3399_gru_sound: Add DAPM pins, kcontrols for jack detection · d0508b4f
      Alper Nebi Yasak authored
      PulseAudio (and perhaps other userspace utilities) can not detect any
      jack for rk3399_gru_sound as the driver doesn't expose related Jack
      kcontrols.
      
      This patch adds two DAPM pins to the headset jack, where the
      snd_soc_card_jack_new() call automatically creates "Headphones Jack" and
      "Headset Mic Jack" kcontrols from them.
      
      With an appropriate ALSA UCM config specifying JackControl fields for
      the "Headphones" and "Headset" (mic) devices, PulseAudio can detect
      plug/unplug events for both of them after this patch.
      Signed-off-by: default avatarAlper Nebi Yasak <alpernebiyasak@gmail.com>
      Link: https://lore.kernel.org/r/20200721182709.6895-1-alpernebiyasak@gmail.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      d0508b4f
    • Fabio Estevam's avatar
      ASoC: wm8962: Do not access WM8962_GPIO_BASE · 658bb297
      Fabio Estevam authored
      According to the WM8962 datasheet, there is no register at address 0x200.
      
      WM8962_GPIO_BASE is just a base address for the GPIO registers and not a
      real register, so remove it from wm8962_readable_register().
      
      Also, Register 515 (WM8962_GPIO_BASE + 3) does not exist, so skip
      its access.
      
      This fixes the following errors:
      
      wm8962 0-001a: ASoC: error at soc_component_read_no_lock on wm8962.0-001a: -16
      wm8962 0-001a: ASoC: error at soc_component_read_no_lock on wm8962.0-001a: -16
      Signed-off-by: default avatarFabio Estevam <festevam@gmail.com>
      Acked-by: default avatarCharles Keepax <ckeepax@opensource.cirrus.com>
      Link: https://lore.kernel.org/r/20200717135959.19212-1-festevam@gmail.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      658bb297
    • Julia Lawall's avatar
      ASoC: SOF: imx: use resource_size · afd842c0
      Julia Lawall authored
      Use resource_size rather than a verbose computation on
      the end and start fields.
      
      The semantic patch that makes this change is as follows:
      (http://coccinelle.lip6.fr/)
      
      <smpl>
      @@ struct resource ptr; @@
      - (ptr.end - ptr.start + 1)
      + resource_size(&ptr)
      </smpl>
      Signed-off-by: default avatarJulia Lawall <Julia.Lawall@inria.fr>
      Link: https://lore.kernel.org/r/1595751933-4952-1-git-send-email-Julia.Lawall@inria.frSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      afd842c0
    • Julia Lawall's avatar
      ASoC: Intel: drop unnecessary list_empty · a383308e
      Julia Lawall authored
      list_for_each_entry_safe is able to handle an empty list.
      The only effect of avoiding the loop is not initializing the
      index variable.
      Drop list_empty tests in cases where these variables are not
      used.
      
      Note that list_for_each_entry_safe is defined in terms of
      list_first_entry, which indicates that it should not be used on an
      empty list.  But in list_for_each_entry_safe, the element obtained by
      list_first_entry is not really accessed, only the address of its
      list_head field is compared to the address of the list head, so the
      list_first_entry is safe.
      
      The semantic patch that makes this change is as follows (with another
      variant for the no brace case): (http://coccinelle.lip6.fr/)
      
      <smpl>
      @@
      expression x,e;
      iterator name list_for_each_entry_safe;
      statement S;
      identifier i,j;
      @@
      -if (!(list_empty(x))) {
         list_for_each_entry_safe(i,j,x,...) S
      - }
       ... when != i
           when != j
      (
        i = e;
      |
      ? j = e;
      )
      </smpl>
      Signed-off-by: default avatarJulia Lawall <Julia.Lawall@inria.fr>
      Link: https://lore.kernel.org/r/1595761112-11003-2-git-send-email-Julia.Lawall@inria.frSigned-off-by: default avatarMark Brown <broonie@kernel.org>
      a383308e
    • Mark Brown's avatar
      Merge series "ASoC: Intel: KMB: TDM Enablement patches" from Michael Sit Wei... · 39473c2c
      Mark Brown authored
      Merge series "ASoC: Intel: KMB: TDM Enablement patches" from Michael Sit Wei Hong <michael.wei.hong.sit@intel.com>:
      
      This patch series is to enable multiple features on the Keembay Platform
      
      Michael Sit Wei Hong (4):
        ASoC: Intel: KMB: Add 8kHz audio support
        ASoC: Intel: KMB: Rework disable channel function
        ASoC: Intel: KMB: Enable TDM audio capture
        dt-bindings: sound: intel,keembay-i2s: Add channel-max property
      
       .../bindings/sound/intel,keembay-i2s.yaml     |   8 +
       sound/soc/intel/keembay/kmb_platform.c        | 137 +++++++++++++-----
       sound/soc/intel/keembay/kmb_platform.h        |   1 +
       3 files changed, 112 insertions(+), 34 deletions(-)
      
      --
      2.17.1
      39473c2c
    • Mark Brown's avatar
      Merge series "ASoC: meson: tdm fixes" from Jerome Brunet <jbrunet@baylibre.com>: · 3d026a8a
      Mark Brown authored
      This patcheset is collection of fixes for the TDM input and output the
      axg audio architecture. Its fixes:
       - slave mode format setting
       - g12 and sm1 skew offset
       - tdm clock inversion
       - standard daifmt props names which don't require a specific prefix
      
      Jerome Brunet (4):
        ASoC: meson: axg-tdm-interface: fix link fmt setup
        ASoC: meson: axg-tdmin: fix g12a skew
        ASoC: meson: axg-tdm-formatters: fix sclk inversion
        ASoC: meson: cards: remove DT_PREFIX for standard daifmt properties
      
       sound/soc/meson/axg-tdm-formatter.c | 11 ++++++-----
       sound/soc/meson/axg-tdm-formatter.h |  1 -
       sound/soc/meson/axg-tdm-interface.c | 26 +++++++++++++++++---------
       sound/soc/meson/axg-tdmin.c         | 16 +++++++++++++++-
       sound/soc/meson/axg-tdmout.c        |  3 ---
       sound/soc/meson/meson-card-utils.c  |  2 +-
       6 files changed, 39 insertions(+), 20 deletions(-)
      
      --
      2.25.4
      3d026a8a