[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZmDEVoC9NUh7Gg7k@surfacebook.localdomain>
Date: Wed, 5 Jun 2024 23:02:30 +0300
From: Andy Shevchenko <andy.shevchenko@...il.com>
To: Herve Codina <herve.codina@...tlin.com>
Cc: Simon Horman <horms@...nel.org>,
Sai Krishna Gajula <saikrishnag@...vell.com>,
Thomas Gleixner <tglx@...utronix.de>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
Lee Jones <lee@...nel.org>, Arnd Bergmann <arnd@...db.de>,
Horatiu Vultur <horatiu.vultur@...rochip.com>,
UNGLinuxDriver@...rochip.com, Andrew Lunn <andrew@...n.ch>,
Heiner Kallweit <hkallweit1@...il.com>,
Russell King <linux@...linux.org.uk>,
Saravana Kannan <saravanak@...gle.com>,
Bjorn Helgaas <bhelgaas@...gle.com>,
Philipp Zabel <p.zabel@...gutronix.de>,
Lars Povlsen <lars.povlsen@...rochip.com>,
Steen Hegelund <Steen.Hegelund@...rochip.com>,
Daniel Machon <daniel.machon@...rochip.com>,
Alexandre Belloni <alexandre.belloni@...tlin.com>,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
netdev@...r.kernel.org, linux-pci@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
Allan Nielsen <allan.nielsen@...rochip.com>,
Luca Ceresoli <luca.ceresoli@...tlin.com>,
Thomas Petazzoni <thomas.petazzoni@...tlin.com>
Subject: Re: [PATCH v2 09/19] irqdomain: Add missing parameter descriptions
in docs
Mon, May 27, 2024 at 06:14:36PM +0200, Herve Codina kirjoitti:
> During compilation, several warning of the following form were raised:
> Function parameter or struct member 'x' not described in 'yyy'
>
> Add the missing function parameter descriptions.
...
> /**
> * irq_domain_translate_onecell() - Generic translate for direct one cell
> * bindings
> + * @d: Interrupt domain involved in the translation
> + * @fwspec: The firmware interrupt specifier to translate
> + * @out_hwirq: Pointer to storage for the hardware interrupt number
> + * @out_type: Pointer to storage for the interrupt type
(kernel-doc perhaps will complain on something missing here)
> */
> int irq_domain_translate_onecell(struct irq_domain *d,
You can go further and run
scripts/kernel-doc -v -none -Wall ...
against this file and fix more issues, like I believe in the above excerpt.
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists