December 2017 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] [PATCH] ALSA: usb-audio: Proper fallback at get_term_name()
by Takashi Iwai 20 Dec '17

20 Dec '17

get_term_name() calls snd_usb_copy_string_desc() for retrieving the name when a specific ID (name field) is given. When this returns an error (zero), however, it simply returns as is. This will end up in a fixed name string in the caller side, which often is meaningless. For giving a bit more useful name string depending on the terminal type, change the get_term_name() function to go through the fallback mode. Signed-off-by: Takashi Iwai <tiwai(a)suse.de> --- sound/usb/mixer.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c index 2b4ceda36291..9afb8ab524c7 100644 --- a/sound/usb/mixer.c +++ b/sound/usb/mixer.c @@ -656,10 +656,14 @@ static int get_term_name(struct mixer_build *state, struct usb_audio_term *iterm unsigned char *name, int maxlen, int term_only) { struct iterm_name_combo *names; + int len; - if (iterm->name) - return snd_usb_copy_string_desc(state, iterm->name, + if (iterm->name) { + len = snd_usb_copy_string_desc(state, iterm->name, name, maxlen); + if (len) + return len; + } /* virtual type - not a real terminal */ if (iterm->type >> 16) { -- 2.15.1

1 0

[alsa-devel] [PATCH 0/3] ASoC: remove dai_drv from snd_soc_component
by Kuninori Morimoto 20 Dec '17

20 Dec '17

Hi Mark These are not important, but cleanup. snd_soc_component is keeping DAI driver pointer, but each DAI has its driver, and component has dai_list. This means, we can reach to all DAI and its driver by using dai_list. We can remove DAI driver pointer from component. Kuninori Morimoto (3): ASoC: hdac_hdmi: keep DAI driver pointer in private data ASoC: soc-core: remove dai_drv from snd_soc_component ASoC: soc.h: Arrange DAI related parameter include/sound/soc.h | 4 +--- sound/soc/codecs/hdac_hdmi.c | 4 +++- sound/soc/soc-core.c | 14 ++++++++++---- 3 files changed, 14 insertions(+), 8 deletions(-) -- 1.9.1

2 6

[alsa-devel] [PATCH 1/2] ASoC: Intel: Skylake: Request IRQ late only after all context are initialized
by Guneshwor Singh 20 Dec '17

20 Dec '17

From: "Subhransu S. Prusty" <subhransu.s.prusty(a)intel.com> Sometimes during boot, panic is observed at sst_dsp_shim_read_unlocked(). This happens when interrupt occurs before the context is initialized. So move the irq initialization only after the context is initialized completely. Signed-off-by: Subhransu S. Prusty <subhransu.s.prusty(a)intel.com> Signed-off-by: Pawse, GuruprasadX <guruprasadx.pawse(a)intel.com> Signed-off-by: Guneshwor Singh <guneshwor.o.singh(a)intel.com> --- sound/soc/intel/skylake/bxt-sst.c | 2 +- sound/soc/intel/skylake/cnl-sst.c | 2 +- sound/soc/intel/skylake/skl-sst-dsp.c | 14 ++++++++++---- sound/soc/intel/skylake/skl-sst-dsp.h | 1 + sound/soc/intel/skylake/skl-sst.c | 2 +- 5 files changed, 14 insertions(+), 7 deletions(-) diff --git a/sound/soc/intel/skylake/bxt-sst.c b/sound/soc/intel/skylake/bxt-sst.c index 1c78b853a7cc..13b1749fbcd5 100644 --- a/sound/soc/intel/skylake/bxt-sst.c +++ b/sound/soc/intel/skylake/bxt-sst.c @@ -582,7 +582,7 @@ int bxt_sst_dsp_init(struct device *dev, void __iomem *mmio_base, int irq, INIT_DELAYED_WORK(&skl->d0i3.work, bxt_set_dsp_D0i3); skl->d0i3.state = SKL_DSP_D0I3_NONE; - return 0; + return skl_dsp_acquire_irq(sst); } EXPORT_SYMBOL_GPL(bxt_sst_dsp_init); diff --git a/sound/soc/intel/skylake/cnl-sst.c b/sound/soc/intel/skylake/cnl-sst.c index 387de388ce29..245df1067ba8 100644 --- a/sound/soc/intel/skylake/cnl-sst.c +++ b/sound/soc/intel/skylake/cnl-sst.c @@ -458,7 +458,7 @@ int cnl_sst_dsp_init(struct device *dev, void __iomem *mmio_base, int irq, cnl->boot_complete = false; init_waitqueue_head(&cnl->boot_wait); - return 0; + return skl_dsp_acquire_irq(sst); } EXPORT_SYMBOL_GPL(cnl_sst_dsp_init); diff --git a/sound/soc/intel/skylake/skl-sst-dsp.c b/sound/soc/intel/skylake/skl-sst-dsp.c index 19ee1d4f3bdf..71e31ad0bb3f 100644 --- a/sound/soc/intel/skylake/skl-sst-dsp.c +++ b/sound/soc/intel/skylake/skl-sst-dsp.c @@ -435,16 +435,22 @@ struct sst_dsp *skl_dsp_ctx_init(struct device *dev, return NULL; } + return sst; +} + +int skl_dsp_acquire_irq(struct sst_dsp *sst) +{ + struct sst_dsp_device *sst_dev = sst->sst_dev; + int ret; + /* Register the ISR */ ret = request_threaded_irq(sst->irq, sst->ops->irq_handler, sst_dev->thread, IRQF_SHARED, "AudioDSP", sst); - if (ret) { + if (ret) dev_err(sst->dev, "unable to grab threaded IRQ %d, disabling device\n", sst->irq); - return NULL; - } - return sst; + return ret; } void skl_dsp_free(struct sst_dsp *dsp) diff --git a/sound/soc/intel/skylake/skl-sst-dsp.h b/sound/soc/intel/skylake/skl-sst-dsp.h index b8e799ed65ef..12fc9a73dc8a 100644 --- a/sound/soc/intel/skylake/skl-sst-dsp.h +++ b/sound/soc/intel/skylake/skl-sst-dsp.h @@ -206,6 +206,7 @@ int skl_cldma_wait_interruptible(struct sst_dsp *ctx); void skl_dsp_set_state_locked(struct sst_dsp *ctx, int state); struct sst_dsp *skl_dsp_ctx_init(struct device *dev, struct sst_dsp_device *sst_dev, int irq); +int skl_dsp_acquire_irq(struct sst_dsp *sst); bool is_skl_dsp_running(struct sst_dsp *ctx); unsigned int skl_dsp_get_enabled_cores(struct sst_dsp *ctx); diff --git a/sound/soc/intel/skylake/skl-sst.c b/sound/soc/intel/skylake/skl-sst.c index a436abf2fe3f..5a7e41b65ef3 100644 --- a/sound/soc/intel/skylake/skl-sst.c +++ b/sound/soc/intel/skylake/skl-sst.c @@ -569,7 +569,7 @@ int skl_sst_dsp_init(struct device *dev, void __iomem *mmio_base, int irq, sst->fw_ops = skl_fw_ops; - return 0; + return skl_dsp_acquire_irq(sst); } EXPORT_SYMBOL_GPL(skl_sst_dsp_init); -- 2.15.1

3 4

[alsa-devel] [PATCH] ASoC: rsnd: fixup ADG register mask
by Kuninori Morimoto 20 Dec '17

20 Dec '17

From: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> BRGCKR should use 0x80770000, instead of 0x80FF0000. R-Car Gen2 xxx_TIMSEL should use 0x0F1F, R-Car Gen3 xxx_TIMSEL should use 0x1F1F. Here, Gen3 doesn't support AVD, thus, both case can use 0x0F1F. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx(a)renesas.com> Reviewed-by: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx(a)renesas.com> --- sound/soc/sh/rcar/adg.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/soc/sh/rcar/adg.c b/sound/soc/sh/rcar/adg.c index 8ddb087..4672688 100644 --- a/sound/soc/sh/rcar/adg.c +++ b/sound/soc/sh/rcar/adg.c @@ -222,7 +222,7 @@ int rsnd_adg_set_cmd_timsel_gen2(struct rsnd_mod *cmd_mod, NULL, &val, NULL); val = val << shift; - mask = 0xffff << shift; + mask = 0x0f1f << shift; rsnd_mod_bset(adg_mod, CMDOUT_TIMSEL, mask, val); @@ -250,7 +250,7 @@ int rsnd_adg_set_src_timesel_gen2(struct rsnd_mod *src_mod, in = in << shift; out = out << shift; - mask = 0xffff << shift; + mask = 0x0f1f << shift; switch (id / 2) { case 0: @@ -380,7 +380,7 @@ int rsnd_adg_ssi_clk_try_start(struct rsnd_mod *ssi_mod, unsigned int rate) ckr = 0x80000000; } - rsnd_mod_bset(adg_mod, BRGCKR, 0x80FF0000, adg->ckr | ckr); + rsnd_mod_bset(adg_mod, BRGCKR, 0x80770000, adg->ckr | ckr); rsnd_mod_write(adg_mod, BRRA, adg->rbga); rsnd_mod_write(adg_mod, BRRB, adg->rbgb); -- 1.9.1

2 1

[alsa-devel] Raspberry Pi - asound/proc/status - state parameter always set to RUNNING?
by Michael Pinnell 20 Dec '17

20 Dec '17

Hi all, I was passed thi smailing list info in this github issue.. https://github.com/opensrc/alsa/issues/26#issuecomment-352616858 Basically, I'm working on a raspberry pi and need to know when sound is playing or not, and I'm trying to use the status file's state parameter to determine when this is. It seems however that my state parameter is always set to RUNNING and has an owner_pid even though that pid is not showing up in ps. Any ideas what may be causing this and how to rectify? Thanks, any help would be much appreciated. Regards, Mike. <http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_cam…> Virus-free. www.avg.com <http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_cam…> <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

1 0

[alsa-devel] [asoc:topic/fsl-ssi 11/18] sound/soc/fsl/fsl_ssi.h:87:0: warning: "SSI_SCR_TCH_EN" redefined
by kbuild test robot 19 Dec '17

19 Dec '17

tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git topic/fsl-ssi head: 52eee84e815e0fbaf9ada848ab5646314a529b61 commit: a818aa5f967ba60522ee0ad181a0c5a96b65d999 [11/18] ASoC: fsl_ssi: Rename registers and fields macros config: arm-allmodconfig (attached as .config) compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross git checkout a818aa5f967ba60522ee0ad181a0c5a96b65d999 # save the attached .config to linux build tree make.cross ARCH=arm All warnings (new ones prefixed by >>): In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:87:0: warning: "SSI_SCR_TCH_EN" redefined #define SSI_SCR_TCH_EN 0x00000100 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:18:0: note: this is the location of the previous definition #define SSI_SCR_TCH_EN (1 << 8) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:88:0: warning: "SSI_SCR_SYS_CLK_EN" redefined #define SSI_SCR_SYS_CLK_EN 0x00000080 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:19:0: note: this is the location of the previous definition #define SSI_SCR_SYS_CLK_EN (1 << 7) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:92:0: warning: "SSI_SCR_I2S_MODE_SLAVE" redefined #define SSI_SCR_I2S_MODE_SLAVE 0x00000040 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:22:0: note: this is the location of the previous definition #define SSI_SCR_I2S_MODE_SLAVE (2 << 5) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:93:0: warning: "SSI_SCR_SYN" redefined #define SSI_SCR_SYN 0x00000010 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:24:0: note: this is the location of the previous definition #define SSI_SCR_SYN (1 << 4) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:94:0: warning: "SSI_SCR_NET" redefined #define SSI_SCR_NET 0x00000008 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:25:0: note: this is the location of the previous definition #define SSI_SCR_NET (1 << 3) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:95:0: warning: "SSI_SCR_RE" redefined #define SSI_SCR_RE 0x00000004 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:26:0: note: this is the location of the previous definition #define SSI_SCR_RE (1 << 2) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:96:0: warning: "SSI_SCR_TE" redefined #define SSI_SCR_TE 0x00000002 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:27:0: note: this is the location of the previous definition #define SSI_SCR_TE (1 << 1) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:97:0: warning: "SSI_SCR_SSIEN" redefined #define SSI_SCR_SSIEN 0x00000001 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:28:0: note: this is the location of the previous definition #define SSI_SCR_SSIEN (1 << 0) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:102:0: warning: "SSI_SISR_CMDAU" redefined #define SSI_SISR_CMDAU 0x00040000 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:32:0: note: this is the location of the previous definition #define SSI_SISR_CMDAU (1 << 18) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:103:0: warning: "SSI_SISR_CMDDU" redefined #define SSI_SISR_CMDDU 0x00020000 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:33:0: note: this is the location of the previous definition #define SSI_SISR_CMDDU (1 << 17) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:104:0: warning: "SSI_SISR_RXT" redefined #define SSI_SISR_RXT 0x00010000 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:34:0: note: this is the location of the previous definition #define SSI_SISR_RXT (1 << 16) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:105:0: warning: "SSI_SISR_RDR1" redefined #define SSI_SISR_RDR1 0x00008000 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:35:0: note: this is the location of the previous definition #define SSI_SISR_RDR1 (1 << 15) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:106:0: warning: "SSI_SISR_RDR0" redefined #define SSI_SISR_RDR0 0x00004000 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:36:0: note: this is the location of the previous definition #define SSI_SISR_RDR0 (1 << 14) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:107:0: warning: "SSI_SISR_TDE1" redefined #define SSI_SISR_TDE1 0x00002000 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:37:0: note: this is the location of the previous definition #define SSI_SISR_TDE1 (1 << 13) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:108:0: warning: "SSI_SISR_TDE0" redefined #define SSI_SISR_TDE0 0x00001000 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:38:0: note: this is the location of the previous definition #define SSI_SISR_TDE0 (1 << 12) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:109:0: warning: "SSI_SISR_ROE1" redefined #define SSI_SISR_ROE1 0x00000800 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:39:0: note: this is the location of the previous definition #define SSI_SISR_ROE1 (1 << 11) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:110:0: warning: "SSI_SISR_ROE0" redefined #define SSI_SISR_ROE0 0x00000400 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:40:0: note: this is the location of the previous definition #define SSI_SISR_ROE0 (1 << 10) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:111:0: warning: "SSI_SISR_TUE1" redefined #define SSI_SISR_TUE1 0x00000200 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:41:0: note: this is the location of the previous definition #define SSI_SISR_TUE1 (1 << 9) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:112:0: warning: "SSI_SISR_TUE0" redefined #define SSI_SISR_TUE0 0x00000100 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:42:0: note: this is the location of the previous definition #define SSI_SISR_TUE0 (1 << 8) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: >> sound/soc/fsl/fsl_ssi.h:113:0: warning: "SSI_SISR_TFS" redefined #define SSI_SISR_TFS 0x00000080 In file included from sound/soc/fsl/eukrea-tlv320.c:31:0: sound/soc/fsl/imx-ssi.h:43:0: note: this is the location of the previous definition #define SSI_SISR_TFS (1 << 7) In file included from sound/soc/fsl/eukrea-tlv320.c:32:0: vim +/SSI_SCR_TCH_EN +87 sound/soc/fsl/fsl_ssi.h 82 83 /* SSI Control Register -- REG_SSI_SCR 0x10 */ 84 #define SSI_SCR_SYNC_TX_FS 0x00001000 85 #define SSI_SCR_RFR_CLK_DIS 0x00000800 86 #define SSI_SCR_TFR_CLK_DIS 0x00000400 > 87 #define SSI_SCR_TCH_EN 0x00000100 > 88 #define SSI_SCR_SYS_CLK_EN 0x00000080 89 #define SSI_SCR_I2S_MODE_MASK 0x00000060 90 #define SSI_SCR_I2S_MODE_NORMAL 0x00000000 91 #define SSI_SCR_I2S_MODE_MASTER 0x00000020 > 92 #define SSI_SCR_I2S_MODE_SLAVE 0x00000040 > 93 #define SSI_SCR_SYN 0x00000010 > 94 #define SSI_SCR_NET 0x00000008 > 95 #define SSI_SCR_RE 0x00000004 > 96 #define SSI_SCR_TE 0x00000002 > 97 #define SSI_SCR_SSIEN 0x00000001 98 99 /* SSI Interrupt Status Register -- REG_SSI_SISR 0x14 */ 100 #define SSI_SISR_RFRC 0x01000000 101 #define SSI_SISR_TFRC 0x00800000 > 102 #define SSI_SISR_CMDAU 0x00040000 > 103 #define SSI_SISR_CMDDU 0x00020000 > 104 #define SSI_SISR_RXT 0x00010000 > 105 #define SSI_SISR_RDR1 0x00008000 > 106 #define SSI_SISR_RDR0 0x00004000 > 107 #define SSI_SISR_TDE1 0x00002000 > 108 #define SSI_SISR_TDE0 0x00001000 > 109 #define SSI_SISR_ROE1 0x00000800 > 110 #define SSI_SISR_ROE0 0x00000400 > 111 #define SSI_SISR_TUE1 0x00000200 > 112 #define SSI_SISR_TUE0 0x00000100 > 113 #define SSI_SISR_TFS 0x00000080 > 114 #define SSI_SISR_RFS 0x00000040 > 115 #define SSI_SISR_TLS 0x00000020 > 116 #define SSI_SISR_RLS 0x00000010 > 117 #define SSI_SISR_RFF1 0x00000008 > 118 #define SSI_SISR_RFF0 0x00000004 > 119 #define SSI_SISR_TFE1 0x00000002 > 120 #define SSI_SISR_TFE0 0x00000001 121 122 /* SSI Interrupt Enable Register -- REG_SSI_SIER 0x18 */ 123 #define SSI_SIER_RFRC_EN 0x01000000 124 #define SSI_SIER_TFRC_EN 0x00800000 > 125 #define SSI_SIER_RDMAE 0x00400000 > 126 #define SSI_SIER_RIE 0x00200000 > 127 #define SSI_SIER_TDMAE 0x00100000 > 128 #define SSI_SIER_TIE 0x00080000 > 129 #define SSI_SIER_CMDAU_EN 0x00040000 > 130 #define SSI_SIER_CMDDU_EN 0x00020000 > 131 #define SSI_SIER_RXT_EN 0x00010000 > 132 #define SSI_SIER_RDR1_EN 0x00008000 > 133 #define SSI_SIER_RDR0_EN 0x00004000 > 134 #define SSI_SIER_TDE1_EN 0x00002000 > 135 #define SSI_SIER_TDE0_EN 0x00001000 > 136 #define SSI_SIER_ROE1_EN 0x00000800 > 137 #define SSI_SIER_ROE0_EN 0x00000400 > 138 #define SSI_SIER_TUE1_EN 0x00000200 > 139 #define SSI_SIER_TUE0_EN 0x00000100 > 140 #define SSI_SIER_TFS_EN 0x00000080 > 141 #define SSI_SIER_RFS_EN 0x00000040 > 142 #define SSI_SIER_TLS_EN 0x00000020 > 143 #define SSI_SIER_RLS_EN 0x00000010 > 144 #define SSI_SIER_RFF1_EN 0x00000008 > 145 #define SSI_SIER_RFF0_EN 0x00000004 > 146 #define SSI_SIER_TFE1_EN 0x00000002 > 147 #define SSI_SIER_TFE0_EN 0x00000001 148 149 /* SSI Transmit Configuration Register -- REG_SSI_STCR 0x1C */ > 150 #define SSI_STCR_TXBIT0 0x00000200 > 151 #define SSI_STCR_TFEN1 0x00000100 > 152 #define SSI_STCR_TFEN0 0x00000080 > 153 #define SSI_STCR_TFDIR 0x00000040 > 154 #define SSI_STCR_TXDIR 0x00000020 > 155 #define SSI_STCR_TSHFD 0x00000010 > 156 #define SSI_STCR_TSCKP 0x00000008 > 157 #define SSI_STCR_TFSI 0x00000004 > 158 #define SSI_STCR_TFSL 0x00000002 > 159 #define SSI_STCR_TEFS 0x00000001 160 161 /* SSI Receive Configuration Register -- REG_SSI_SRCR 0x20 */ 162 #define SSI_SRCR_RXEXT 0x00000400 > 163 #define SSI_SRCR_RXBIT0 0x00000200 > 164 #define SSI_SRCR_RFEN1 0x00000100 > 165 #define SSI_SRCR_RFEN0 0x00000080 > 166 #define SSI_SRCR_RFDIR 0x00000040 > 167 #define SSI_SRCR_RXDIR 0x00000020 > 168 #define SSI_SRCR_RSHFD 0x00000010 > 169 #define SSI_SRCR_RSCKP 0x00000008 > 170 #define SSI_SRCR_RFSI 0x00000004 > 171 #define SSI_SRCR_RFSL 0x00000002 > 172 #define SSI_SRCR_REFS 0x00000001 173 174 /* 175 * SSI Transmit Clock Control Register -- REG_SSI_STCCR 0x24 176 * SSI Receive Clock Control Register -- REG_SSI_SRCCR 0x28 177 */ 178 #define SSI_SxCCR_DIV2_SHIFT 18 179 #define SSI_SxCCR_DIV2 0x00040000 180 #define SSI_SxCCR_PSR_SHIFT 17 181 #define SSI_SxCCR_PSR 0x00020000 182 #define SSI_SxCCR_WL_SHIFT 13 183 #define SSI_SxCCR_WL_MASK 0x0001E000 184 #define SSI_SxCCR_WL(x) \ 185 (((((x) / 2) - 1) << SSI_SxCCR_WL_SHIFT) & SSI_SxCCR_WL_MASK) 186 #define SSI_SxCCR_DC_SHIFT 8 187 #define SSI_SxCCR_DC_MASK 0x00001F00 188 #define SSI_SxCCR_DC(x) \ 189 ((((x) - 1) << SSI_SxCCR_DC_SHIFT) & SSI_SxCCR_DC_MASK) 190 #define SSI_SxCCR_PM_SHIFT 0 191 #define SSI_SxCCR_PM_MASK 0x000000FF 192 #define SSI_SxCCR_PM(x) \ 193 ((((x) - 1) << SSI_SxCCR_PM_SHIFT) & SSI_SxCCR_PM_MASK) 194 195 /* 196 * SSI FIFO Control/Status Register -- REG_SSI_SFCSR 0x2c 197 * 198 * Tx or Rx FIFO Counter -- SSI_SFCSR_xFCNTy Read-Only 199 * Tx or Rx FIFO Watermarks -- SSI_SFCSR_xFWMy Read/Write 200 */ 201 #define SSI_SFCSR_RFCNT1_SHIFT 28 202 #define SSI_SFCSR_RFCNT1_MASK 0xF0000000 > 203 #define SSI_SFCSR_RFCNT1(x) \ 204 (((x) & SSI_SFCSR_RFCNT1_MASK) >> SSI_SFCSR_RFCNT1_SHIFT) 205 #define SSI_SFCSR_TFCNT1_SHIFT 24 206 #define SSI_SFCSR_TFCNT1_MASK 0x0F000000 > 207 #define SSI_SFCSR_TFCNT1(x) \ 208 (((x) & SSI_SFCSR_TFCNT1_MASK) >> SSI_SFCSR_TFCNT1_SHIFT) 209 #define SSI_SFCSR_RFWM1_SHIFT 20 210 #define SSI_SFCSR_RFWM1_MASK 0x00F00000 > 211 #define SSI_SFCSR_RFWM1(x) \ 212 (((x) << SSI_SFCSR_RFWM1_SHIFT) & SSI_SFCSR_RFWM1_MASK) 213 #define SSI_SFCSR_TFWM1_SHIFT 16 214 #define SSI_SFCSR_TFWM1_MASK 0x000F0000 > 215 #define SSI_SFCSR_TFWM1(x) \ 216 (((x) << SSI_SFCSR_TFWM1_SHIFT) & SSI_SFCSR_TFWM1_MASK) 217 #define SSI_SFCSR_RFCNT0_SHIFT 12 218 #define SSI_SFCSR_RFCNT0_MASK 0x0000F000 > 219 #define SSI_SFCSR_RFCNT0(x) \ 220 (((x) & SSI_SFCSR_RFCNT0_MASK) >> SSI_SFCSR_RFCNT0_SHIFT) 221 #define SSI_SFCSR_TFCNT0_SHIFT 8 222 #define SSI_SFCSR_TFCNT0_MASK 0x00000F00 > 223 #define SSI_SFCSR_TFCNT0(x) \ 224 (((x) & SSI_SFCSR_TFCNT0_MASK) >> SSI_SFCSR_TFCNT0_SHIFT) 225 #define SSI_SFCSR_RFWM0_SHIFT 4 > 226 #define SSI_SFCSR_RFWM0_MASK 0x000000F0 > 227 #define SSI_SFCSR_RFWM0(x) \ 228 (((x) << SSI_SFCSR_RFWM0_SHIFT) & SSI_SFCSR_RFWM0_MASK) 229 #define SSI_SFCSR_TFWM0_SHIFT 0 > 230 #define SSI_SFCSR_TFWM0_MASK 0x0000000F > 231 #define SSI_SFCSR_TFWM0(x) \ 232 (((x) << SSI_SFCSR_TFWM0_SHIFT) & SSI_SFCSR_TFWM0_MASK) 233 234 /* SSI Test Register -- REG_SSI_STR 0x30 */ > 235 #define SSI_STR_TEST 0x00008000 > 236 #define SSI_STR_RCK2TCK 0x00004000 > 237 #define SSI_STR_RFS2TFS 0x00002000 > 238 #define SSI_STR_RXSTATE(x) (((x) >> 8) & 0x1F) > 239 #define SSI_STR_TXD2RXD 0x00000080 > 240 #define SSI_STR_TCK2RCK 0x00000040 > 241 #define SSI_STR_TFS2RFS 0x00000020 > 242 #define SSI_STR_TXSTATE(x) ((x) & 0x1F) 243 244 /* SSI Option Register -- REG_SSI_SOR 0x34 */ > 245 #define SSI_SOR_CLKOFF 0x00000040 > 246 #define SSI_SOR_RX_CLR 0x00000020 > 247 #define SSI_SOR_TX_CLR 0x00000010 > 248 #define SSI_SOR_INIT 0x00000008 249 #define SSI_SOR_WAIT_SHIFT 1 > 250 #define SSI_SOR_WAIT_MASK 0x00000006 > 251 #define SSI_SOR_WAIT(x) (((x) & 3) << SSI_SOR_WAIT_SHIFT) > 252 #define SSI_SOR_SYNRST 0x00000001 253 254 /* SSI AC97 Control Register -- REG_SSI_SACNT 0x38 */ 255 #define SSI_SACNT_FRDIV(x) (((x) & 0x3f) << 5) > 256 #define SSI_SACNT_WR 0x00000010 > 257 #define SSI_SACNT_RD 0x00000008 258 #define SSI_SACNT_RDWR_MASK 0x00000018 > 259 #define SSI_SACNT_TIF 0x00000004 > 260 #define SSI_SACNT_FV 0x00000002 > 261 #define SSI_SACNT_AC97EN 0x00000001 262 263 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation

2 1

[alsa-devel] [trivial PATCH] treewide: Align function definition open/close braces
by Joe Perches 19 Dec '17

19 Dec '17

Some functions definitions have either the initial open brace and/or the closing brace outside of column 1. Move those braces to column 1. This allows various function analyzers like gnu complexity to work properly for these modified functions. Miscellanea: o Remove extra trailing ; and blank line from xfs_agf_verify Signed-off-by: Joe Perches <joe(a)perches.com> --- git diff -w shows no difference other than the above 'Miscellanea' (this is against -next, but it applies against Linus' tree with a couple offsets) arch/x86/include/asm/atomic64_32.h | 2 +- drivers/acpi/custom_method.c | 2 +- drivers/acpi/fan.c | 2 +- drivers/gpu/drm/amd/display/dc/core/dc.c | 2 +- drivers/media/i2c/msp3400-kthreads.c | 2 +- drivers/message/fusion/mptsas.c | 2 +- drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c | 2 +- drivers/net/wireless/ath/ath9k/xmit.c | 2 +- drivers/platform/x86/eeepc-laptop.c | 2 +- drivers/rtc/rtc-ab-b5ze-s3.c | 2 +- drivers/scsi/dpt_i2o.c | 2 +- drivers/scsi/sym53c8xx_2/sym_glue.c | 2 +- fs/locks.c | 2 +- fs/ocfs2/stack_user.c | 2 +- fs/xfs/libxfs/xfs_alloc.c | 5 ++--- fs/xfs/xfs_export.c | 2 +- kernel/audit.c | 6 +++--- kernel/trace/trace_printk.c | 4 ++-- lib/raid6/sse2.c | 14 +++++++------- sound/soc/fsl/fsl_dma.c | 2 +- 20 files changed, 30 insertions(+), 31 deletions(-) diff --git a/arch/x86/include/asm/atomic64_32.h b/arch/x86/include/asm/atomic64_32.h index 97c46b8169b7..d4d4883080fa 100644 --- a/arch/x86/include/asm/atomic64_32.h +++ b/arch/x86/include/asm/atomic64_32.h @@ -122,7 +122,7 @@ static inline long long atomic64_read(const atomic64_t *v) long long r; alternative_atomic64(read, "=&A" (r), "c" (v) : "memory"); return r; - } +} /** * atomic64_add_return - add and return diff --git a/drivers/acpi/custom_method.c b/drivers/acpi/custom_method.c index c68e72414a67..e967c1173ba3 100644 --- a/drivers/acpi/custom_method.c +++ b/drivers/acpi/custom_method.c @@ -94,7 +94,7 @@ static void __exit acpi_custom_method_exit(void) { if (cm_dentry) debugfs_remove(cm_dentry); - } +} module_init(acpi_custom_method_init); module_exit(acpi_custom_method_exit); diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c index 6cf4988206f2..3563103590c6 100644 --- a/drivers/acpi/fan.c +++ b/drivers/acpi/fan.c @@ -219,7 +219,7 @@ fan_set_cur_state(struct thermal_cooling_device *cdev, unsigned long state) return fan_set_state_acpi4(device, state); else return fan_set_state(device, state); - } +} static const struct thermal_cooling_device_ops fan_cooling_ops = { .get_max_state = fan_get_max_state, diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c index d1488d5ee028..1e0d1e7c5324 100644 --- a/drivers/gpu/drm/amd/display/dc/core/dc.c +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c @@ -461,7 +461,7 @@ static void disable_dangling_plane(struct dc *dc, struct dc_state *context) ******************************************************************************/ struct dc *dc_create(const struct dc_init_data *init_params) - { +{ struct dc *dc = kzalloc(sizeof(*dc), GFP_KERNEL); unsigned int full_pipe_count; diff --git a/drivers/media/i2c/msp3400-kthreads.c b/drivers/media/i2c/msp3400-kthreads.c index 4dd01e9f553b..dc6cb8d475b3 100644 --- a/drivers/media/i2c/msp3400-kthreads.c +++ b/drivers/media/i2c/msp3400-kthreads.c @@ -885,7 +885,7 @@ static int msp34xxg_modus(struct i2c_client *client) } static void msp34xxg_set_source(struct i2c_client *client, u16 reg, int in) - { +{ struct msp_state *state = to_state(i2c_get_clientdata(client)); int source, matrix; diff --git a/drivers/message/fusion/mptsas.c b/drivers/message/fusion/mptsas.c index 345f6035599e..69a62d23514b 100644 --- a/drivers/message/fusion/mptsas.c +++ b/drivers/message/fusion/mptsas.c @@ -2968,7 +2968,7 @@ mptsas_exp_repmanufacture_info(MPT_ADAPTER *ioc, mutex_unlock(&ioc->sas_mgmt.mutex); out: return ret; - } +} static void mptsas_parse_device_info(struct sas_identify *identify, diff --git a/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c b/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c index 3dd973475125..0ea141ece19e 100644 --- a/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c +++ b/drivers/net/ethernet/qlogic/netxen/netxen_nic_init.c @@ -603,7 +603,7 @@ static struct uni_table_desc *nx_get_table_desc(const u8 *unirom, int section) static int netxen_nic_validate_header(struct netxen_adapter *adapter) - { +{ const u8 *unirom = adapter->fw->data; struct uni_table_desc *directory = (struct uni_table_desc *) &unirom[0]; u32 fw_file_size = adapter->fw->size; diff --git a/drivers/net/wireless/ath/ath9k/xmit.c b/drivers/net/wireless/ath/ath9k/xmit.c index bd438062a6db..baedc7186b10 100644 --- a/drivers/net/wireless/ath/ath9k/xmit.c +++ b/drivers/net/wireless/ath/ath9k/xmit.c @@ -196,7 +196,7 @@ ath_tid_pull(struct ath_atx_tid *tid) } return skb; - } +} static struct sk_buff *ath_tid_dequeue(struct ath_atx_tid *tid) { diff --git a/drivers/platform/x86/eeepc-laptop.c b/drivers/platform/x86/eeepc-laptop.c index 5a681962899c..4c38904a8a32 100644 --- a/drivers/platform/x86/eeepc-laptop.c +++ b/drivers/platform/x86/eeepc-laptop.c @@ -492,7 +492,7 @@ static void eeepc_platform_exit(struct eeepc_laptop *eeepc) * potentially bad time, such as a timer interrupt. */ static void tpd_led_update(struct work_struct *work) - { +{ struct eeepc_laptop *eeepc; eeepc = container_of(work, struct eeepc_laptop, tpd_led_work); diff --git a/drivers/rtc/rtc-ab-b5ze-s3.c b/drivers/rtc/rtc-ab-b5ze-s3.c index a319bf1e49de..ef5c16dfabfa 100644 --- a/drivers/rtc/rtc-ab-b5ze-s3.c +++ b/drivers/rtc/rtc-ab-b5ze-s3.c @@ -648,7 +648,7 @@ static int abb5zes3_rtc_set_alarm(struct device *dev, struct rtc_wkalrm *alarm) ret); return ret; - } +} /* Enable or disable battery low irq generation */ static inline int _abb5zes3_rtc_battery_low_irq_enable(struct regmap *regmap, diff --git a/drivers/scsi/dpt_i2o.c b/drivers/scsi/dpt_i2o.c index fd172b0890d3..a00d822e3142 100644 --- a/drivers/scsi/dpt_i2o.c +++ b/drivers/scsi/dpt_i2o.c @@ -3524,7 +3524,7 @@ static int adpt_i2o_systab_send(adpt_hba* pHba) #endif return ret; - } +} /*============================================================================ diff --git a/drivers/scsi/sym53c8xx_2/sym_glue.c b/drivers/scsi/sym53c8xx_2/sym_glue.c index 791a2182de53..7320d5fe4cbc 100644 --- a/drivers/scsi/sym53c8xx_2/sym_glue.c +++ b/drivers/scsi/sym53c8xx_2/sym_glue.c @@ -1393,7 +1393,7 @@ static struct Scsi_Host *sym_attach(struct scsi_host_template *tpnt, int unit, scsi_host_put(shost); return NULL; - } +} /* diff --git a/fs/locks.c b/fs/locks.c index 21b4dfa289ee..d2399d001afe 100644 --- a/fs/locks.c +++ b/fs/locks.c @@ -559,7 +559,7 @@ static const struct lock_manager_operations lease_manager_ops = { * Initialize a lease, use the default lock manager operations */ static int lease_init(struct file *filp, long type, struct file_lock *fl) - { +{ if (assign_type(fl, type) != 0) return -EINVAL; diff --git a/fs/ocfs2/stack_user.c b/fs/ocfs2/stack_user.c index dae9eb7c441e..d2fb97b173da 100644 --- a/fs/ocfs2/stack_user.c +++ b/fs/ocfs2/stack_user.c @@ -398,7 +398,7 @@ static int ocfs2_control_do_setnode_msg(struct file *file, static int ocfs2_control_do_setversion_msg(struct file *file, struct ocfs2_control_message_setv *msg) - { +{ long major, minor; char *ptr = NULL; struct ocfs2_control_private *p = file->private_data; diff --git a/fs/xfs/libxfs/xfs_alloc.c b/fs/xfs/libxfs/xfs_alloc.c index 0da80019a917..217108f765d5 100644 --- a/fs/xfs/libxfs/xfs_alloc.c +++ b/fs/xfs/libxfs/xfs_alloc.c @@ -2401,7 +2401,7 @@ static bool xfs_agf_verify( struct xfs_mount *mp, struct xfs_buf *bp) - { +{ struct xfs_agf *agf = XFS_BUF_TO_AGF(bp); if (xfs_sb_version_hascrc(&mp->m_sb)) { @@ -2449,8 +2449,7 @@ xfs_agf_verify( be32_to_cpu(agf->agf_refcount_level) > XFS_BTREE_MAXLEVELS)) return false; - return true;; - + return true; } static void diff --git a/fs/xfs/xfs_export.c b/fs/xfs/xfs_export.c index fe1bfee35898..7d5c355d78b5 100644 --- a/fs/xfs/xfs_export.c +++ b/fs/xfs/xfs_export.c @@ -122,7 +122,7 @@ xfs_nfs_get_inode( struct super_block *sb, u64 ino, u32 generation) - { +{ xfs_mount_t *mp = XFS_M(sb); xfs_inode_t *ip; int error; diff --git a/kernel/audit.c b/kernel/audit.c index 227db99b0f19..d97e8f0f73ca 100644 --- a/kernel/audit.c +++ b/kernel/audit.c @@ -443,15 +443,15 @@ static int audit_set_failure(u32 state) * Drop any references inside the auditd connection tracking struct and free * the memory. */ - static void auditd_conn_free(struct rcu_head *rcu) - { +static void auditd_conn_free(struct rcu_head *rcu) +{ struct auditd_connection *ac; ac = container_of(rcu, struct auditd_connection, rcu); put_pid(ac->pid); put_net(ac->net); kfree(ac); - } +} /** * auditd_set - Set/Reset the auditd connection state diff --git a/kernel/trace/trace_printk.c b/kernel/trace/trace_printk.c index ad1d6164e946..50f44b7b2b32 100644 --- a/kernel/trace/trace_printk.c +++ b/kernel/trace/trace_printk.c @@ -196,7 +196,7 @@ struct notifier_block module_trace_bprintk_format_nb = { }; int __trace_bprintk(unsigned long ip, const char *fmt, ...) - { +{ int ret; va_list ap; @@ -214,7 +214,7 @@ int __trace_bprintk(unsigned long ip, const char *fmt, ...) EXPORT_SYMBOL_GPL(__trace_bprintk); int __ftrace_vbprintk(unsigned long ip, const char *fmt, va_list ap) - { +{ if (unlikely(!fmt)) return 0; diff --git a/lib/raid6/sse2.c b/lib/raid6/sse2.c index 1d2276b007ee..8191e1d0d2fb 100644 --- a/lib/raid6/sse2.c +++ b/lib/raid6/sse2.c @@ -91,7 +91,7 @@ static void raid6_sse21_gen_syndrome(int disks, size_t bytes, void **ptrs) static void raid6_sse21_xor_syndrome(int disks, int start, int stop, size_t bytes, void **ptrs) - { +{ u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; @@ -200,9 +200,9 @@ static void raid6_sse22_gen_syndrome(int disks, size_t bytes, void **ptrs) kernel_fpu_end(); } - static void raid6_sse22_xor_syndrome(int disks, int start, int stop, +static void raid6_sse22_xor_syndrome(int disks, int start, int stop, size_t bytes, void **ptrs) - { +{ u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; @@ -265,7 +265,7 @@ static void raid6_sse22_gen_syndrome(int disks, size_t bytes, void **ptrs) asm volatile("sfence" : : : "memory"); kernel_fpu_end(); - } +} const struct raid6_calls raid6_sse2x2 = { raid6_sse22_gen_syndrome, @@ -366,9 +366,9 @@ static void raid6_sse24_gen_syndrome(int disks, size_t bytes, void **ptrs) kernel_fpu_end(); } - static void raid6_sse24_xor_syndrome(int disks, int start, int stop, +static void raid6_sse24_xor_syndrome(int disks, int start, int stop, size_t bytes, void **ptrs) - { +{ u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; @@ -471,7 +471,7 @@ static void raid6_sse24_gen_syndrome(int disks, size_t bytes, void **ptrs) } asm volatile("sfence" : : : "memory"); kernel_fpu_end(); - } +} const struct raid6_calls raid6_sse2x4 = { diff --git a/sound/soc/fsl/fsl_dma.c b/sound/soc/fsl/fsl_dma.c index 0c11f434a374..ec619f51d336 100644 --- a/sound/soc/fsl/fsl_dma.c +++ b/sound/soc/fsl/fsl_dma.c @@ -879,7 +879,7 @@ static const struct snd_pcm_ops fsl_dma_ops = { }; static int fsl_soc_dma_probe(struct platform_device *pdev) - { +{ struct dma_object *dma; struct device_node *np = pdev->dev.of_node; struct device_node *ssi_np;

11 10

[alsa-devel] [PATCH for-4.15] ASoC: rt5514-spi: only enable wakeup when fully initialized
by Brian Norris 19 Dec '17

19 Dec '17

If an rt5514-spi device is probed but the platform hasn't linked it in, we might never fully request the SPI IRQ, nor configure the rt5514 DSP, but we still might try to enable the SPI IRQ (enable_irq_wake()). This is bad, and among other things, can cause the interrupt to trigger every time we try to suspend the system (e.g., because the interrupt trigger setting was never set properly). Instead of setting our wakeup capabilities in the SPI driver probe routine, let's wait until we've actually requested the IRQ. Fixes issues seen on the "kevin" Chromebook (Samsung Chromebook Plus). Fixes: 58f1c07d23cd ("ASoC: rt5514: Voice wakeup support.") Signed-off-by: Brian Norris <briannorris(a)chromium.org> --- The patch that introduced the wakeup was in 4.15-rc1. I think this qualifies as a bugfix, and should probably go into 4.15. Other relevant patches that are useful, and fix similar (or the same) issues: https://patchwork.kernel.org/patch/10067725/ [PATCH] ASoC: rockchip: Use dummy_dai for rt5514 dsp dailink https://patchwork.kernel.org/patch/10116761/ [PATCH for-4.15] ASoC: rt5514: don't assume rt5514 component was "attached" sound/soc/codecs/rt5514-spi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/codecs/rt5514-spi.c b/sound/soc/codecs/rt5514-spi.c index 9255afcf2c3a..48e4b9002caf 100644 --- a/sound/soc/codecs/rt5514-spi.c +++ b/sound/soc/codecs/rt5514-spi.c @@ -289,6 +289,8 @@ static int rt5514_spi_pcm_probe(struct snd_soc_platform *platform) dev_err(&rt5514_spi->dev, "%s Failed to reguest IRQ: %d\n", __func__, ret); + else + device_init_wakeup(rt5514_dsp->dev, true); } return 0; @@ -456,8 +458,6 @@ static int rt5514_spi_probe(struct spi_device *spi) return ret; } - device_init_wakeup(&spi->dev, true); - return 0; } -- 2.15.1.504.g5279b80103-goog

