On 7/9/21 8:53 PM, Ashok Reddy Soma wrote:
> From: Michal Simek <michal.si...@xilinx.com>
>
> Fix these kernel doc warnings:
> drivers/mmc/zynq_sdhci.c:181: warning: contents before sections
> drivers/mmc/zynq_sdhci.c:236: warning: contents before sections
> drivers/mmc/zynq_sdhci.c:291: warning: contents before sections
> drivers/mmc/zynq_sdhci.c:297: warning: Function parameter or member 'degrees'
> not described in 'sdhci_versal_sdcardclk_set_phase'
> drivers/mmc/zynq_sdhci.c:354: warning: contents before sections
> drivers/mmc/zynq_sdhci.c:360: warning: Function parameter or member 'degrees'
> not described in 'sdhci_versal_sampleclk_set_phase'
> drivers/mmc/zynq_sdhci.c:467: warning: contents before sections
>
> Signed-off-by: Michal Simek <michal.si...@xilinx.com>
> Signed-off-by: Ashok Reddy Soma <ashok.reddy.s...@xilinx.com>
Reviewed-by: Jaehoon Chung <jh80.ch...@samsung.com>
Best Regards,
Jaehoon Chung
> ---
>
> Changes in v3:
> - Change "@degree" to "@degrees:" in sdhci_zynqmp_sampleclk_set_phase()
> and sdhci_versal_sdcardclk_set_phase() and update description with
> these warnings.
>
> Changes in v2:
> - Removed @degree warning from commit description since it is fixed in
> patch 1/6.
>
> drivers/mmc/zynq_sdhci.c | 24 ++++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/mmc/zynq_sdhci.c b/drivers/mmc/zynq_sdhci.c
> index 95d42ccef4..47862c0bf5 100644
> --- a/drivers/mmc/zynq_sdhci.c
> +++ b/drivers/mmc/zynq_sdhci.c
> @@ -181,11 +181,11 @@ static int arasan_sdhci_execute_tuning(struct mmc *mmc,
> u8 opcode)
> /**
> * sdhci_zynqmp_sdcardclk_set_phase - Set the SD Output Clock Tap Delays
> *
> - * Set the SD Output Clock Tap Delays for Output path
> - *
> * @host: Pointer to the sdhci_host structure.
> * @degrees: The clock phase shift between 0 - 359.
> * Return: 0
> + *
> + * Set the SD Output Clock Tap Delays for Output path
> */
> static int sdhci_zynqmp_sdcardclk_set_phase(struct sdhci_host *host,
> int degrees)
> @@ -237,11 +237,11 @@ static int sdhci_zynqmp_sdcardclk_set_phase(struct
> sdhci_host *host,
> /**
> * sdhci_zynqmp_sampleclk_set_phase - Set the SD Input Clock Tap Delays
> *
> - * Set the SD Input Clock Tap Delays for Input path
> - *
> * @host: Pointer to the sdhci_host structure.
> * @degrees: The clock phase shift between 0 - 359.
> * Return: 0
> + *
> + * Set the SD Input Clock Tap Delays for Input path
> */
> static int sdhci_zynqmp_sampleclk_set_phase(struct sdhci_host *host,
> int degrees)
> @@ -293,11 +293,11 @@ static int sdhci_zynqmp_sampleclk_set_phase(struct
> sdhci_host *host,
> /**
> * sdhci_versal_sdcardclk_set_phase - Set the SD Output Clock Tap Delays
> *
> - * Set the SD Output Clock Tap Delays for Output path
> - *
> * @host: Pointer to the sdhci_host structure.
> - * @degrees The clock phase shift between 0 - 359.
> + * @degrees: The clock phase shift between 0 - 359.
> * Return: 0
> + *
> + * Set the SD Output Clock Tap Delays for Output path
> */
> static int sdhci_versal_sdcardclk_set_phase(struct sdhci_host *host,
> int degrees)
> @@ -355,11 +355,11 @@ static int sdhci_versal_sdcardclk_set_phase(struct
> sdhci_host *host,
> /**
> * sdhci_versal_sampleclk_set_phase - Set the SD Input Clock Tap Delays
> *
> - * Set the SD Input Clock Tap Delays for Input path
> - *
> * @host: Pointer to the sdhci_host structure.
> - * @degrees The clock phase shift between 0 - 359.
> + * @degrees: The clock phase shift between 0 - 359.
> * Return: 0
> + *
> + * Set the SD Input Clock Tap Delays for Input path
> */
> static int sdhci_versal_sampleclk_set_phase(struct sdhci_host *host,
> int degrees)
> @@ -467,9 +467,9 @@ static void arasan_dt_read_clk_phase(struct udevice *dev,
> unsigned char timing,
> /**
> * arasan_dt_parse_clk_phases - Read Tap Delay values from DT
> *
> - * Called at initialization to parse the values of Tap Delays.
> - *
> * @dev: Pointer to our struct udevice.
> + *
> + * Called at initialization to parse the values of Tap Delays.
> */
> static void arasan_dt_parse_clk_phases(struct udevice *dev)
> {
>
