* [PATCH 0/6] ASoC: fsl: fix kernel-doc
@ 2020-07-02 17:22 Pierre-Louis Bossart
2020-07-02 17:22 ` [PATCH 1/6] ASoC: fsl: fsl_ssi_dbg: remove spurious kernel-doc comment start Pierre-Louis Bossart
` (5 more replies)
0 siblings, 6 replies; 15+ messages in thread
From: Pierre-Louis Bossart @ 2020-07-02 17:22 UTC (permalink / raw)
To: alsa-devel; +Cc: tiwai, broonie, Pierre-Louis Bossart
This cleanup patchset is only about kernel-doc, mostly trivial edits
and format correction.
Pierre-Louis Bossart (6):
ASoC: fsl: fsl_ssi_dbg: remove spurious kernel-doc comment start
ASoC: fsl: fsl_ssi: fix kernel-doc
ASoC: fsl: fsl-asoc-card: fix kernel-doc
ASoC: fsl: fsl_spdif: fix kernel-doc
ASoC: fsl: fsl_asrc: fix kernel-doc
ASoC: fsl: fsl_esai: fix kernel-doc
sound/soc/fsl/fsl-asoc-card.c | 21 +++++------
sound/soc/fsl/fsl_asrc.c | 57 +++++++++++++++++++---------
sound/soc/fsl/fsl_esai.c | 32 +++++++++-------
sound/soc/fsl/fsl_spdif.c | 5 ++-
sound/soc/fsl/fsl_ssi.c | 70 ++++++++++++++++++++++-------------
sound/soc/fsl/fsl_ssi_dbg.c | 4 +-
6 files changed, 115 insertions(+), 74 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 1/6] ASoC: fsl: fsl_ssi_dbg: remove spurious kernel-doc comment start
2020-07-02 17:22 [PATCH 0/6] ASoC: fsl: fix kernel-doc Pierre-Louis Bossart
@ 2020-07-02 17:22 ` Pierre-Louis Bossart
2020-07-02 18:43 ` Nicolin Chen
2020-07-02 17:22 ` [PATCH 2/6] ASoC: fsl: fsl_ssi: fix kernel-doc Pierre-Louis Bossart
` (4 subsequent siblings)
5 siblings, 1 reply; 15+ messages in thread
From: Pierre-Louis Bossart @ 2020-07-02 17:22 UTC (permalink / raw)
To: alsa-devel
Cc: Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Pierre-Louis Bossart, Liam Girdwood, Nicolin Chen, broonie,
Fabio Estevam, open list
Fix W=1 warnings. There is no kernel-doc here.
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/fsl/fsl_ssi_dbg.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/fsl/fsl_ssi_dbg.c b/sound/soc/fsl/fsl_ssi_dbg.c
index 2a20ee23dc52..2c46c55f0a88 100644
--- a/sound/soc/fsl/fsl_ssi_dbg.c
+++ b/sound/soc/fsl/fsl_ssi_dbg.c
@@ -78,7 +78,7 @@ void fsl_ssi_dbg_isr(struct fsl_ssi_dbg *dbg, u32 sisr)
dbg->stats.tfe0++;
}
-/**
+/*
* Show the statistics of a flag only if its interrupt is enabled
*
* Compilers will optimize it to a no-op if the interrupt is disabled
@@ -90,7 +90,7 @@ void fsl_ssi_dbg_isr(struct fsl_ssi_dbg *dbg, u32 sisr)
} while (0)
-/**
+/*
* Display the statistics for the current SSI device
*
* To avoid confusion, only show those counts that are enabled
--
2.25.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/6] ASoC: fsl: fsl_ssi: fix kernel-doc
2020-07-02 17:22 [PATCH 0/6] ASoC: fsl: fix kernel-doc Pierre-Louis Bossart
2020-07-02 17:22 ` [PATCH 1/6] ASoC: fsl: fsl_ssi_dbg: remove spurious kernel-doc comment start Pierre-Louis Bossart
@ 2020-07-02 17:22 ` Pierre-Louis Bossart
2020-07-02 18:44 ` Nicolin Chen
2020-07-02 17:22 ` [PATCH 3/6] ASoC: fsl: fsl-asoc-card: " Pierre-Louis Bossart
` (3 subsequent siblings)
5 siblings, 1 reply; 15+ messages in thread
From: Pierre-Louis Bossart @ 2020-07-02 17:22 UTC (permalink / raw)
To: alsa-devel
Cc: Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Pierre-Louis Bossart, Liam Girdwood, Nicolin Chen, broonie,
Fabio Estevam, open list
Fix W=1 warnings. The kernel-doc support is partial, add more
descriptions and follow proper syntax
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/fsl/fsl_ssi.c | 70 ++++++++++++++++++++++++++---------------
1 file changed, 44 insertions(+), 26 deletions(-)
diff --git a/sound/soc/fsl/fsl_ssi.c b/sound/soc/fsl/fsl_ssi.c
index 1a2fa7f18142..7ec80b240563 100644
--- a/sound/soc/fsl/fsl_ssi.c
+++ b/sound/soc/fsl/fsl_ssi.c
@@ -203,12 +203,10 @@ struct fsl_ssi_soc_data {
};
/**
- * fsl_ssi: per-SSI private data
- *
+ * struct fsl_ssi - per-SSI private data
* @regs: Pointer to the regmap registers
* @irq: IRQ of this SSI
* @cpu_dai_drv: CPU DAI driver for this device
- *
* @dai_fmt: DAI configuration this device is currently used with
* @streams: Mask of current active streams: BIT(TX) and BIT(RX)
* @i2s_net: I2S and Network mode configurations of SCR register
@@ -221,38 +219,29 @@ struct fsl_ssi_soc_data {
* @slot_width: Width of each DAI slot
* @slots: Number of slots
* @regvals: Specific RX/TX register settings
- *
* @clk: Clock source to access register
* @baudclk: Clock source to generate bit and frame-sync clocks
* @baudclk_streams: Active streams that are using baudclk
- *
* @regcache_sfcsr: Cache sfcsr register value during suspend and resume
* @regcache_sacnt: Cache sacnt register value during suspend and resume
- *
* @dma_params_tx: DMA transmit parameters
* @dma_params_rx: DMA receive parameters
* @ssi_phys: physical address of the SSI registers
- *
* @fiq_params: FIQ stream filtering parameters
- *
* @card_pdev: Platform_device pointer to register a sound card for PowerPC or
* to register a CODEC platform device for AC97
* @card_name: Platform_device name to register a sound card for PowerPC or
* to register a CODEC platform device for AC97
* @card_idx: The index of SSI to register a sound card for PowerPC or
* to register a CODEC platform device for AC97
- *
* @dbg_stats: Debugging statistics
- *
* @soc: SoC specific data
* @dev: Pointer to &pdev->dev
- *
* @fifo_watermark: The FIFO watermark setting. Notifies DMA when there are
* @fifo_watermark or fewer words in TX fifo or
* @fifo_watermark or more empty words in RX fifo.
* @dma_maxburst: Max number of words to transfer in one go. So far,
* this is always the same as fifo_watermark.
- *
* @ac97_reg_lock: Mutex lock to serialize AC97 register access operations
*/
struct fsl_ssi {
@@ -374,7 +363,9 @@ static bool fsl_ssi_is_i2s_cbm_cfs(struct fsl_ssi *ssi)
}
/**
- * Interrupt handler to gather states
+ * fsl_ssi_irq - Interrupt handler to gather states
+ * @irq: irq number
+ * @dev_id: context
*/
static irqreturn_t fsl_ssi_isr(int irq, void *dev_id)
{
@@ -395,7 +386,10 @@ static irqreturn_t fsl_ssi_isr(int irq, void *dev_id)
}
/**
- * Set SCR, SIER, STCR and SRCR registers with cached values in regvals
+ * fsl_ssi_config_enable - Set SCR, SIER, STCR and SRCR registers with
+ * cached values in regvals
+ * @ssi: SSI context
+ * @tx: direction
*
* Notes:
* 1) For offline_config SoCs, enable all necessary bits of both streams
@@ -474,7 +468,7 @@ static void fsl_ssi_config_enable(struct fsl_ssi *ssi, bool tx)
ssi->streams |= BIT(dir);
}
-/**
+/*
* Exclude bits that are used by the opposite stream
*
* When both streams are active, disabling some bits for the current stream
@@ -495,7 +489,10 @@ static void fsl_ssi_config_enable(struct fsl_ssi *ssi, bool tx)
((vals) & _ssi_xor_shared_bits(vals, avals, aactive))
/**
- * Unset SCR, SIER, STCR and SRCR registers with cached values in regvals
+ * fsl_ssi_config_disable - Unset SCR, SIER, STCR and SRCR registers
+ * with cached values in regvals
+ * @ssi: SSI context
+ * @tx: direction
*
* Notes:
* 1) For offline_config SoCs, to avoid online reconfigurations, disable all
@@ -577,7 +574,9 @@ static void fsl_ssi_tx_ac97_saccst_setup(struct fsl_ssi *ssi)
}
/**
- * Cache critical bits of SIER, SRCR, STCR and SCR to later set them safely
+ * fsl_ssi_setup_regvals - Cache critical bits of SIER, SRCR, STCR and
+ * SCR to later set them safely
+ * @ssi: SSI context
*/
static void fsl_ssi_setup_regvals(struct fsl_ssi *ssi)
{
@@ -661,9 +660,12 @@ static void fsl_ssi_shutdown(struct snd_pcm_substream *substream,
}
/**
- * Configure Digital Audio Interface bit clock
+ * fsl_ssi_set_bclk - Configure Digital Audio Interface bit clock
+ * @substream: ASoC substream
+ * @dai: pointer to DAI
+ * @hw_params: pointers to hw_params
*
- * Note: This function can be only called when using SSI as DAI master
+ * Notes: This function can be only called when using SSI as DAI master
*
* Quick instruction for parameters:
* freq: Output BCLK frequency = samplerate * slots * slot_width
@@ -782,7 +784,10 @@ static int fsl_ssi_set_bclk(struct snd_pcm_substream *substream,
}
/**
- * Configure SSI based on PCM hardware parameters
+ * fsl_ssi_hw_params - Configure SSI based on PCM hardware parameters
+ * @substream: ASoC substream
+ * @hw_params: pointers to hw_params
+ * @dai: pointer to DAI
*
* Notes:
* 1) SxCCR.WL bits are critical bits that require SSI to be temporarily
@@ -997,7 +1002,9 @@ static int _fsl_ssi_set_dai_fmt(struct fsl_ssi *ssi, unsigned int fmt)
}
/**
- * Configure Digital Audio Interface (DAI) Format
+ * fsl_ssi_set_dai_fmt - Configure Digital Audio Interface (DAI) Format
+ * @dai: pointer to DAI
+ * @fmt: format mask
*/
static int fsl_ssi_set_dai_fmt(struct snd_soc_dai *dai, unsigned int fmt)
{
@@ -1011,7 +1018,12 @@ static int fsl_ssi_set_dai_fmt(struct snd_soc_dai *dai, unsigned int fmt)
}
/**
- * Set TDM slot number and slot width
+ * fsl_ssi_set_dai_tdm_slot - Set TDM slot number and slot width
+ * @dai: pointer to DAI
+ * @tx_mask: mask for TX
+ * @rx_mask: mask for RX
+ * @slots: number of slots
+ * @slot_width: number of bits per slot
*/
static int fsl_ssi_set_dai_tdm_slot(struct snd_soc_dai *dai, u32 tx_mask,
u32 rx_mask, int slots, int slot_width)
@@ -1055,7 +1067,10 @@ static int fsl_ssi_set_dai_tdm_slot(struct snd_soc_dai *dai, u32 tx_mask,
}
/**
- * Start or stop SSI and corresponding DMA transaction.
+ * fsl_ssi_trigger - Start or stop SSI and corresponding DMA transaction.
+ * @substream: ASoC substream
+ * @cmd: trigger command
+ * @dai: pointer to DAI
*
* The DMA channel is in external master start and pause mode, which
* means the SSI completely controls the flow of data.
@@ -1239,7 +1254,8 @@ static struct snd_ac97_bus_ops fsl_ssi_ac97_ops = {
};
/**
- * Initialize SSI registers
+ * fsl_ssi_hw_init - Initialize SSI registers
+ * @ssi: SSI context
*/
static int fsl_ssi_hw_init(struct fsl_ssi *ssi)
{
@@ -1268,7 +1284,8 @@ static int fsl_ssi_hw_init(struct fsl_ssi *ssi)
}
/**
- * Clear SSI registers
+ * fsl_ssi_hw_clean - Clear SSI registers
+ * @ssi: SSI context
*/
static void fsl_ssi_hw_clean(struct fsl_ssi *ssi)
{
@@ -1285,7 +1302,8 @@ static void fsl_ssi_hw_clean(struct fsl_ssi *ssi)
regmap_update_bits(ssi->regs, REG_SSI_SCR, SSI_SCR_SSIEN, 0);
}
}
-/**
+
+/*
* Make every character in a string lower-case
*/
static void make_lowercase(char *s)
--
2.25.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 3/6] ASoC: fsl: fsl-asoc-card: fix kernel-doc
2020-07-02 17:22 [PATCH 0/6] ASoC: fsl: fix kernel-doc Pierre-Louis Bossart
2020-07-02 17:22 ` [PATCH 1/6] ASoC: fsl: fsl_ssi_dbg: remove spurious kernel-doc comment start Pierre-Louis Bossart
2020-07-02 17:22 ` [PATCH 2/6] ASoC: fsl: fsl_ssi: fix kernel-doc Pierre-Louis Bossart
@ 2020-07-02 17:22 ` Pierre-Louis Bossart
2020-07-02 18:47 ` Nicolin Chen
2020-07-02 17:22 ` [PATCH 4/6] ASoC: fsl: fsl_spdif: " Pierre-Louis Bossart
` (2 subsequent siblings)
5 siblings, 1 reply; 15+ messages in thread
From: Pierre-Louis Bossart @ 2020-07-02 17:22 UTC (permalink / raw)
To: alsa-devel
Cc: Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Pierre-Louis Bossart, Liam Girdwood, Nicolin Chen, broonie,
Fabio Estevam, open list
Fix W=1 warnings. Kernel-doc syntax was not properly used.
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/fsl/fsl-asoc-card.c | 21 +++++++++------------
1 file changed, 9 insertions(+), 12 deletions(-)
diff --git a/sound/soc/fsl/fsl-asoc-card.c b/sound/soc/fsl/fsl-asoc-card.c
index 57ea1b072326..91220b96e043 100644
--- a/sound/soc/fsl/fsl-asoc-card.c
+++ b/sound/soc/fsl/fsl-asoc-card.c
@@ -33,8 +33,7 @@
#define DAI_FMT_BASE (SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_NB_NF)
/**
- * CODEC private data
- *
+ * struct codec_priv - CODEC private data
* @mclk_freq: Clock rate of MCLK
* @mclk_id: MCLK (or main clock) id for set_sysclk()
* @fll_id: FLL (or secordary clock) id for set_sysclk()
@@ -48,11 +47,10 @@ struct codec_priv {
};
/**
- * CPU private data
- *
- * @sysclk_freq[2]: SYSCLK rates for set_sysclk()
- * @sysclk_dir[2]: SYSCLK directions for set_sysclk()
- * @sysclk_id[2]: SYSCLK ids for set_sysclk()
+ * struct cpu_priv - CPU private data
+ * @sysclk_freq: SYSCLK rates for set_sysclk()
+ * @sysclk_dir: SYSCLK directions for set_sysclk()
+ * @sysclk_id: SYSCLK ids for set_sysclk()
* @slot_width: Slot width of each frame
*
* Note: [1] for tx and [0] for rx
@@ -65,9 +63,8 @@ struct cpu_priv {
};
/**
- * Freescale Generic ASOC card private data
- *
- * @dai_link[3]: DAI link structure including normal one and DPCM link
+ * struct fsl_asoc_card_priv - struct Freescale Generic ASOC card private data
+ * @dai_link: DAI link structure including normal one and DPCM link
* @pdev: platform device pointer
* @codec_priv: CODEC private data
* @cpu_priv: CPU private data
@@ -94,8 +91,8 @@ struct fsl_asoc_card_priv {
char name[32];
};
-/**
- * This dapm route map exsits for DPCM link only.
+/*
+ * This dapm route map exits for DPCM link only.
* The other routes shall go through Device Tree.
*
* Note: keep all ASRC routes in the second half
--
2.25.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 4/6] ASoC: fsl: fsl_spdif: fix kernel-doc
2020-07-02 17:22 [PATCH 0/6] ASoC: fsl: fix kernel-doc Pierre-Louis Bossart
` (2 preceding siblings ...)
2020-07-02 17:22 ` [PATCH 3/6] ASoC: fsl: fsl-asoc-card: " Pierre-Louis Bossart
@ 2020-07-02 17:22 ` Pierre-Louis Bossart
2020-07-02 18:48 ` Nicolin Chen
2020-07-02 17:22 ` [PATCH 5/6] ASoC: fsl: fsl_asrc: " Pierre-Louis Bossart
2020-07-02 17:22 ` [PATCH 6/6] ASoC: fsl: fsl_esai: " Pierre-Louis Bossart
5 siblings, 1 reply; 15+ messages in thread
From: Pierre-Louis Bossart @ 2020-07-02 17:22 UTC (permalink / raw)
To: alsa-devel
Cc: Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Pierre-Louis Bossart, Liam Girdwood, Nicolin Chen, broonie,
Fabio Estevam, open list
Fix W=1 warnings. kernel-doc syntax was not followed and missing parameter
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/fsl/fsl_spdif.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/sound/soc/fsl/fsl_spdif.c b/sound/soc/fsl/fsl_spdif.c
index 5b2689ae63d4..9fb95c6ee7ba 100644
--- a/sound/soc/fsl/fsl_spdif.c
+++ b/sound/soc/fsl/fsl_spdif.c
@@ -81,8 +81,8 @@ struct spdif_mixer_control {
};
/**
- * fsl_spdif_priv: Freescale SPDIF private data
- *
+ * struct fsl_spdif_priv - Freescale SPDIF private data
+ * @soc: SPDIF soc data
* @fsl_spdif_control: SPDIF control data
* @cpu_dai_drv: cpu dai driver
* @pdev: platform device pointer
@@ -100,6 +100,7 @@ struct spdif_mixer_control {
* @spbaclk: SPBA clock (optional, depending on SoC design)
* @dma_params_tx: DMA parameters for transmit channel
* @dma_params_rx: DMA parameters for receive channel
+ * @regcache_srpc: regcache for SRPC
*/
struct fsl_spdif_priv {
const struct fsl_spdif_soc_data *soc;
--
2.25.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 5/6] ASoC: fsl: fsl_asrc: fix kernel-doc
2020-07-02 17:22 [PATCH 0/6] ASoC: fsl: fix kernel-doc Pierre-Louis Bossart
` (3 preceding siblings ...)
2020-07-02 17:22 ` [PATCH 4/6] ASoC: fsl: fsl_spdif: " Pierre-Louis Bossart
@ 2020-07-02 17:22 ` Pierre-Louis Bossart
2020-07-02 18:51 ` Nicolin Chen
2020-07-02 17:22 ` [PATCH 6/6] ASoC: fsl: fsl_esai: " Pierre-Louis Bossart
5 siblings, 1 reply; 15+ messages in thread
From: Pierre-Louis Bossart @ 2020-07-02 17:22 UTC (permalink / raw)
To: alsa-devel
Cc: Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Pierre-Louis Bossart, Liam Girdwood, Nicolin Chen, broonie,
Fabio Estevam, open list
Fix W=1 warnings. fix kernel doc and describe arguments.
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/fsl/fsl_asrc.c | 57 +++++++++++++++++++++++++++-------------
1 file changed, 39 insertions(+), 18 deletions(-)
diff --git a/sound/soc/fsl/fsl_asrc.c b/sound/soc/fsl/fsl_asrc.c
index 462ce9f9ab48..02c81d2e34ad 100644
--- a/sound/soc/fsl/fsl_asrc.c
+++ b/sound/soc/fsl/fsl_asrc.c
@@ -37,7 +37,7 @@ static struct snd_pcm_hw_constraint_list fsl_asrc_rate_constraints = {
.list = supported_asrc_rate,
};
-/**
+/*
* The following tables map the relationship between asrc_inclk/asrc_outclk in
* fsl_asrc.h and the registers of ASRCSR
*/
@@ -68,7 +68,7 @@ static unsigned char output_clk_map_imx53[ASRC_CLK_MAP_LEN] = {
0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7, 0x7,
};
-/**
+/*
* i.MX8QM/i.MX8QXP uses the same map for input and output.
* clk_map_imx8qm[0] is for i.MX8QM asrc0
* clk_map_imx8qm[1] is for i.MX8QM asrc1
@@ -102,16 +102,17 @@ static unsigned char clk_map_imx8qxp[2][ASRC_CLK_MAP_LEN] = {
};
/**
- * Select the pre-processing and post-processing options
+ * fsl_asrc_sel_proc - Select the pre-processing and post-processing options
+ * @inrate: input sample rate
+ * @outrate: output sample rate
+ * @pre_proc: return value for pre-processing option
+ * @post_proc: return value for post-processing option
+ *
* Make sure to exclude following unsupported cases before
* calling this function:
* 1) inrate > 8.125 * outrate
* 2) inrate > 16.125 * outrate
*
- * inrate: input sample rate
- * outrate: output sample rate
- * pre_proc: return value for pre-processing option
- * post_proc: return value for post-processing option
*/
static void fsl_asrc_sel_proc(int inrate, int outrate,
int *pre_proc, int *post_proc)
@@ -148,7 +149,9 @@ static void fsl_asrc_sel_proc(int inrate, int outrate,
}
/**
- * Request ASRC pair
+ * fsl_asrc_request_pair - Request ASRC pair
+ * @channels: number of channels
+ * @pair: pointer to pair
*
* It assigns pair by the order of A->C->B because allocation of pair B,
* within range [ANCA, ANCA+ANCB-1], depends on the channels of pair A
@@ -193,7 +196,8 @@ static int fsl_asrc_request_pair(int channels, struct fsl_asrc_pair *pair)
}
/**
- * Release ASRC pair
+ * fsl_asrc_release_pair - Release ASRC pair
+ * @pair: pair to release
*
* It clears the resource from asrc and releases the occupied channels.
*/
@@ -217,7 +221,10 @@ static void fsl_asrc_release_pair(struct fsl_asrc_pair *pair)
}
/**
- * Configure input and output thresholds
+ * fsl_asrc_set_watermarks- configure input and output thresholds
+ * @pair: pointer to pair
+ * @in: input threshold
+ * @out: output threshold
*/
static void fsl_asrc_set_watermarks(struct fsl_asrc_pair *pair, u32 in, u32 out)
{
@@ -234,7 +241,9 @@ static void fsl_asrc_set_watermarks(struct fsl_asrc_pair *pair, u32 in, u32 out)
}
/**
- * Calculate the total divisor between asrck clock rate and sample rate
+ * fsl_asrc_cal_asrck_divisor - Calculate the total divisor between asrck clock rate and sample rate
+ * @pair: pointer to pair
+ * @div: divider
*
* It follows the formula clk_rate = samplerate * (2 ^ prescaler) * divider
*/
@@ -250,7 +259,10 @@ static u32 fsl_asrc_cal_asrck_divisor(struct fsl_asrc_pair *pair, u32 div)
}
/**
- * Calculate and set the ratio for Ideal Ratio mode only
+ * fsl_asrc_set_ideal_ratio - Calculate and set the ratio for Ideal Ratio mode only
+ * @pair: pointer to pair
+ * @inrate: input rate
+ * @outrate: output rate
*
* The ratio is a 32-bit fixed point value with 26 fractional bits.
*/
@@ -293,7 +305,9 @@ static int fsl_asrc_set_ideal_ratio(struct fsl_asrc_pair *pair,
}
/**
- * Configure the assigned ASRC pair
+ * fsl_asrc_config_pair - Configure the assigned ASRC pair
+ * @pair: pointer to pair
+ * @use_ideal_rate: boolean configuration
*
* It configures those ASRC registers according to a configuration instance
* of struct asrc_config which includes in/output sample rate, width, channel
@@ -508,7 +522,8 @@ static int fsl_asrc_config_pair(struct fsl_asrc_pair *pair, bool use_ideal_rate)
}
/**
- * Start the assigned ASRC pair
+ * fsl_asrc_start_pair - Start the assigned ASRC pair
+ * @pair: pointer to pair
*
* It enables the assigned pair and makes it stopped at the stall level.
*/
@@ -539,7 +554,8 @@ static void fsl_asrc_start_pair(struct fsl_asrc_pair *pair)
}
/**
- * Stop the assigned ASRC pair
+ * fsl_asrc_stop_pair - Stop the assigned ASRC pair
+ * @pair: pointer to pair
*/
static void fsl_asrc_stop_pair(struct fsl_asrc_pair *pair)
{
@@ -552,7 +568,9 @@ static void fsl_asrc_stop_pair(struct fsl_asrc_pair *pair)
}
/**
- * Get DMA channel according to the pair and direction.
+ * fsl_asrc_get_dma_channel- Get DMA channel according to the pair and direction.
+ * @pair: pointer to pair
+ * @dir: DMA direction
*/
static struct dma_chan *fsl_asrc_get_dma_channel(struct fsl_asrc_pair *pair,
bool dir)
@@ -896,7 +914,8 @@ static const struct regmap_config fsl_asrc_regmap_config = {
};
/**
- * Initialize ASRC registers with a default configurations
+ * fsl_asrc_init - Initialize ASRC registers with a default configuration
+ * @asrc: ASRC context
*/
static int fsl_asrc_init(struct fsl_asrc *asrc)
{
@@ -930,7 +949,9 @@ static int fsl_asrc_init(struct fsl_asrc *asrc)
}
/**
- * Interrupt handler for ASRC
+ * fsl_asrc_isr- Interrupt handler for ASRC
+ * @irq: irq number
+ * @dev_id: ASRC context
*/
static irqreturn_t fsl_asrc_isr(int irq, void *dev_id)
{
--
2.25.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 6/6] ASoC: fsl: fsl_esai: fix kernel-doc
2020-07-02 17:22 [PATCH 0/6] ASoC: fsl: fix kernel-doc Pierre-Louis Bossart
` (4 preceding siblings ...)
2020-07-02 17:22 ` [PATCH 5/6] ASoC: fsl: fsl_asrc: " Pierre-Louis Bossart
@ 2020-07-02 17:22 ` Pierre-Louis Bossart
2020-07-02 18:55 ` Nicolin Chen
5 siblings, 1 reply; 15+ messages in thread
From: Pierre-Louis Bossart @ 2020-07-02 17:22 UTC (permalink / raw)
To: alsa-devel
Cc: Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Pierre-Louis Bossart, Liam Girdwood, Nicolin Chen, broonie,
Fabio Estevam, open list
Fix W=1 warnings. Fix kernel-doc syntax and add missing parameters.
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
---
sound/soc/fsl/fsl_esai.c | 32 ++++++++++++++++++--------------
1 file changed, 18 insertions(+), 14 deletions(-)
diff --git a/sound/soc/fsl/fsl_esai.c b/sound/soc/fsl/fsl_esai.c
index cbcb70d6f8c8..a1db69061b4b 100644
--- a/sound/soc/fsl/fsl_esai.c
+++ b/sound/soc/fsl/fsl_esai.c
@@ -22,8 +22,7 @@
SNDRV_PCM_FMTBIT_S24_LE)
/**
- * fsl_esai_soc_data: soc specific data
- *
+ * struct fsl_esai_soc_data - soc specific data
* @imx: for imx platform
* @reset_at_xrun: flags for enable reset operaton
*/
@@ -33,8 +32,7 @@ struct fsl_esai_soc_data {
};
/**
- * fsl_esai: ESAI private data
- *
+ * struct fsl_esai - ESAI private data
* @dma_params_rx: DMA parameters for receive channel
* @dma_params_tx: DMA parameters for transmit channel
* @pdev: platform device pointer
@@ -49,6 +47,8 @@ struct fsl_esai_soc_data {
* @fifo_depth: depth of tx/rx FIFO
* @slot_width: width of each DAI slot
* @slots: number of slots
+ * @tx_mask: slot mask for TX
+ * @rx_mask: slot mask for RX
* @channels: channel num for tx or rx
* @hck_rate: clock rate of desired HCKx clock
* @sck_rate: clock rate of desired SCKx clock
@@ -157,13 +157,15 @@ static irqreturn_t esai_isr(int irq, void *devid)
}
/**
- * This function is used to calculate the divisors of psr, pm, fp and it is
- * supposed to be called in set_dai_sysclk() and set_bclk().
+ * fsl_esai_divisor_cal - This function is used to calculate the
+ * divisors of psr, pm, fp and it is supposed to be called in
+ * set_dai_sysclk() and set_bclk().
*
+ * @dai: pointer to DAI
+ * @tx: current setting is for playback or capture
* @ratio: desired overall ratio for the paticipating dividers
* @usefp: for HCK setting, there is no need to set fp divider
* @fp: bypass other dividers by setting fp directly if fp != 0
- * @tx: current setting is for playback or capture
*/
static int fsl_esai_divisor_cal(struct snd_soc_dai *dai, bool tx, u32 ratio,
bool usefp, u32 fp)
@@ -250,13 +252,12 @@ static int fsl_esai_divisor_cal(struct snd_soc_dai *dai, bool tx, u32 ratio,
}
/**
- * This function mainly configures the clock frequency of MCLK (HCKT/HCKR)
- *
- * @Parameters:
- * clk_id: The clock source of HCKT/HCKR
+ * fsl_esai_set_dai_sysclk - This function mainly configures the clock frequency of MCLK (HCKT/HCKR)
+ * @dai: pointer to DAI
+ * @clk_id: The clock source of HCKT/HCKR
* (Input from outside; output from inside, FSYS or EXTAL)
- * freq: The required clock rate of HCKT/HCKR
- * dir: The clock direction of HCKT/HCKR
+ * @freq: The required clock rate of HCKT/HCKR
+ * @dir: The clock direction of HCKT/HCKR
*
* Note: If the direction is input, we do not care about clk_id.
*/
@@ -358,7 +359,10 @@ static int fsl_esai_set_dai_sysclk(struct snd_soc_dai *dai, int clk_id,
}
/**
- * This function configures the related dividers according to the bclk rate
+ * fsl_esai_set_bclk - This function configures the related dividers according to the bclk rate
+ * @dai: pointer to DAI
+ * @tx: direction boolean
+ * @freq: bclk freq
*/
static int fsl_esai_set_bclk(struct snd_soc_dai *dai, bool tx, u32 freq)
{
--
2.25.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 1/6] ASoC: fsl: fsl_ssi_dbg: remove spurious kernel-doc comment start
2020-07-02 17:22 ` [PATCH 1/6] ASoC: fsl: fsl_ssi_dbg: remove spurious kernel-doc comment start Pierre-Louis Bossart
@ 2020-07-02 18:43 ` Nicolin Chen
0 siblings, 0 replies; 15+ messages in thread
From: Nicolin Chen @ 2020-07-02 18:43 UTC (permalink / raw)
To: Pierre-Louis Bossart
Cc: alsa-devel, Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Liam Girdwood, broonie, Fabio Estevam, open list
On Thu, Jul 02, 2020 at 12:22:22PM -0500, Pierre-Louis Bossart wrote:
> Fix W=1 warnings. There is no kernel-doc here.
>
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 2/6] ASoC: fsl: fsl_ssi: fix kernel-doc
2020-07-02 17:22 ` [PATCH 2/6] ASoC: fsl: fsl_ssi: fix kernel-doc Pierre-Louis Bossart
@ 2020-07-02 18:44 ` Nicolin Chen
0 siblings, 0 replies; 15+ messages in thread
From: Nicolin Chen @ 2020-07-02 18:44 UTC (permalink / raw)
To: Pierre-Louis Bossart
Cc: alsa-devel, Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Liam Girdwood, broonie, Fabio Estevam, open list
On Thu, Jul 02, 2020 at 12:22:23PM -0500, Pierre-Louis Bossart wrote:
> Fix W=1 warnings. The kernel-doc support is partial, add more
> descriptions and follow proper syntax
>
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/6] ASoC: fsl: fsl-asoc-card: fix kernel-doc
2020-07-02 17:22 ` [PATCH 3/6] ASoC: fsl: fsl-asoc-card: " Pierre-Louis Bossart
@ 2020-07-02 18:47 ` Nicolin Chen
2020-07-02 19:03 ` Pierre-Louis Bossart
0 siblings, 1 reply; 15+ messages in thread
From: Nicolin Chen @ 2020-07-02 18:47 UTC (permalink / raw)
To: Pierre-Louis Bossart
Cc: alsa-devel, Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Liam Girdwood, broonie, Fabio Estevam, open list
On Thu, Jul 02, 2020 at 12:22:24PM -0500, Pierre-Louis Bossart wrote:
> Fix W=1 warnings. Kernel-doc syntax was not properly used.
>
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> ---
> sound/soc/fsl/fsl-asoc-card.c | 21 +++++++++------------
> 1 file changed, 9 insertions(+), 12 deletions(-)
>
> diff --git a/sound/soc/fsl/fsl-asoc-card.c b/sound/soc/fsl/fsl-asoc-card.c
> /**
> - * Freescale Generic ASOC card private data
> + * struct fsl_asoc_card_priv - struct Freescale Generic ASOC card private data
Just a nit, can you drop the "struct" before "Freescale"?
Other parts of your changes don't really add this word.
Otherwise,
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 4/6] ASoC: fsl: fsl_spdif: fix kernel-doc
2020-07-02 17:22 ` [PATCH 4/6] ASoC: fsl: fsl_spdif: " Pierre-Louis Bossart
@ 2020-07-02 18:48 ` Nicolin Chen
0 siblings, 0 replies; 15+ messages in thread
From: Nicolin Chen @ 2020-07-02 18:48 UTC (permalink / raw)
To: Pierre-Louis Bossart
Cc: alsa-devel, Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Liam Girdwood, broonie, Fabio Estevam, open list
On Thu, Jul 02, 2020 at 12:22:25PM -0500, Pierre-Louis Bossart wrote:
> Fix W=1 warnings. kernel-doc syntax was not followed and missing parameter
>
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 5/6] ASoC: fsl: fsl_asrc: fix kernel-doc
2020-07-02 17:22 ` [PATCH 5/6] ASoC: fsl: fsl_asrc: " Pierre-Louis Bossart
@ 2020-07-02 18:51 ` Nicolin Chen
0 siblings, 0 replies; 15+ messages in thread
From: Nicolin Chen @ 2020-07-02 18:51 UTC (permalink / raw)
To: Pierre-Louis Bossart
Cc: alsa-devel, Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Liam Girdwood, broonie, Fabio Estevam, open list
On Thu, Jul 02, 2020 at 12:22:26PM -0500, Pierre-Louis Bossart wrote:
> Fix W=1 warnings. fix kernel doc and describe arguments.
>
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 6/6] ASoC: fsl: fsl_esai: fix kernel-doc
2020-07-02 17:22 ` [PATCH 6/6] ASoC: fsl: fsl_esai: " Pierre-Louis Bossart
@ 2020-07-02 18:55 ` Nicolin Chen
2020-07-02 19:03 ` Pierre-Louis Bossart
0 siblings, 1 reply; 15+ messages in thread
From: Nicolin Chen @ 2020-07-02 18:55 UTC (permalink / raw)
To: Pierre-Louis Bossart
Cc: alsa-devel, Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Liam Girdwood, broonie, Fabio Estevam, open list
On Thu, Jul 02, 2020 at 12:22:27PM -0500, Pierre-Louis Bossart wrote:
> Fix W=1 warnings. Fix kernel-doc syntax and add missing parameters.
>
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> + * fsl_esai_set_dai_sysclk - This function mainly configures the clock frequency of MCLK (HCKT/HCKR)
Can drop "This function mainly"
> /**
> - * This function configures the related dividers according to the bclk rate
> + * fsl_esai_set_bclk - This function configures the related dividers according to the bclk rate
Here too -- dropping "This function"
Otherwise,
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
Thanks!
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/6] ASoC: fsl: fsl-asoc-card: fix kernel-doc
2020-07-02 18:47 ` Nicolin Chen
@ 2020-07-02 19:03 ` Pierre-Louis Bossart
0 siblings, 0 replies; 15+ messages in thread
From: Pierre-Louis Bossart @ 2020-07-02 19:03 UTC (permalink / raw)
To: Nicolin Chen
Cc: alsa-devel, Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Liam Girdwood, broonie, Fabio Estevam, open list
On 7/2/20 1:47 PM, Nicolin Chen wrote:
> On Thu, Jul 02, 2020 at 12:22:24PM -0500, Pierre-Louis Bossart wrote:
>> Fix W=1 warnings. Kernel-doc syntax was not properly used.
>>
>> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
>> ---
>> sound/soc/fsl/fsl-asoc-card.c | 21 +++++++++------------
>> 1 file changed, 9 insertions(+), 12 deletions(-)
>>
>> diff --git a/sound/soc/fsl/fsl-asoc-card.c b/sound/soc/fsl/fsl-asoc-card.c
>
>> /**
>> - * Freescale Generic ASOC card private data
>> + * struct fsl_asoc_card_priv - struct Freescale Generic ASOC card private data
>
> Just a nit, can you drop the "struct" before "Freescale"?
> Other parts of your changes don't really add this word.
will do, thanks for spotting this.
> Otherwise,
> Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 6/6] ASoC: fsl: fsl_esai: fix kernel-doc
2020-07-02 18:55 ` Nicolin Chen
@ 2020-07-02 19:03 ` Pierre-Louis Bossart
0 siblings, 0 replies; 15+ messages in thread
From: Pierre-Louis Bossart @ 2020-07-02 19:03 UTC (permalink / raw)
To: Nicolin Chen
Cc: alsa-devel, Timur Tabi, Xiubo Li, tiwai,
open list:FREESCALE SOC SOUND DRIVERS, Takashi Iwai,
Liam Girdwood, broonie, Fabio Estevam, open list
On 7/2/20 1:55 PM, Nicolin Chen wrote:
> On Thu, Jul 02, 2020 at 12:22:27PM -0500, Pierre-Louis Bossart wrote:
>> Fix W=1 warnings. Fix kernel-doc syntax and add missing parameters.
>>
>> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
>
>> + * fsl_esai_set_dai_sysclk - This function mainly configures the clock frequency of MCLK (HCKT/HCKR)
>
> Can drop "This function mainly"
>
>> /**
>> - * This function configures the related dividers according to the bclk rate
>> + * fsl_esai_set_bclk - This function configures the related dividers according to the bclk rate
>
> Here too -- dropping "This function"
will fix both, thanks for reviewing the edits.
>
> Otherwise,
> Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
>
> Thanks!
>
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2020-07-02 19:05 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-02 17:22 [PATCH 0/6] ASoC: fsl: fix kernel-doc Pierre-Louis Bossart
2020-07-02 17:22 ` [PATCH 1/6] ASoC: fsl: fsl_ssi_dbg: remove spurious kernel-doc comment start Pierre-Louis Bossart
2020-07-02 18:43 ` Nicolin Chen
2020-07-02 17:22 ` [PATCH 2/6] ASoC: fsl: fsl_ssi: fix kernel-doc Pierre-Louis Bossart
2020-07-02 18:44 ` Nicolin Chen
2020-07-02 17:22 ` [PATCH 3/6] ASoC: fsl: fsl-asoc-card: " Pierre-Louis Bossart
2020-07-02 18:47 ` Nicolin Chen
2020-07-02 19:03 ` Pierre-Louis Bossart
2020-07-02 17:22 ` [PATCH 4/6] ASoC: fsl: fsl_spdif: " Pierre-Louis Bossart
2020-07-02 18:48 ` Nicolin Chen
2020-07-02 17:22 ` [PATCH 5/6] ASoC: fsl: fsl_asrc: " Pierre-Louis Bossart
2020-07-02 18:51 ` Nicolin Chen
2020-07-02 17:22 ` [PATCH 6/6] ASoC: fsl: fsl_esai: " Pierre-Louis Bossart
2020-07-02 18:55 ` Nicolin Chen
2020-07-02 19:03 ` Pierre-Louis Bossart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).