* [PATCH net-next] driver core: auxiliary bus: Fix documentation of auxiliary_device
@ 2024-07-17 17:29 Saeed Mahameed
2024-07-18 0:02 ` Jakub Kicinski
2024-07-18 5:30 ` patchwork-bot+netdevbpf
0 siblings, 2 replies; 4+ messages in thread
From: Saeed Mahameed @ 2024-07-17 17:29 UTC (permalink / raw)
To: David S. Miller, Jakub Kicinski, Paolo Abeni, Eric Dumazet
Cc: Saeed Mahameed, netdev, Tariq Toukan, Gal Pressman,
Leon Romanovsky, Shay Drory, Stephen Rothwell
From: Shay Drory <shayd@nvidia.com>
Fix the documentation of the below field of struct auxiliary_device
include/linux/auxiliary_bus.h:150: warning: Function parameter or struct member 'sysfs' not described in 'auxiliary_device'
include/linux/auxiliary_bus.h:150: warning: Excess struct member 'irqs' description in 'auxiliary_device'
include/linux/auxiliary_bus.h:150: warning: Excess struct member 'lock' description in 'auxiliary_device'
include/linux/auxiliary_bus.h:150: warning: Excess struct member 'irq_dir_exists' description in 'auxiliary_device'
Fixes: a808878308a8 ("driver core: auxiliary bus: show auxiliary device IRQs")
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Shay Drory <shayd@nvidia.com>
Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
---
include/linux/auxiliary_bus.h | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/include/linux/auxiliary_bus.h b/include/linux/auxiliary_bus.h
index 3ba4487c9cd9..1539bbd263d2 100644
--- a/include/linux/auxiliary_bus.h
+++ b/include/linux/auxiliary_bus.h
@@ -58,9 +58,10 @@
* in
* @name: Match name found by the auxiliary device driver,
* @id: unique identitier if multiple devices of the same name are exported,
- * @irqs: irqs xarray contains irq indices which are used by the device,
- * @lock: Synchronize irq sysfs creation,
- * @irq_dir_exists: whether "irqs" directory exists,
+ * @sysfs: embedded struct which hold all sysfs related fields,
+ * @sysfs.irqs: irqs xarray contains irq indices which are used by the device,
+ * @sysfs.lock: Synchronize irq sysfs creation,
+ * @sysfs.irq_dir_exists: whether "irqs" directory exists,
*
* An auxiliary_device represents a part of its parent device's functionality.
* It is given a name that, combined with the registering drivers
--
2.45.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] driver core: auxiliary bus: Fix documentation of auxiliary_device
2024-07-17 17:29 [PATCH net-next] driver core: auxiliary bus: Fix documentation of auxiliary_device Saeed Mahameed
@ 2024-07-18 0:02 ` Jakub Kicinski
2024-07-18 4:55 ` Greg Kroah-Hartman
2024-07-18 5:30 ` patchwork-bot+netdevbpf
1 sibling, 1 reply; 4+ messages in thread
From: Jakub Kicinski @ 2024-07-18 0:02 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Saeed Mahameed, David S. Miller, Paolo Abeni, Eric Dumazet,
Saeed Mahameed, netdev, Tariq Toukan, Gal Pressman,
Leon Romanovsky, Shay Drory, Stephen Rothwell
On Wed, 17 Jul 2024 10:29:16 -0700 Saeed Mahameed wrote:
> From: Shay Drory <shayd@nvidia.com>
>
> Fix the documentation of the below field of struct auxiliary_device
>
> include/linux/auxiliary_bus.h:150: warning: Function parameter or struct member 'sysfs' not described in 'auxiliary_device'
> include/linux/auxiliary_bus.h:150: warning: Excess struct member 'irqs' description in 'auxiliary_device'
> include/linux/auxiliary_bus.h:150: warning: Excess struct member 'lock' description in 'auxiliary_device'
> include/linux/auxiliary_bus.h:150: warning: Excess struct member 'irq_dir_exists' description in 'auxiliary_device'
>
> Fixes: a808878308a8 ("driver core: auxiliary bus: show auxiliary device IRQs")
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: Shay Drory <shayd@nvidia.com>
> Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
Thanks!
Greg, please let us know if you'd like to handle this yourself.
Otherwise we'll ship it to Linus tomorrow or Friday.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] driver core: auxiliary bus: Fix documentation of auxiliary_device
2024-07-18 0:02 ` Jakub Kicinski
@ 2024-07-18 4:55 ` Greg Kroah-Hartman
0 siblings, 0 replies; 4+ messages in thread
From: Greg Kroah-Hartman @ 2024-07-18 4:55 UTC (permalink / raw)
To: Jakub Kicinski
Cc: Saeed Mahameed, David S. Miller, Paolo Abeni, Eric Dumazet,
Saeed Mahameed, netdev, Tariq Toukan, Gal Pressman,
Leon Romanovsky, Shay Drory, Stephen Rothwell
On Wed, Jul 17, 2024 at 05:02:54PM -0700, Jakub Kicinski wrote:
> On Wed, 17 Jul 2024 10:29:16 -0700 Saeed Mahameed wrote:
> > From: Shay Drory <shayd@nvidia.com>
> >
> > Fix the documentation of the below field of struct auxiliary_device
> >
> > include/linux/auxiliary_bus.h:150: warning: Function parameter or struct member 'sysfs' not described in 'auxiliary_device'
> > include/linux/auxiliary_bus.h:150: warning: Excess struct member 'irqs' description in 'auxiliary_device'
> > include/linux/auxiliary_bus.h:150: warning: Excess struct member 'lock' description in 'auxiliary_device'
> > include/linux/auxiliary_bus.h:150: warning: Excess struct member 'irq_dir_exists' description in 'auxiliary_device'
> >
> > Fixes: a808878308a8 ("driver core: auxiliary bus: show auxiliary device IRQs")
> > Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> > Signed-off-by: Shay Drory <shayd@nvidia.com>
> > Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
>
> Thanks!
>
> Greg, please let us know if you'd like to handle this yourself.
I can't, it came in through your tree :)
> Otherwise we'll ship it to Linus tomorrow or Friday.
Please do so.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] driver core: auxiliary bus: Fix documentation of auxiliary_device
2024-07-17 17:29 [PATCH net-next] driver core: auxiliary bus: Fix documentation of auxiliary_device Saeed Mahameed
2024-07-18 0:02 ` Jakub Kicinski
@ 2024-07-18 5:30 ` patchwork-bot+netdevbpf
1 sibling, 0 replies; 4+ messages in thread
From: patchwork-bot+netdevbpf @ 2024-07-18 5:30 UTC (permalink / raw)
To: Saeed Mahameed
Cc: davem, kuba, pabeni, edumazet, saeedm, netdev, tariqt, gal,
leonro, shayd, sfr
Hello:
This patch was applied to netdev/net.git (main)
by Jakub Kicinski <kuba@kernel.org>:
On Wed, 17 Jul 2024 10:29:16 -0700 you wrote:
> From: Shay Drory <shayd@nvidia.com>
>
> Fix the documentation of the below field of struct auxiliary_device
>
> include/linux/auxiliary_bus.h:150: warning: Function parameter or struct member 'sysfs' not described in 'auxiliary_device'
> include/linux/auxiliary_bus.h:150: warning: Excess struct member 'irqs' description in 'auxiliary_device'
> include/linux/auxiliary_bus.h:150: warning: Excess struct member 'lock' description in 'auxiliary_device'
> include/linux/auxiliary_bus.h:150: warning: Excess struct member 'irq_dir_exists' description in 'auxiliary_device'
>
> [...]
Here is the summary with links:
- [net-next] driver core: auxiliary bus: Fix documentation of auxiliary_device
https://git.kernel.org/netdev/net/c/c14112a5574f
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-07-18 5:30 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-17 17:29 [PATCH net-next] driver core: auxiliary bus: Fix documentation of auxiliary_device Saeed Mahameed
2024-07-18 0:02 ` Jakub Kicinski
2024-07-18 4:55 ` Greg Kroah-Hartman
2024-07-18 5:30 ` patchwork-bot+netdevbpf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).