* [PATCH 00/10] Rid W=1 warnings from Bus
@ 2021-05-26 8:10 Lee Jones
2021-05-26 8:10 ` [PATCH 07/10] bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg' Lee Jones
2021-05-26 12:19 ` [PATCH 00/10] Rid W=1 warnings from Bus Laurentiu Tudor
0 siblings, 2 replies; 8+ messages in thread
From: Lee Jones @ 2021-05-26 8:10 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Andy Gross, Bjorn Andersson, German Rivera,
Laurentiu Tudor, Linus Walleij, linux-arm-msm, linux-omap,
Philipp Zabel, Stuart Yoder, Tony Lindgren
This set is part of a larger effort attempting to clean-up W=1
kernel builds, which are currently overwhelmingly riddled with
niggly little warnings.
Lee Jones (10):
bus: fsl-mc: mc-io: Supply function names for 'fsl_create_mc_io()' and
'fsl_destroy_mc_io()'
bus: fsl-mc: mc-sys: Supply missing function names in kernel-doc
headers
bus: fsl-mc: fsl-mc-bus: Demote a bunch of non-conformant kernel-doc
headers and help others
bus: fsl-mc: dprc: Fix a couple of misspelling and formatting issues
bus: fsl-mc: dprc-driver: Fix some missing/incorrect function
parameter descriptions
bus: fsl-mc: fsl-mc-allocator: Fix misspelling of 'new_mc_adev' and
demote non-kernel-doc headers
bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg'
bus: fsl-mc-msi: Fix a little doc-rot pertaining to 'np' to 'fwnode'
conversion
bus: ti-sysc: Correct misdocumentation of 'sysc_ioremap()'
bus: fsl-mc: mc-io: Correct misdocumentation of 'dpmcp_dev' param
drivers/bus/fsl-mc/dprc-driver.c | 8 +++++---
drivers/bus/fsl-mc/dprc.c | 4 ++--
drivers/bus/fsl-mc/fsl-mc-allocator.c | 10 +++++-----
drivers/bus/fsl-mc/fsl-mc-bus.c | 19 ++++++++++---------
drivers/bus/fsl-mc/fsl-mc-msi.c | 2 +-
drivers/bus/fsl-mc/mc-io.c | 6 +++---
drivers/bus/fsl-mc/mc-sys.c | 19 ++++++++++---------
drivers/bus/qcom-ebi2.c | 4 ++--
drivers/bus/ti-sysc.c | 2 +-
9 files changed, 39 insertions(+), 35 deletions(-)
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: German Rivera <German.Rivera@freescale.com>
Cc: Laurentiu Tudor <laurentiu.tudor@nxp.com>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-omap@vger.kernel.org
Cc: Philipp Zabel <p.zabel@pengutronix.de>
Cc: Stuart Yoder <stuyoder@gmail.com>
Cc: Tony Lindgren <tony@atomide.com>
--
2.31.1
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 07/10] bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg'
2021-05-26 8:10 [PATCH 00/10] Rid W=1 warnings from Bus Lee Jones
@ 2021-05-26 8:10 ` Lee Jones
2021-05-26 22:33 ` Linus Walleij
2021-05-27 2:22 ` Bjorn Andersson
2021-05-26 12:19 ` [PATCH 00/10] Rid W=1 warnings from Bus Laurentiu Tudor
1 sibling, 2 replies; 8+ messages in thread
From: Lee Jones @ 2021-05-26 8:10 UTC (permalink / raw)
To: lee.jones
Cc: linux-kernel, Andy Gross, Bjorn Andersson, Linus Walleij, linux-arm-msm
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(-)
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
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 07/10] bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg'
2021-05-26 8:10 ` [PATCH 07/10] bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg' Lee Jones
@ 2021-05-26 22:33 ` Linus Walleij
2021-05-27 2:22 ` Bjorn Andersson
1 sibling, 0 replies; 8+ messages in thread
From: Linus Walleij @ 2021-05-26 22:33 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Andy Gross, Bjorn Andersson, MSM
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 07/10] bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg'
2021-05-26 8:10 ` [PATCH 07/10] bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg' Lee Jones
2021-05-26 22:33 ` Linus Walleij
@ 2021-05-27 2:22 ` Bjorn Andersson
1 sibling, 0 replies; 8+ messages in thread
From: Bjorn Andersson @ 2021-05-27 2:22 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Andy Gross, Linus Walleij, linux-arm-msm
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
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 00/10] Rid W=1 warnings from Bus
2021-05-26 8:10 [PATCH 00/10] Rid W=1 warnings from Bus Lee Jones
2021-05-26 8:10 ` [PATCH 07/10] bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg' Lee Jones
@ 2021-05-26 12:19 ` Laurentiu Tudor
2021-06-17 9:49 ` Lee Jones
1 sibling, 1 reply; 8+ messages in thread
From: Laurentiu Tudor @ 2021-05-26 12:19 UTC (permalink / raw)
To: Lee Jones
Cc: linux-kernel, Andy Gross, Bjorn Andersson, German Rivera,
Linus Walleij, linux-arm-msm, linux-omap, Philipp Zabel,
Stuart Yoder, Tony Lindgren
Hi Lee,
On 5/26/2021 11:10 AM, Lee Jones wrote:
> This set is part of a larger effort attempting to clean-up W=1
> kernel builds, which are currently overwhelmingly riddled with
> niggly little warnings.
>
> Lee Jones (10):
> bus: fsl-mc: mc-io: Supply function names for 'fsl_create_mc_io()' and
> 'fsl_destroy_mc_io()'
> bus: fsl-mc: mc-sys: Supply missing function names in kernel-doc
> headers
> bus: fsl-mc: fsl-mc-bus: Demote a bunch of non-conformant kernel-doc
> headers and help others
> bus: fsl-mc: dprc: Fix a couple of misspelling and formatting issues
> bus: fsl-mc: dprc-driver: Fix some missing/incorrect function
> parameter descriptions
> bus: fsl-mc: fsl-mc-allocator: Fix misspelling of 'new_mc_adev' and
> demote non-kernel-doc headers
> bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg'
> bus: fsl-mc-msi: Fix a little doc-rot pertaining to 'np' to 'fwnode'
> conversion
> bus: ti-sysc: Correct misdocumentation of 'sysc_ioremap()'
> bus: fsl-mc: mc-io: Correct misdocumentation of 'dpmcp_dev' param
>
> drivers/bus/fsl-mc/dprc-driver.c | 8 +++++---
> drivers/bus/fsl-mc/dprc.c | 4 ++--
> drivers/bus/fsl-mc/fsl-mc-allocator.c | 10 +++++-----
> drivers/bus/fsl-mc/fsl-mc-bus.c | 19 ++++++++++---------
> drivers/bus/fsl-mc/fsl-mc-msi.c | 2 +-
> drivers/bus/fsl-mc/mc-io.c | 6 +++---
> drivers/bus/fsl-mc/mc-sys.c | 19 ++++++++++---------
Thanks for this. For drivers/bus/fsl-mc/*:
Reviewed-by: Laurentiu Tudor <laurentiu.tudor@nxp.com>
---
Best Regards, Laurentiu
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 00/10] Rid W=1 warnings from Bus
2021-05-26 12:19 ` [PATCH 00/10] Rid W=1 warnings from Bus Laurentiu Tudor
@ 2021-06-17 9:49 ` Lee Jones
2021-06-17 10:11 ` Laurentiu Tudor
0 siblings, 1 reply; 8+ messages in thread
From: Lee Jones @ 2021-06-17 9:49 UTC (permalink / raw)
To: Laurentiu Tudor
Cc: linux-kernel, Andy Gross, Bjorn Andersson, German Rivera,
Linus Walleij, linux-arm-msm, linux-omap, Philipp Zabel,
Stuart Yoder, Tony Lindgren
On Wed, 26 May 2021, Laurentiu Tudor wrote:
> Hi Lee,
>
> On 5/26/2021 11:10 AM, Lee Jones wrote:
> > This set is part of a larger effort attempting to clean-up W=1
> > kernel builds, which are currently overwhelmingly riddled with
> > niggly little warnings.
> >
> > Lee Jones (10):
> > bus: fsl-mc: mc-io: Supply function names for 'fsl_create_mc_io()' and
> > 'fsl_destroy_mc_io()'
> > bus: fsl-mc: mc-sys: Supply missing function names in kernel-doc
> > headers
> > bus: fsl-mc: fsl-mc-bus: Demote a bunch of non-conformant kernel-doc
> > headers and help others
> > bus: fsl-mc: dprc: Fix a couple of misspelling and formatting issues
> > bus: fsl-mc: dprc-driver: Fix some missing/incorrect function
> > parameter descriptions
> > bus: fsl-mc: fsl-mc-allocator: Fix misspelling of 'new_mc_adev' and
> > demote non-kernel-doc headers
> > bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg'
> > bus: fsl-mc-msi: Fix a little doc-rot pertaining to 'np' to 'fwnode'
> > conversion
> > bus: ti-sysc: Correct misdocumentation of 'sysc_ioremap()'
> > bus: fsl-mc: mc-io: Correct misdocumentation of 'dpmcp_dev' param
> >
> > drivers/bus/fsl-mc/dprc-driver.c | 8 +++++---
> > drivers/bus/fsl-mc/dprc.c | 4 ++--
> > drivers/bus/fsl-mc/fsl-mc-allocator.c | 10 +++++-----
> > drivers/bus/fsl-mc/fsl-mc-bus.c | 19 ++++++++++---------
> > drivers/bus/fsl-mc/fsl-mc-msi.c | 2 +-
> > drivers/bus/fsl-mc/mc-io.c | 6 +++---
> > drivers/bus/fsl-mc/mc-sys.c | 19 ++++++++++---------
>
> Thanks for this. For drivers/bus/fsl-mc/*:
>
> Reviewed-by: Laurentiu Tudor <laurentiu.tudor@nxp.com>
Any idea who will take the 'fsl-mc' patches please?
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 00/10] Rid W=1 warnings from Bus
2021-06-17 9:49 ` Lee Jones
@ 2021-06-17 10:11 ` Laurentiu Tudor
2021-06-17 10:57 ` Greg KH
0 siblings, 1 reply; 8+ messages in thread
From: Laurentiu Tudor @ 2021-06-17 10:11 UTC (permalink / raw)
To: Lee Jones, Greg KH
Cc: linux-kernel, Andy Gross, Bjorn Andersson, German Rivera,
Linus Walleij, linux-arm-msm, linux-omap, Philipp Zabel,
Stuart Yoder, Tony Lindgren
On 6/17/2021 12:49 PM, Lee Jones wrote:
> On Wed, 26 May 2021, Laurentiu Tudor wrote:
>
>> Hi Lee,
>>
>> On 5/26/2021 11:10 AM, Lee Jones wrote:
>>> This set is part of a larger effort attempting to clean-up W=1
>>> kernel builds, which are currently overwhelmingly riddled with
>>> niggly little warnings.
>>>
>>> Lee Jones (10):
>>> bus: fsl-mc: mc-io: Supply function names for 'fsl_create_mc_io()' and
>>> 'fsl_destroy_mc_io()'
>>> bus: fsl-mc: mc-sys: Supply missing function names in kernel-doc
>>> headers
>>> bus: fsl-mc: fsl-mc-bus: Demote a bunch of non-conformant kernel-doc
>>> headers and help others
>>> bus: fsl-mc: dprc: Fix a couple of misspelling and formatting issues
>>> bus: fsl-mc: dprc-driver: Fix some missing/incorrect function
>>> parameter descriptions
>>> bus: fsl-mc: fsl-mc-allocator: Fix misspelling of 'new_mc_adev' and
>>> demote non-kernel-doc headers
>>> bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg'
>>> bus: fsl-mc-msi: Fix a little doc-rot pertaining to 'np' to 'fwnode'
>>> conversion
>>> bus: ti-sysc: Correct misdocumentation of 'sysc_ioremap()'
>>> bus: fsl-mc: mc-io: Correct misdocumentation of 'dpmcp_dev' param
>>>
>>> drivers/bus/fsl-mc/dprc-driver.c | 8 +++++---
>>> drivers/bus/fsl-mc/dprc.c | 4 ++--
>>> drivers/bus/fsl-mc/fsl-mc-allocator.c | 10 +++++-----
>>> drivers/bus/fsl-mc/fsl-mc-bus.c | 19 ++++++++++---------
>>> drivers/bus/fsl-mc/fsl-mc-msi.c | 2 +-
>>> drivers/bus/fsl-mc/mc-io.c | 6 +++---
>>> drivers/bus/fsl-mc/mc-sys.c | 19 ++++++++++---------
>>
>> Thanks for this. For drivers/bus/fsl-mc/*:
>>
>> Reviewed-by: Laurentiu Tudor <laurentiu.tudor@nxp.com>
>
> Any idea who will take the 'fsl-mc' patches please?
>
Usually GregKH (added in the thread) picks them up through his char-misc
tree.
---
Best Regards, Laurentiu
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 00/10] Rid W=1 warnings from Bus
2021-06-17 10:11 ` Laurentiu Tudor
@ 2021-06-17 10:57 ` Greg KH
0 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2021-06-17 10:57 UTC (permalink / raw)
To: Laurentiu Tudor
Cc: Lee Jones, linux-kernel, Andy Gross, Bjorn Andersson,
German Rivera, Linus Walleij, linux-arm-msm, linux-omap,
Philipp Zabel, Stuart Yoder, Tony Lindgren
On Thu, Jun 17, 2021 at 01:11:39PM +0300, Laurentiu Tudor wrote:
>
>
> On 6/17/2021 12:49 PM, Lee Jones wrote:
> > On Wed, 26 May 2021, Laurentiu Tudor wrote:
> >
> >> Hi Lee,
> >>
> >> On 5/26/2021 11:10 AM, Lee Jones wrote:
> >>> This set is part of a larger effort attempting to clean-up W=1
> >>> kernel builds, which are currently overwhelmingly riddled with
> >>> niggly little warnings.
> >>>
> >>> Lee Jones (10):
> >>> bus: fsl-mc: mc-io: Supply function names for 'fsl_create_mc_io()' and
> >>> 'fsl_destroy_mc_io()'
> >>> bus: fsl-mc: mc-sys: Supply missing function names in kernel-doc
> >>> headers
> >>> bus: fsl-mc: fsl-mc-bus: Demote a bunch of non-conformant kernel-doc
> >>> headers and help others
> >>> bus: fsl-mc: dprc: Fix a couple of misspelling and formatting issues
> >>> bus: fsl-mc: dprc-driver: Fix some missing/incorrect function
> >>> parameter descriptions
> >>> bus: fsl-mc: fsl-mc-allocator: Fix misspelling of 'new_mc_adev' and
> >>> demote non-kernel-doc headers
> >>> bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg'
> >>> bus: fsl-mc-msi: Fix a little doc-rot pertaining to 'np' to 'fwnode'
> >>> conversion
> >>> bus: ti-sysc: Correct misdocumentation of 'sysc_ioremap()'
> >>> bus: fsl-mc: mc-io: Correct misdocumentation of 'dpmcp_dev' param
> >>>
> >>> drivers/bus/fsl-mc/dprc-driver.c | 8 +++++---
> >>> drivers/bus/fsl-mc/dprc.c | 4 ++--
> >>> drivers/bus/fsl-mc/fsl-mc-allocator.c | 10 +++++-----
> >>> drivers/bus/fsl-mc/fsl-mc-bus.c | 19 ++++++++++---------
> >>> drivers/bus/fsl-mc/fsl-mc-msi.c | 2 +-
> >>> drivers/bus/fsl-mc/mc-io.c | 6 +++---
> >>> drivers/bus/fsl-mc/mc-sys.c | 19 ++++++++++---------
> >>
> >> Thanks for this. For drivers/bus/fsl-mc/*:
> >>
> >> Reviewed-by: Laurentiu Tudor <laurentiu.tudor@nxp.com>
> >
> > Any idea who will take the 'fsl-mc' patches please?
> >
>
> Usually GregKH (added in the thread) picks them up through his char-misc
> tree.
If you resend just the fsl-mc patches as a series, I can pick them up
that way. Otherwise trying to pick out individual ones here is pretty
much impossible...
thanks,
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-06-17 10:57 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-26 8:10 [PATCH 00/10] Rid W=1 warnings from Bus Lee Jones
2021-05-26 8:10 ` [PATCH 07/10] bus: qcom-ebi2: Fix incorrect documentation for '{slow,fast}_cfg' Lee Jones
2021-05-26 22:33 ` Linus Walleij
2021-05-27 2:22 ` Bjorn Andersson
2021-05-26 12:19 ` [PATCH 00/10] Rid W=1 warnings from Bus Laurentiu Tudor
2021-06-17 9:49 ` Lee Jones
2021-06-17 10:11 ` Laurentiu Tudor
2021-06-17 10:57 ` Greg KH
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).