From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.3 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 063AEC433DB for ; Mon, 8 Feb 2021 06:46:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B4F6864DA8 for ; Mon, 8 Feb 2021 06:46:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229917AbhBHGqo (ORCPT ); Mon, 8 Feb 2021 01:46:44 -0500 Received: from mail.kernel.org ([198.145.29.99]:56792 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229908AbhBHGqn (ORCPT ); Mon, 8 Feb 2021 01:46:43 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id 3E47364E59; Mon, 8 Feb 2021 06:45:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1612766761; bh=laV8svMkkIZgkEjMImPDqav9pFAGfCHDdtEUcbdDnvY=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=TGskujnEd6wHa7WkmlrA/txwqRTcJGBrt+G8G29pmkA5kZJTpP9p0uA0OBvZIqSvD JxUKjVIlJL7BN44ZogKx96Zw8qus91VU68qdycGrSWnEl44/V2NTQUC1OLOTCdF4Gu FGXrZfB1UlGCm0us6h06+bTEmrWyNdKdO1cQE50Xn9aYIA9cVXbE7c/MBRqdiuZ1jz yRAPPdGXm7pgspO8bff3EiWF+togadvuTOEFB2flyG4QXvLieVjLI1n9IWbF4cIrlT j4bPAIhCO/OgcnQj5qsi4LQSz7dkKfKSzdR46yYTRQHb0tQHuVB44086JDEOFAZDLc NgM8lbk2wAojg== Subject: Re: [PATCH 00/21] [Set 2] Rid W=1 warnings from Clock To: Lee Jones Cc: linux-kernel@vger.kernel.org, Ahmad Fatoum , Andy Gross , Avi Fishman , Benjamin Fair , Bjorn Andersson , Boris BREZILLON , Chen-Yu Tsai , =?UTF-8?Q?Emilio_L=c3=b3pez?= , Fabio Estevam , Geert Uytterhoeven , Jan Kotas , Jernej Skrabec , Jonathan Hunter , linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, linux-omap@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-tegra@vger.kernel.org, Loc Ho , Maxime Ripard , Michael Turquette , Michal Simek , Nancy Yuen , Nuvoton Technologies , NXP Linux Team , openbmc@lists.ozlabs.org, Patrick Venture , Pengutronix Kernel Team , Peter De Schrijver , Philipp Zabel , Prashant Gaikwad , Rajan Vaja , Rajeev Kumar , Richard Woodruff , Russell King , Sascha Hauer , Shawn Guo , Shiraz Hashim , =?UTF-8?Q?S=c3=b6ren_Brinkmann?= , Stephen Boyd , Tali Perry , Thierry Reding , Tomer Maimon , Viresh Kumar References: <20210126124540.3320214-1-lee.jones@linaro.org> From: Tero Kristo Message-ID: Date: Mon, 8 Feb 2021 08:45:50 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20210126124540.3320214-1-lee.jones@linaro.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org On 26/01/2021 14:45, 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. > > This is the last set. Clock is clean after this. > > Lee Jones (21): > clk: zynq: pll: Fix kernel-doc formatting in 'clk_register_zynq_pll's > header > clk: ti: clkt_dpll: Fix some kernel-doc misdemeanours > clk: ti: dpll3xxx: Fix some kernel-doc headers and promote other > worthy ones > clk: qcom: clk-regmap: Provide missing description for > 'devm_clk_register_regmap()'s dev param > clk: sunxi: clk-sun9i-core: Demote non-conformant kernel-doc headers > clk: sunxi: clk-usb: Demote obvious kernel-doc abuse > clk: tegra: clk-tegra30: Remove unused variable 'reg' > clk: clkdev: Ignore suggestion to use gnu_printf() as it's not > appropriate here > clk: tegra: cvb: Provide missing description for > 'tegra_cvb_add_opp_table()'s align param > clk: ti: dpll44xx: Fix some potential doc-rot > clk: renesas: renesas-cpg-mssr: Fix formatting issues for > 'smstpcr_saved's documentation > clk: sunxi: clk-sun6i-ar100: Demote non-conformant kernel-doc header > clk: qcom: gcc-ipq4019: Remove unused variable 'ret' > clk: clk-fixed-mmio: Demote obvious kernel-doc abuse > clk: clk-npcm7xx: Remove unused static const tables 'npcm7xx_gates' > and 'npcm7xx_divs_fx' > clk: qcom: mmcc-msm8974: Remove unused static const tables > 'mmcc_xo_mmpll0_1_2_gpll0{map}' > clk: clk-xgene: Add description for 'mask' and fix formatting for > 'flags' > clk: qcom: clk-rpm: Remove a bunch of superfluous code > clk: spear: Move prototype to accessible header > clk: imx: Move 'imx6sl_set_wait_clk()'s prototype out to accessible > header > clk: zynqmp: divider: Add missing description for 'max_div' > > arch/arm/mach-imx/common.h | 1 - > arch/arm/mach-imx/cpuidle-imx6sl.c | 1 + > arch/arm/mach-imx/pm-imx6.c | 1 + > arch/arm/mach-spear/generic.h | 12 --- > arch/arm/mach-spear/spear13xx.c | 1 + > drivers/clk/clk-fixed-mmio.c | 2 +- > drivers/clk/clk-npcm7xx.c | 108 ------------------------- > drivers/clk/clk-xgene.c | 5 +- > drivers/clk/clkdev.c | 7 ++ > drivers/clk/imx/clk-imx6sl.c | 1 + > drivers/clk/qcom/clk-regmap.c | 1 + > drivers/clk/qcom/clk-rpm.c | 63 --------------- > drivers/clk/qcom/gcc-ipq4019.c | 7 +- > drivers/clk/qcom/mmcc-msm8974.c | 16 ---- > drivers/clk/renesas/renesas-cpg-mssr.c | 4 +- > drivers/clk/spear/spear1310_clock.c | 1 + > drivers/clk/spear/spear1340_clock.c | 1 + > drivers/clk/sunxi/clk-sun6i-ar100.c | 2 +- > drivers/clk/sunxi/clk-sun9i-core.c | 8 +- > drivers/clk/sunxi/clk-usb.c | 2 +- > drivers/clk/tegra/clk-tegra30.c | 5 +- > drivers/clk/tegra/cvb.c | 1 + > drivers/clk/ti/clkt_dpll.c | 3 +- > drivers/clk/ti/dpll3xxx.c | 20 ++--- > drivers/clk/ti/dpll44xx.c | 6 +- For the TI portions: Reviewed-by: Tero Kristo > drivers/clk/zynq/pll.c | 12 +-- > drivers/clk/zynqmp/divider.c | 1 + > include/linux/clk/imx.h | 15 ++++ > include/linux/clk/spear.h | 23 ++++++ > 29 files changed, 92 insertions(+), 238 deletions(-) > create mode 100644 include/linux/clk/imx.h > create mode 100644 include/linux/clk/spear.h > > Cc: Ahmad Fatoum > Cc: Andy Gross > Cc: Avi Fishman > Cc: Benjamin Fair > Cc: Bjorn Andersson > Cc: Boris BREZILLON > Cc: Chen-Yu Tsai > Cc: "Emilio López" > Cc: Fabio Estevam > Cc: Geert Uytterhoeven > Cc: Jan Kotas > Cc: Jernej Skrabec > Cc: Jonathan Hunter > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-arm-msm@vger.kernel.org > Cc: linux-clk@vger.kernel.org > Cc: linux-omap@vger.kernel.org > Cc: linux-renesas-soc@vger.kernel.org > Cc: linux-tegra@vger.kernel.org > Cc: Loc Ho > Cc: Maxime Ripard > Cc: Michael Turquette > Cc: Michal Simek > Cc: Nancy Yuen > Cc: Nuvoton Technologies > Cc: NXP Linux Team > Cc: openbmc@lists.ozlabs.org > Cc: Patrick Venture > Cc: Pengutronix Kernel Team > Cc: Peter De Schrijver > Cc: Philipp Zabel > Cc: Prashant Gaikwad > Cc: Rajan Vaja > Cc: Rajeev Kumar > Cc: Richard Woodruff > Cc: Russell King > Cc: Sascha Hauer > Cc: Shawn Guo > Cc: Shiraz Hashim > Cc: "Sören Brinkmann" > Cc: Stephen Boyd > Cc: Tali Perry > Cc: Tero Kristo > Cc: Thierry Reding > Cc: Tomer Maimon > Cc: Viresh Kumar >