From nobody Wed Feb 11 18:28:00 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id CCF47CA0EC1 for ; Mon, 11 Sep 2023 21:19:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1344005AbjIKVNE (ORCPT ); Mon, 11 Sep 2023 17:13:04 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41558 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S243825AbjIKRwp (ORCPT ); Mon, 11 Sep 2023 13:52:45 -0400 Received: from mail-qv1-xf2d.google.com (mail-qv1-xf2d.google.com [IPv6:2607:f8b0:4864:20::f2d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4C999DF for ; Mon, 11 Sep 2023 10:52:40 -0700 (PDT) Received: by mail-qv1-xf2d.google.com with SMTP id 6a1803df08f44-655d1cf74faso19152156d6.1 for ; Mon, 11 Sep 2023 10:52:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1694454759; x=1695059559; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=HSGb/bzAvx1o0jtUr8Ii5AAazodLa4hyLaXmw0np69Y=; b=DhEAam3BbR9XCuHzV3Zk9FzTUJlqSqrwI/VgC26QJ1qEZ+ltAQFtmJZVhlsux6Amb+ 3irAQup6LLL18AKJh/CoevutGZU/gFtiDZ6JDlusAuLEZvB5qZxQdpig9O4gDOJB7NSb p6xiYUzkRdd0OLaaRNuSFoe2VZM5kwx7/U483MFtvVFhbDGwuf7zJA/OpbW01QlQpmYf nZkvn9jSEhO5rjM6dU/KCd9driHN4I8/vLB1X2PZMy+LnIPEYtQgXWj2ozzg6qrWC1/H /kBqE2nM7RGXg3k6jz6JnoeYU1YyA4VAxslMASAwJCRPVGCHLNICHfZCDENFxkPJISXH TUuA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1694454759; x=1695059559; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=HSGb/bzAvx1o0jtUr8Ii5AAazodLa4hyLaXmw0np69Y=; b=uPoDa+/IYBDLZiqlnLwXDii3eIN9qI7STSbCzioac/kmpaQhdLKeRHjEnEf7QAMGRn kXprSDGlkp+S0Yfd1JPnnl8NPTKs3tDPKL8mifxWHWk0T6vOFcyi1rTcDOxzyLMi7NZy RpoPdvk9b1bbi/4Bud9VrWW58OVSD8g10+XrpV0A7S9Zv2cKY3sfgxhazMN1/9EjD72i Bz/1yxydMYpm/+XKT1gPiV632ZoSyCUaTJ3HgObrlC1Ocojy6FkuXZlTMdyJFNILXivi ucLqax1NPYqoJW9y0mR6b4xXc0DbhnmPGp6cJSSYgqUbWiM+sKuG1x0K14DGlF+t34DN p7Ew== X-Gm-Message-State: AOJu0YyQ47pmX+FBCi0frF8lY2WUttNd0Eogm1uLO0kwWUPfdxBUdgiM I5f+YtVb+v8LGaPN4bu0948bSg== X-Google-Smtp-Source: AGHT+IG65tLFaGRoRMpgzKk0ogF2TeonEdlTNo/aDVlOmsA1gbxv+nd8ZS7QwN4rpf1idWUNGwTTbA== X-Received: by 2002:a0c:d610:0:b0:649:bf3:6dbe with SMTP id c16-20020a0cd610000000b006490bf36dbemr9680030qvj.62.1694454759379; Mon, 11 Sep 2023 10:52:39 -0700 (PDT) Received: from maple.home ([142.114.20.193]) by smtp.gmail.com with ESMTPSA id s17-20020a0ca611000000b006431027ac44sm3118898qva.83.2023.09.11.10.52.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 11 Sep 2023 10:52:38 -0700 (PDT) From: Ralph Siemsen To: Geert Uytterhoeven , Miquel Raynal , Michael Turquette , Stephen Boyd Cc: linux-renesas-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Ralph Siemsen , kernel test robot Subject: [PATCH] clk: renesas: r9a06g032: fix kerneldoc warning Date: Mon, 11 Sep 2023 13:52:15 -0400 Message-Id: <20230911175215.263009-1-ralph.siemsen@linaro.org> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" This fixes the following W=3D1 warning during build: > drivers/clk/renesas/r9a06g032-clocks.c:119: warning: Function parameter o= r member 'dual' not described in 'r9a06g032_clkdesc' Added documentation for member 'dual'. Also added names for the other structures in the same union, with documentation. Adjusted names of members within the 'div' structure to avoid duplication. Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202309101314.kTRoxND5-lkp@int= el.com/ Signed-off-by: Ralph Siemsen --- Second post, as the Subject: line accidentally got messed up previously. drivers/clk/renesas/r9a06g032-clocks.c | 64 ++++++++++++++------------ 1 file changed, 34 insertions(+), 30 deletions(-) diff --git a/drivers/clk/renesas/r9a06g032-clocks.c b/drivers/clk/renesas/r= 9a06g032-clocks.c index 55db63c7041a..61296c81f9b5 100644 --- a/drivers/clk/renesas/r9a06g032-clocks.c +++ b/drivers/clk/renesas/r9a06g032-clocks.c @@ -102,19 +102,22 @@ enum gate_type { * @source: the ID+1 of the parent clock element. * Root clock uses ID of ~0 (PARENT_ID); * @gate: clock enable/disable - * @div_min: smallest permitted clock divider - * @div_max: largest permitted clock divider - * @reg: clock divider register offset, in 32-bit words - * @div_table: optional list of fixed clock divider values; + * @div: substructure for clock divider + * @div.min: smallest permitted clock divider + * @div.max: largest permitted clock divider + * @div.reg: clock divider register offset, in 32-bit words + * @div.table: optional list of fixed clock divider values; * must be in ascending order, zero for unused - * @div: divisor for fixed-factor clock - * @mul: multiplier for fixed-factor clock - * @group: UART group, 0=3DUART0/1/2, 1=3DUART3/4/5/6/7 - * @sel: select either g1/r1 or g2/r2 as clock source - * @g1: 1st source gate (clock enable/disable) - * @r1: 1st source reset (module reset) - * @g2: 2nd source gate (clock enable/disable) - * @r2: 2nd source reset (module reset) + * @ffc: substructure for fixed-factor clocks + * @ffc.div: divisor for fixed-factor clock + * @ffc.mul: multiplier for fixed-factor clock + * @dual: substructure for dual clock gates + * @dual.group: UART group, 0=3DUART0/1/2, 1=3DUART3/4/5/6/7 + * @dual.sel: select either g1/r1 or g2/r2 as clock source + * @dual.g1: 1st source gate (clock enable/disable) + * @dual.r1: 1st source reset (module reset) + * @dual.g2: 2nd source gate (clock enable/disable) + * @dual.r2: 2nd source reset (module reset) * * Describes a single element in the clock tree hierarchy. * As there are quite a large number of clock elements, this @@ -131,13 +134,13 @@ struct r9a06g032_clkdesc { struct r9a06g032_gate gate; /* type =3D K_DIV */ struct { - unsigned int div_min:10, div_max:10, reg:10; - u16 div_table[4]; - }; + unsigned int min:10, max:10, reg:10; + u16 table[4]; + } div; /* type =3D K_FFC */ struct { u16 div, mul; - }; + } ffc; /* type =3D K_DUALGATE */ struct { uint16_t group:1; @@ -178,26 +181,26 @@ struct r9a06g032_clkdesc { .type =3D K_FFC, \ .index =3D R9A06G032_##_idx, \ .name =3D _n, \ - .div =3D _div, \ - .mul =3D _mul \ + .ffc.div =3D _div, \ + .ffc.mul =3D _mul \ } #define D_FFC(_idx, _n, _src, _div) { \ .type =3D K_FFC, \ .index =3D R9A06G032_##_idx, \ .source =3D 1 + R9A06G032_##_src, \ .name =3D _n, \ - .div =3D _div, \ - .mul =3D 1 \ + .ffc.div =3D _div, \ + .ffc.mul =3D 1 \ } #define D_DIV(_idx, _n, _src, _reg, _min, _max, ...) { \ .type =3D K_DIV, \ .index =3D R9A06G032_##_idx, \ .source =3D 1 + R9A06G032_##_src, \ .name =3D _n, \ - .reg =3D _reg, \ - .div_min =3D _min, \ - .div_max =3D _max, \ - .div_table =3D { __VA_ARGS__ } \ + .div.reg =3D _reg, \ + .div.min =3D _min, \ + .div.max =3D _max, \ + .div.table =3D { __VA_ARGS__ } \ } #define D_UGATE(_idx, _n, _src, _g, _g1, _r1, _g2, _r2) { \ .type =3D K_DUALGATE, \ @@ -1063,14 +1066,14 @@ r9a06g032_register_div(struct r9a06g032_priv *clock= s, =20 div->clocks =3D clocks; div->index =3D desc->index; - div->reg =3D desc->reg; + div->reg =3D desc->div.reg; div->hw.init =3D &init; - div->min =3D desc->div_min; - div->max =3D desc->div_max; + div->min =3D desc->div.min; + div->max =3D desc->div.max; /* populate (optional) divider table fixed values */ for (i =3D 0; i < ARRAY_SIZE(div->table) && - i < ARRAY_SIZE(desc->div_table) && desc->div_table[i]; i++) { - div->table[div->table_size++] =3D desc->div_table[i]; + i < ARRAY_SIZE(desc->div.table) && desc->div.table[i]; i++) { + div->table[div->table_size++] =3D desc->div.table[i]; } =20 clk =3D clk_register(NULL, &div->hw); @@ -1333,7 +1336,8 @@ static int __init r9a06g032_clocks_probe(struct platf= orm_device *pdev) case K_FFC: clk =3D clk_register_fixed_factor(NULL, d->name, parent_name, 0, - d->mul, d->div); + d->ffc.mul, + d->ffc.div); break; case K_GATE: clk =3D r9a06g032_register_gate(clocks, parent_name, d); --=20 2.25.1