netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).