2 1

[alsa-devel] [PATCH 00/10] ASoC: fsl_ssi: Clean up - coding style level
by Nicolin Chen 19 Dec '17

19 Dec '17

==Background== The fsl_ssi driver was designed for PPC originally and then it has been updated to support different modes for i.MX Series, including SDMA, I2S Master mode, AC97 and older i.MXs with FIQ, by different contributors for different use cases in different coding styles. Additionally, in order to fix/work-around hardware bugs and design flaws, the driver made a lot of compromise so now its program flow looks very complicated and it's getting hard to maintain or update. So I am going to clean up the driver in both coding style level and program flow level. ==Introduction== This series of patches is the first set to clean up fsl_ssi driver in the coding style level. Any patch here is not supposed to change the program flow. ==Verification== Theoretically, it only needs code review, build and sanity tests. I have done build and sanity tests on an i.MX 6 platform by building using imx_v6_v7_defconfig and testing with I2S slave/master mode. Nicolin Chen (10): ASoC: fsl_ssi: Remove unused struct device ASoC: fsl_ssi: Rename fsl_ssi_private to fsl_ssi ASoC: fsl_ssi: Cache pdev->dev pointer ASoC: fsl_ssi: Refine all comments ASoC: fsl_ssi: Rename registers and fields macros ASoC: fsl_ssi: Refine indentations and wrappings ASoC: fsl_ssi: Refine printk outputs ASoC: fsl_ssi: Rename cpu_dai parameter to dai ASoC: fsl_ssi: Rename scr_val to scr ASoC: fsl_ssi: Replace fsl_ssi_rxtx_reg_val with fsl_ssi_regvals sound/soc/fsl/fsl_ssi.c | 1338 +++++++++++++++++++------------------------ sound/soc/fsl/fsl_ssi.h | 422 ++++++++------ sound/soc/fsl/fsl_ssi_dbg.c | 59 +- 3 files changed, 846 insertions(+), 973 deletions(-) -- 2.7.4

