| Message ID | 20201103152838.1290217-1-lee.jones@linaro.org |
|---|---|
| Headers | show |
| Series | Rid W=1 warnings in SoC | expand |
On Tue 03 Nov 09:28 CST 2020, 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. > For patches 2, 3, 12, 15, 16, 17, 18, 19, 20, 21, 22 (i.e. the soc/qcom patches): Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> and applied towards 5.11 Regards, Bjorn > Lee Jones (25): > soc: bcm: brcmstb: pm: pm-arm: Provide prototype for > brcmstb_pm_s3_finish() > soc: qcom: qcom_aoss: Remove set but unused variable 'tlen' > soc: qcom: qcom_aoss: Add missing description for 'cooling_devs' > soc: fsl: dpio: qbman-portal: Fix a bunch of kernel-doc misdemeanours > soc: rockchip: io-domain: Remove incorrect and incomplete comment > header > soc: ti: knav_qmss_queue: Remove set but unchecked variable 'ret' > soc: ti: knav_qmss_queue: Fix a whole host of function documentation > issues > soc: ti: knav_dma: Fix a kernel function doc formatting issue > soc: ti: pm33xx: Remove set but unused variable 'ret' > soc: ti: wkup_m3_ipc: Document 'm3_ipc' parameter throughout > soc: fsl: qe: qe_common: Fix misnamed function attribute 'addr' > soc: qcom: qcom-geni-se: Fix misnamed function parameter 'rx_rfr' > soc: tegra: fuse: speedo-tegra124: Remove some set but unused > variables > soc: samsung: s3c-pm-check: Fix incorrectly named variable 'val' > soc: qcom: rpmh: Fix possible doc-rot in rpmh_write()'s header > soc: qcom: smem: Fix formatting and missing documentation issues > soc: qcom: smsm: Fix some kernel-doc formatting and naming problems > soc: qcom: wcnss_ctrl: Demote non-conformant struct header and fix > function headers > soc: qcom: smp2p: Remove unused struct attribute provide another > soc: qcom: llcc-qcom: Fix expected kernel-doc formatting > soc: qcom: rpmhpd: Provide some missing struct member descriptions > soc: qcom: kryo-l2-accessors: Fix misnaming of 'val' > soc: ti: k3-ringacc: Provide documentation for 'k3_ring's 'state' > soc: tegra: fuse: speedo-tegra210: Remove a group of set but unused > variables > soc: fsl: qbman: qman: Remove unused variable 'dequeue_wq' > > drivers/soc/bcm/brcmstb/pm/pm-arm.c | 2 + > drivers/soc/fsl/dpio/qbman-portal.c | 18 +++++-- > drivers/soc/fsl/qbman/qman.c | 8 +-- > drivers/soc/fsl/qe/qe_common.c | 2 +- > drivers/soc/qcom/kryo-l2-accessors.c | 2 +- > drivers/soc/qcom/llcc-qcom.c | 2 +- > drivers/soc/qcom/qcom-geni-se.c | 5 +- > drivers/soc/qcom/qcom_aoss.c | 4 +- > drivers/soc/qcom/rpmh.c | 2 +- > drivers/soc/qcom/rpmhpd.c | 3 ++ > drivers/soc/qcom/smem.c | 3 +- > drivers/soc/qcom/smp2p.c | 3 +- > drivers/soc/qcom/smsm.c | 4 +- > drivers/soc/qcom/wcnss_ctrl.c | 8 +-- > drivers/soc/rockchip/io-domain.c | 3 -- > drivers/soc/samsung/s3c-pm-check.c | 2 +- > drivers/soc/tegra/fuse/speedo-tegra124.c | 7 ++- > drivers/soc/tegra/fuse/speedo-tegra210.c | 8 +-- > drivers/soc/ti/k3-ringacc.c | 1 + > drivers/soc/ti/knav_dma.c | 2 +- > drivers/soc/ti/knav_qmss_queue.c | 62 ++++++++++++------------ > drivers/soc/ti/pm33xx.c | 4 +- > drivers/soc/ti/wkup_m3_ipc.c | 8 ++- > 23 files changed, 86 insertions(+), 77 deletions(-) > > Cc: act <dmalek@jlc.net> > Cc: Andy Gross <agross@kernel.org> > Cc: bcm-kernel-feedback-list@broadcom.com > Cc: Ben Dooks <ben@simtec.co.uk> > Cc: Bjorn Andersson <bjorn.andersson@linaro.org> > Cc: Cyril Chemparathy <cyril@ti.com> > Cc: Dan Malek <dan@embeddedalley.com> > Cc: Dave Gerlach <d-gerlach@ti.com> > Cc: Doug Anderson <dianders@chromium.org> > Cc: Florian Fainelli <f.fainelli@gmail.com> > Cc: Heiko Stuebner <heiko@sntech.de> > Cc: Jonathan Hunter <jonathanh@nvidia.com> > Cc: Krzysztof Kozlowski <krzk@kernel.org> > Cc: Liam Girdwood <lgirdwood@gmail.com> > Cc: linux-arm-msm@vger.kernel.org > Cc: linuxppc-dev@lists.ozlabs.org > Cc: linux-rockchip@lists.infradead.org > Cc: linux-samsung-soc@vger.kernel.org > Cc: linux-tegra@vger.kernel.org > Cc: Li Yang <leoyang.li@nxp.com> > Cc: Mark Brown <broonie@kernel.org> > Cc: Qiang Zhao <qiang.zhao@nxp.com> > Cc: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> > Cc: Roy Pledge <Roy.Pledge@nxp.com> > Cc: Sandeep Nair <sandeep_n@ti.com> > Cc: Santosh Shilimkar <ssantosh@kernel.org> > Cc: Scott Wood <scottwood@freescale.com> > Cc: "Software, Inc" <source@mvista.com> > Cc: Thierry Reding <thierry.reding@gmail.com> > Cc: Vitaly Bordug <vbordug@ru.mvista.com> > Cc: YueHaibing <yuehaibing@huawei.com> > > -- > 2.25.1 >
On Tue, 3 Nov 2020 15:28:13 +0000, 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 (25): > soc: bcm: brcmstb: pm: pm-arm: Provide prototype for > brcmstb_pm_s3_finish() > soc: qcom: qcom_aoss: Remove set but unused variable 'tlen' > soc: qcom: qcom_aoss: Add missing description for 'cooling_devs' > soc: fsl: dpio: qbman-portal: Fix a bunch of kernel-doc misdemeanours > soc: rockchip: io-domain: Remove incorrect and incomplete comment > header > soc: ti: knav_qmss_queue: Remove set but unchecked variable 'ret' > soc: ti: knav_qmss_queue: Fix a whole host of function documentation > issues > soc: ti: knav_dma: Fix a kernel function doc formatting issue > soc: ti: pm33xx: Remove set but unused variable 'ret' > soc: ti: wkup_m3_ipc: Document 'm3_ipc' parameter throughout > soc: fsl: qe: qe_common: Fix misnamed function attribute 'addr' > soc: qcom: qcom-geni-se: Fix misnamed function parameter 'rx_rfr' > soc: tegra: fuse: speedo-tegra124: Remove some set but unused > variables > soc: samsung: s3c-pm-check: Fix incorrectly named variable 'val' > soc: qcom: rpmh: Fix possible doc-rot in rpmh_write()'s header > soc: qcom: smem: Fix formatting and missing documentation issues > soc: qcom: smsm: Fix some kernel-doc formatting and naming problems > soc: qcom: wcnss_ctrl: Demote non-conformant struct header and fix > function headers > soc: qcom: smp2p: Remove unused struct attribute provide another > soc: qcom: llcc-qcom: Fix expected kernel-doc formatting > soc: qcom: rpmhpd: Provide some missing struct member descriptions > soc: qcom: kryo-l2-accessors: Fix misnaming of 'val' > soc: ti: k3-ringacc: Provide documentation for 'k3_ring's 'state' > soc: tegra: fuse: speedo-tegra210: Remove a group of set but unused > variables > soc: fsl: qbman: qman: Remove unused variable 'dequeue_wq' > > [...] Applied, thanks! [1/1] soc: rockchip: io-domain: Remove incorrect and incomplete comment header commit: a6a3a24c129d229a0eb26b329ab617e2a04245dd Best regards, -- Heiko Stuebner <heiko@sntech.de>
On Tue, Nov 3, 2020 at 9:29 AM Lee Jones <lee.jones@linaro.org> 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 (25): > soc: fsl: dpio: qbman-portal: Fix a bunch of kernel-doc misdemeanours > soc: fsl: qe: qe_common: Fix misnamed function attribute 'addr' > soc: fsl: qbman: qman: Remove unused variable 'dequeue_wq' The above are applied for next. Thanks. Regards, Leo > > drivers/soc/bcm/brcmstb/pm/pm-arm.c | 2 + > drivers/soc/fsl/dpio/qbman-portal.c | 18 +++++-- > drivers/soc/fsl/qbman/qman.c | 8 +-- > drivers/soc/fsl/qe/qe_common.c | 2 +- > drivers/soc/qcom/kryo-l2-accessors.c | 2 +- > drivers/soc/qcom/llcc-qcom.c | 2 +- > drivers/soc/qcom/qcom-geni-se.c | 5 +- > drivers/soc/qcom/qcom_aoss.c | 4 +- > drivers/soc/qcom/rpmh.c | 2 +- > drivers/soc/qcom/rpmhpd.c | 3 ++ > drivers/soc/qcom/smem.c | 3 +- > drivers/soc/qcom/smp2p.c | 3 +- > drivers/soc/qcom/smsm.c | 4 +- > drivers/soc/qcom/wcnss_ctrl.c | 8 +-- > drivers/soc/rockchip/io-domain.c | 3 -- > drivers/soc/samsung/s3c-pm-check.c | 2 +- > drivers/soc/tegra/fuse/speedo-tegra124.c | 7 ++- > drivers/soc/tegra/fuse/speedo-tegra210.c | 8 +-- > drivers/soc/ti/k3-ringacc.c | 1 + > drivers/soc/ti/knav_dma.c | 2 +- > drivers/soc/ti/knav_qmss_queue.c | 62 ++++++++++++------------ > drivers/soc/ti/pm33xx.c | 4 +- > drivers/soc/ti/wkup_m3_ipc.c | 8 ++- > 23 files changed, 86 insertions(+), 77 deletions(-) > > Cc: act <dmalek@jlc.net> > Cc: Andy Gross <agross@kernel.org> > Cc: bcm-kernel-feedback-list@broadcom.com > Cc: Ben Dooks <ben@simtec.co.uk> > Cc: Bjorn Andersson <bjorn.andersson@linaro.org> > Cc: Cyril Chemparathy <cyril@ti.com> > Cc: Dan Malek <dan@embeddedalley.com> > Cc: Dave Gerlach <d-gerlach@ti.com> > Cc: Doug Anderson <dianders@chromium.org> > Cc: Florian Fainelli <f.fainelli@gmail.com> > Cc: Heiko Stuebner <heiko@sntech.de> > Cc: Jonathan Hunter <jonathanh@nvidia.com> > Cc: Krzysztof Kozlowski <krzk@kernel.org> > Cc: Liam Girdwood <lgirdwood@gmail.com> > Cc: linux-arm-msm@vger.kernel.org > Cc: linuxppc-dev@lists.ozlabs.org > Cc: linux-rockchip@lists.infradead.org > Cc: linux-samsung-soc@vger.kernel.org > Cc: linux-tegra@vger.kernel.org > Cc: Li Yang <leoyang.li@nxp.com> > Cc: Mark Brown <broonie@kernel.org> > Cc: Qiang Zhao <qiang.zhao@nxp.com> > Cc: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> > Cc: Roy Pledge <Roy.Pledge@nxp.com> > Cc: Sandeep Nair <sandeep_n@ti.com> > Cc: Santosh Shilimkar <ssantosh@kernel.org> > Cc: Scott Wood <scottwood@freescale.com> > Cc: "Software, Inc" <source@mvista.com> > Cc: Thierry Reding <thierry.reding@gmail.com> > Cc: Vitaly Bordug <vbordug@ru.mvista.com> > Cc: YueHaibing <yuehaibing@huawei.com> > > -- > 2.25.1 >
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 (25): soc: bcm: brcmstb: pm: pm-arm: Provide prototype for brcmstb_pm_s3_finish() soc: qcom: qcom_aoss: Remove set but unused variable 'tlen' soc: qcom: qcom_aoss: Add missing description for 'cooling_devs' soc: fsl: dpio: qbman-portal: Fix a bunch of kernel-doc misdemeanours soc: rockchip: io-domain: Remove incorrect and incomplete comment header soc: ti: knav_qmss_queue: Remove set but unchecked variable 'ret' soc: ti: knav_qmss_queue: Fix a whole host of function documentation issues soc: ti: knav_dma: Fix a kernel function doc formatting issue soc: ti: pm33xx: Remove set but unused variable 'ret' soc: ti: wkup_m3_ipc: Document 'm3_ipc' parameter throughout soc: fsl: qe: qe_common: Fix misnamed function attribute 'addr' soc: qcom: qcom-geni-se: Fix misnamed function parameter 'rx_rfr' soc: tegra: fuse: speedo-tegra124: Remove some set but unused variables soc: samsung: s3c-pm-check: Fix incorrectly named variable 'val' soc: qcom: rpmh: Fix possible doc-rot in rpmh_write()'s header soc: qcom: smem: Fix formatting and missing documentation issues soc: qcom: smsm: Fix some kernel-doc formatting and naming problems soc: qcom: wcnss_ctrl: Demote non-conformant struct header and fix function headers soc: qcom: smp2p: Remove unused struct attribute provide another soc: qcom: llcc-qcom: Fix expected kernel-doc formatting soc: qcom: rpmhpd: Provide some missing struct member descriptions soc: qcom: kryo-l2-accessors: Fix misnaming of 'val' soc: ti: k3-ringacc: Provide documentation for 'k3_ring's 'state' soc: tegra: fuse: speedo-tegra210: Remove a group of set but unused variables soc: fsl: qbman: qman: Remove unused variable 'dequeue_wq' drivers/soc/bcm/brcmstb/pm/pm-arm.c | 2 + drivers/soc/fsl/dpio/qbman-portal.c | 18 +++++-- drivers/soc/fsl/qbman/qman.c | 8 +-- drivers/soc/fsl/qe/qe_common.c | 2 +- drivers/soc/qcom/kryo-l2-accessors.c | 2 +- drivers/soc/qcom/llcc-qcom.c | 2 +- drivers/soc/qcom/qcom-geni-se.c | 5 +- drivers/soc/qcom/qcom_aoss.c | 4 +- drivers/soc/qcom/rpmh.c | 2 +- drivers/soc/qcom/rpmhpd.c | 3 ++ drivers/soc/qcom/smem.c | 3 +- drivers/soc/qcom/smp2p.c | 3 +- drivers/soc/qcom/smsm.c | 4 +- drivers/soc/qcom/wcnss_ctrl.c | 8 +-- drivers/soc/rockchip/io-domain.c | 3 -- drivers/soc/samsung/s3c-pm-check.c | 2 +- drivers/soc/tegra/fuse/speedo-tegra124.c | 7 ++- drivers/soc/tegra/fuse/speedo-tegra210.c | 8 +-- drivers/soc/ti/k3-ringacc.c | 1 + drivers/soc/ti/knav_dma.c | 2 +- drivers/soc/ti/knav_qmss_queue.c | 62 ++++++++++++------------ drivers/soc/ti/pm33xx.c | 4 +- drivers/soc/ti/wkup_m3_ipc.c | 8 ++- 23 files changed, 86 insertions(+), 77 deletions(-) Cc: act <dmalek@jlc.net> Cc: Andy Gross <agross@kernel.org> Cc: bcm-kernel-feedback-list@broadcom.com Cc: Ben Dooks <ben@simtec.co.uk> Cc: Bjorn Andersson <bjorn.andersson@linaro.org> Cc: Cyril Chemparathy <cyril@ti.com> Cc: Dan Malek <dan@embeddedalley.com> Cc: Dave Gerlach <d-gerlach@ti.com> Cc: Doug Anderson <dianders@chromium.org> Cc: Florian Fainelli <f.fainelli@gmail.com> Cc: Heiko Stuebner <heiko@sntech.de> Cc: Jonathan Hunter <jonathanh@nvidia.com> Cc: Krzysztof Kozlowski <krzk@kernel.org> Cc: Liam Girdwood <lgirdwood@gmail.com> Cc: linux-arm-msm@vger.kernel.org Cc: linuxppc-dev@lists.ozlabs.org Cc: linux-rockchip@lists.infradead.org Cc: linux-samsung-soc@vger.kernel.org Cc: linux-tegra@vger.kernel.org Cc: Li Yang <leoyang.li@nxp.com> Cc: Mark Brown <broonie@kernel.org> Cc: Qiang Zhao <qiang.zhao@nxp.com> Cc: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com> Cc: Roy Pledge <Roy.Pledge@nxp.com> Cc: Sandeep Nair <sandeep_n@ti.com> Cc: Santosh Shilimkar <ssantosh@kernel.org> Cc: Scott Wood <scottwood@freescale.com> Cc: "Software, Inc" <source@mvista.com> Cc: Thierry Reding <thierry.reding@gmail.com> Cc: Vitaly Bordug <vbordug@ru.mvista.com> Cc: YueHaibing <yuehaibing@huawei.com> -- 2.25.1