From patchwork Tue Aug 10 00:54:31 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Randy Dunlap X-Patchwork-Id: 12427659 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=-17.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_GIT 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 759B5C4338F for ; Tue, 10 Aug 2021 00:57:14 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 2C21E6101E for ; Tue, 10 Aug 2021 00:57:14 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 2C21E6101E Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=9MVJKag776z3cKL0HtOwjVBkgY43Dfg7fvF6es9MN18=; b=2VePDoVioBiOZd b0I0Wsi4l2nyQkEA9rdoEqb1XWrP72q2/RIgMquVsXT4jXOJlJU8ARFpkfVxLvBlChlOwTRb0SUwb liFPsZIRVqcL80HBxJPLPlltRaVBgoA+KKrOmgfhxBAhX/UE0u8VoPmc2ZqKt4SvMRuTvlJ4+XXwR j81/XavqugdOkg2rLTUPeUfVrIPHlGLS+09J/2bHzQ3je4eAiq6HshfCzK4SuU6qFBNu0gBvAHMrM 1ivxwKNIP7as2ymKrRPLvnGgK4RRBa58QIXub2czpsL5zhFrdBFWWI5folYjBMdQjKQaw0WjHCQ8w fkgoBgCT7YPD5cPhPLeQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mDG2M-002Bda-V0; Tue, 10 Aug 2021 00:54:35 +0000 Received: from [2601:1c0:6280:3f0::aa0b] (helo=bombadil.infradead.org) by bombadil.infradead.org with esmtpsa (Exim 4.94.2 #2 (Red Hat Linux)) id 1mDG2K-002BdO-5f; Tue, 10 Aug 2021 00:54:32 +0000 From: Randy Dunlap To: linux-kernel@vger.kernel.org Cc: Randy Dunlap , kernel test robot , Claudiu Beznea , Michael Turquette , Stephen Boyd , Eugen Hristev , linux-clk@vger.kernel.org, linux-doc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Jonathan Corbet Subject: [PATCH] clk/at91: remove kernel-doc warnings in sama7g5.c Date: Mon, 9 Aug 2021 17:54:31 -0700 Message-Id: <20210810005431.22007-1-rdunlap@infradead.org> X-Mailer: git-send-email 2.31.1 MIME-Version: 1.0 X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Fix multiple kernel-doc warnings in sama7g5.c. There are several enums and structs that were not identified as such. There are also several anonymous structs (that scripts/kernel-doc has problems with), so add struct names to them. Fixes the following warnings: drivers/clk/at91/sama7g5.c:39: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst * PLL clocks identifiers drivers/clk/at91/sama7g5.c:60: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst * PLL type identifiers drivers/clk/at91/sama7g5.c:122: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst * PLL clocks description drivers/clk/at91/sama7g5.c:289: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst * Master clock (MCK[1..4]) description drivers/clk/at91/sama7g5.c:341: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst * System clock description drivers/clk/at91/sama7g5.c:365: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst * Peripheral clock description drivers/clk/at91/sama7g5.c:453: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst * Generic clock description drivers/clk/at91/sama7g5.c:339: warning: Function parameter or member 'ep_chg_id' not described in 'master_clock' Some kernel-doc warnings are still printed, but they are all due to problems in the kernel-doc script (if anyone is interested in playing with some Perl :). These warnings are: sama7g5.c:287: warning: Function parameter or member '{' not described in 'sama7g5_plls' sama7g5.c:287: warning: Function parameter or member '' not described in 'sama7g5_plls' sama7g5.c:287: warning: Function parameter or member '}' not described in 'sama7g5_plls' sama7g5.c:339: warning: Function parameter or member 'sama7g5_mckx' not described in 'master_clock' sama7g5.c:339: warning: Function parameter or member '}' not described in 'master_clock' sama7g5.c:360: warning: Function parameter or member 'sama7g5_systemck' not described in 'system_clock' sama7g5.c:451: warning: Function parameter or member 'sama7g5_periphck' not described in 'periph_clock' sama7g5.c:451: warning: Function parameter or member '' not described in 'periph_clock' sama7g5.c:451: warning: Function parameter or member '}' not described in 'periph_clock' sama7g5.c:841: warning: Function parameter or member 'sama7g5_gck' not described in 'generic_clock' sama7g5.c:841: warning: Function parameter or member '}' not described in 'generic_clock' Signed-off-by: Randy Dunlap Reported-by: kernel test robot Cc: Claudiu Beznea Cc: Michael Turquette Cc: Stephen Boyd Cc: Eugen Hristev Cc: linux-clk@vger.kernel.org Cc: linux-doc@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org Cc: Jonathan Corbet --- drivers/clk/at91/sama7g5.c | 27 ++++++++++++++------------- 1 file changed, 14 insertions(+), 13 deletions(-) --- linux-next-20210809.orig/drivers/clk/at91/sama7g5.c +++ linux-next-20210809/drivers/clk/at91/sama7g5.c @@ -36,7 +36,7 @@ static DEFINE_SPINLOCK(pmc_mck0_lock); static DEFINE_SPINLOCK(pmc_mckX_lock); /** - * PLL clocks identifiers + * enum pll_ids - PLL clocks identifiers * @PLL_ID_CPU: CPU PLL identifier * @PLL_ID_SYS: System PLL identifier * @PLL_ID_DDR: DDR PLL identifier @@ -44,6 +44,7 @@ static DEFINE_SPINLOCK(pmc_mckX_lock); * @PLL_ID_BAUD: Baud PLL identifier * @PLL_ID_AUDIO: Audio PLL identifier * @PLL_ID_ETH: Ethernet PLL identifier + * @PLL_ID_MAX: Number of PLL_ID values (1 more than the last valid PLL_ID) */ enum pll_ids { PLL_ID_CPU, @@ -57,7 +58,7 @@ enum pll_ids { }; /** - * PLL type identifiers + * enum pll_type - PLL type identifiers * @PLL_TYPE_FRAC: fractional PLL identifier * @PLL_TYPE_DIV: divider PLL identifier */ @@ -119,7 +120,7 @@ static const struct clk_pll_characterist }; /** - * PLL clocks description + * struct sama7g5_plls - PLL clocks description * @n: clock name * @p: clock parent * @l: clock layout @@ -128,7 +129,7 @@ static const struct clk_pll_characterist * @f: clock flags * @eid: export index in sama7g5->chws[] array */ -static const struct { +static const struct sama7g5_plls { const char *n; const char *p; const struct clk_pll_layout *l; @@ -286,17 +287,17 @@ static const struct { }; /** - * Master clock (MCK[1..4]) description + * struct master_clock - Master clock (MCK[1..4]) description * @n: clock name * @ep: extra parents names array - * @ep_chg_chg_id: index in parents array that specifies the changeable + * @ep_chg_id: index in parents array that specifies the changeable * parent * @ep_count: extra parents count * @ep_mux_table: mux table for extra parents * @id: clock id * @c: true if clock is critical and cannot be disabled */ -static const struct { +static const struct master_clock { const char *n; const char *ep[4]; int ep_chg_id; @@ -338,12 +339,12 @@ static const struct { }; /** - * System clock description + * struct system_clock - System clock description * @n: clock name * @p: clock parent name * @id: clock id */ -static const struct { +static const struct system_clock { const char *n; const char *p; u8 id; @@ -362,14 +363,14 @@ static const struct { static u32 sama7g5_prog_mux_table[] = { 0, 1, 2, 5, 6, 7, 8, 9, 10, }; /** - * Peripheral clock description + * struct periph_clock - Peripheral clock description * @n: clock name * @p: clock parent name * @r: clock range values * @id: clock id * @chgp: index in parent array of the changeable parent */ -static const struct { +static const struct periph_clock { const char *n; const char *p; struct clk_range r; @@ -450,7 +451,7 @@ static const struct { }; /** - * Generic clock description + * struct generic_clock - Generic clock description * @n: clock name * @pp: PLL parents * @pp_mux_table: PLL parents mux table @@ -459,7 +460,7 @@ static const struct { * @pp_count: PLL parents count * @id: clock id */ -static const struct { +static const struct generic_clock { const char *n; const char *pp[8]; const char pp_mux_table[8];