diff mbox series

[07/10] bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg'

Message ID 20210526081038.544942-8-lee.jones@linaro.org
State Accepted
Commit ee704ebfd548a67e521f6f86949facc56ebc59e7
Headers show
Series None | expand

Commit Message

Lee Jones May 26, 2021, 8:10 a.m. UTC
Fixes the following W=1 kernel build warning(s):

 drivers/bus/qcom-ebi2.c:112: warning: Function parameter or member 'slow_cfg' not described in 'cs_data'
 drivers/bus/qcom-ebi2.c:112: warning: Function parameter or member 'fast_cfg' not described in 'cs_data'

Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>

---
 drivers/bus/qcom-ebi2.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
2.31.1

Comments

Linus Walleij May 26, 2021, 10:33 p.m. UTC | #1
On Wed, May 26, 2021 at 10:10 AM Lee Jones <lee.jones@linaro.org> wrote:

> Fixes the following W=1 kernel build warning(s):

>

>  drivers/bus/qcom-ebi2.c:112: warning: Function parameter or member 'slow_cfg' not described in 'cs_data'

>  drivers/bus/qcom-ebi2.c:112: warning: Function parameter or member 'fast_cfg' not described in 'cs_data'

>

> Cc: Andy Gross <agross@kernel.org>

> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>

> Cc: Linus Walleij <linus.walleij@linaro.org>

> Cc: linux-arm-msm@vger.kernel.org

> Signed-off-by: Lee Jones <lee.jones@linaro.org>


Reviewed-by: Linus Walleij <linus.walleij@linaro.org


Thanks Lee!

Yours,
Linus Walleij
Bjorn Andersson May 27, 2021, 2:22 a.m. UTC | #2
On Wed 26 May 03:10 CDT 2021, Lee Jones wrote:

> Fixes the following W=1 kernel build warning(s):

> 

>  drivers/bus/qcom-ebi2.c:112: warning: Function parameter or member 'slow_cfg' not described in 'cs_data'

>  drivers/bus/qcom-ebi2.c:112: warning: Function parameter or member 'fast_cfg' not described in 'cs_data'

> 

> Cc: Andy Gross <agross@kernel.org>

> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>

> Cc: Linus Walleij <linus.walleij@linaro.org>

> Cc: linux-arm-msm@vger.kernel.org

> Signed-off-by: Lee Jones <lee.jones@linaro.org>


Thanks Lee, I've picked this up in the qcom tree.

Regards,
Bjorn

> ---

>  drivers/bus/qcom-ebi2.c | 4 ++--

>  1 file changed, 2 insertions(+), 2 deletions(-)

> 

> diff --git a/drivers/bus/qcom-ebi2.c b/drivers/bus/qcom-ebi2.c

> index 0b8f53a688b8a..663c827492223 100644

> --- a/drivers/bus/qcom-ebi2.c

> +++ b/drivers/bus/qcom-ebi2.c

> @@ -102,8 +102,8 @@

>  /**

>   * struct cs_data - struct with info on a chipselect setting

>   * @enable_mask: mask to enable the chipselect in the EBI2 config

> - * @slow_cfg0: offset to XMEMC slow CS config

> - * @fast_cfg1: offset to XMEMC fast CS config

> + * @slow_cfg: offset to XMEMC slow CS config

> + * @fast_cfg: offset to XMEMC fast CS config

>   */

>  struct cs_data {

>  	u32 enable_mask;

> -- 

> 2.31.1

>
diff mbox series

Patch

diff --git a/drivers/bus/qcom-ebi2.c b/drivers/bus/qcom-ebi2.c
index 0b8f53a688b8a..663c827492223 100644
--- a/drivers/bus/qcom-ebi2.c
+++ b/drivers/bus/qcom-ebi2.c
@@ -102,8 +102,8 @@ 
 /**
  * struct cs_data - struct with info on a chipselect setting
  * @enable_mask: mask to enable the chipselect in the EBI2 config
- * @slow_cfg0: offset to XMEMC slow CS config
- * @fast_cfg1: offset to XMEMC fast CS config
+ * @slow_cfg: offset to XMEMC slow CS config
+ * @fast_cfg: offset to XMEMC fast CS config
  */
 struct cs_data {
 	u32 enable_mask;