4 22

[alsa-devel] [PATCH v2 00/11] ASoC: fsl_ssi: Clean up - coding style level
by Nicolin Chen 19 Dec '17

19 Dec '17

==Changelog== v1->v2 * Dropped one patch to remove "struct device" * Revised PATCH-03 "Refine all comments" * Revised PATCH-05 "Refine indentations and wrappings" * Rebased all other patches * Added PATCH-10 "Rename i2smode to i2s_net" * Added PATCH-11 "Define ternary macros to simplify code" # Detialed changes are described in each updated patch. ==Background== The fsl_ssi driver was designed for PPC originally and then it has been updated to support different modes for i.MX Series, including SDMA, I2S Master mode, AC97 and older i.MXs with FIQ, by different contributors for different use cases in different coding styles. Additionally, in order to fix/work-around hardware bugs and design flaws, the driver made a lot of compromise so now its program flow looks very complicated and it's getting hard to maintain or update. So I am going to clean up the driver on both coding style level and program flow level. ==Introduction== This series of patches is the first set to clean up fsl_ssi driver in the coding style level. Any patch here is not supposed to change the program flow. ==Verification== Theoretically, since these patches do not change program flow, they only need code review, build or sanity tests. I have done build and sanity tests on an i.MX6SoloX with WM8962 using imx_v6_v7_defconfig and playback/record tests in I2S Master/Slave modes. Nicolin Chen (11): ASoC: fsl_ssi: Rename fsl_ssi_private to fsl_ssi ASoC: fsl_ssi: Cache pdev->dev pointer ASoC: fsl_ssi: Refine all comments ASoC: fsl_ssi: Rename registers and fields macros ASoC: fsl_ssi: Refine indentations and wrappings ASoC: fsl_ssi: Refine printk outputs ASoC: fsl_ssi: Rename cpu_dai parameter to dai ASoC: fsl_ssi: Rename scr_val to scr ASoC: fsl_ssi: Replace fsl_ssi_rxtx_reg_val with fsl_ssi_regvals ASoC: fsl_ssi: Rename i2smode to i2s_net ASoC: fsl_ssi: Define ternary macros to simplify code sound/soc/fsl/fsl_ssi.c | 1373 +++++++++++++++++++------------------------ sound/soc/fsl/fsl_ssi.h | 427 ++++++++------ sound/soc/fsl/fsl_ssi_dbg.c | 59 +- 3 files changed, 876 insertions(+), 983 deletions(-) -- 2.7.4

3 19