netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RESEND PATCH net] net: phy: aquantia: add missing include guards
@ 2024-06-27 10:58 Bartosz Golaszewski
  2024-06-27 21:59 ` Jakub Kicinski
  0 siblings, 1 reply; 4+ messages in thread
From: Bartosz Golaszewski @ 2024-06-27 10:58 UTC (permalink / raw)
  To: Andrew Lunn, Heiner Kallweit, Russell King, David S . Miller,
	Eric Dumazet, Jakub Kicinski, Paolo Abeni
  Cc: netdev, linux-kernel, Bartosz Golaszewski

From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

The header is missing the include guards so add them.

Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Fixes: fb470f70fea7 ("net: phy: aquantia: add hwmon support")
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
---
Added the Fixes: tag and resending separately as a backportable fix.

 drivers/net/phy/aquantia/aquantia.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/net/phy/aquantia/aquantia.h b/drivers/net/phy/aquantia/aquantia.h
index c0e1fd9d7152..b8502793962e 100644
--- a/drivers/net/phy/aquantia/aquantia.h
+++ b/drivers/net/phy/aquantia/aquantia.h
@@ -6,6 +6,9 @@
  * Author: Heiner Kallweit <hkallweit1@gmail.com>
  */
 
+#ifndef AQUANTIA_H
+#define AQUANTIA_H
+
 #include <linux/device.h>
 #include <linux/phy.h>
 
@@ -198,3 +201,5 @@ int aqr_phy_led_hw_control_set(struct phy_device *phydev, u8 index,
 int aqr_phy_led_active_low_set(struct phy_device *phydev, int index, bool enable);
 int aqr_phy_led_polarity_set(struct phy_device *phydev, int index,
 			     unsigned long modes);
+
+#endif /* AQUANTIA_H */
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [RESEND PATCH net] net: phy: aquantia: add missing include guards
  2024-06-27 10:58 [RESEND PATCH net] net: phy: aquantia: add missing include guards Bartosz Golaszewski
@ 2024-06-27 21:59 ` Jakub Kicinski
  2024-06-28  7:56   ` Bartosz Golaszewski
  0 siblings, 1 reply; 4+ messages in thread
From: Jakub Kicinski @ 2024-06-27 21:59 UTC (permalink / raw)
  To: Bartosz Golaszewski
  Cc: Andrew Lunn, Heiner Kallweit, Russell King, David S . Miller,
	Eric Dumazet, Paolo Abeni, netdev, linux-kernel,
	Bartosz Golaszewski

On Thu, 27 Jun 2024 12:58:45 +0200 Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> 
> The header is missing the include guards so add them.
> 
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> Fixes: fb470f70fea7 ("net: phy: aquantia: add hwmon support")
> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

You say net but it doesn't apply:

Applying: net: phy: aquantia: add missing include guards
error: patch failed: drivers/net/phy/aquantia/aquantia.h:198
error: drivers/net/phy/aquantia/aquantia.h: patch does not apply
Patch failed at 0001 net: phy: aquantia: add missing include guards
-- 
pw-bot: cr

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RESEND PATCH net] net: phy: aquantia: add missing include guards
  2024-06-27 21:59 ` Jakub Kicinski
@ 2024-06-28  7:56   ` Bartosz Golaszewski
  2024-06-28 23:59     ` Jakub Kicinski
  0 siblings, 1 reply; 4+ messages in thread
From: Bartosz Golaszewski @ 2024-06-28  7:56 UTC (permalink / raw)
  To: Jakub Kicinski
  Cc: Andrew Lunn, Heiner Kallweit, Russell King, David S . Miller,
	Eric Dumazet, Paolo Abeni, netdev, linux-kernel,
	Bartosz Golaszewski

On Thu, Jun 27, 2024 at 11:59 PM Jakub Kicinski <kuba@kernel.org> wrote:
>
> On Thu, 27 Jun 2024 12:58:45 +0200 Bartosz Golaszewski wrote:
> > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> >
> > The header is missing the include guards so add them.
> >
> > Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> > Fixes: fb470f70fea7 ("net: phy: aquantia: add hwmon support")
> > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
>
> You say net but it doesn't apply:
>
> Applying: net: phy: aquantia: add missing include guards
> error: patch failed: drivers/net/phy/aquantia/aquantia.h:198
> error: drivers/net/phy/aquantia/aquantia.h: patch does not apply
> Patch failed at 0001 net: phy: aquantia: add missing include guards
> --
> pw-bot: cr

I resent a rebased version. However I noticed I forgot the 'net'
prefix as I split it out of the previous series. Sorry, I hope it can
still be applied without resending again.

Bart

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RESEND PATCH net] net: phy: aquantia: add missing include guards
  2024-06-28  7:56   ` Bartosz Golaszewski
@ 2024-06-28 23:59     ` Jakub Kicinski
  0 siblings, 0 replies; 4+ messages in thread
From: Jakub Kicinski @ 2024-06-28 23:59 UTC (permalink / raw)
  To: Bartosz Golaszewski
  Cc: Andrew Lunn, Heiner Kallweit, Russell King, David S . Miller,
	Eric Dumazet, Paolo Abeni, netdev, linux-kernel,
	Bartosz Golaszewski

On Fri, 28 Jun 2024 09:56:31 +0200 Bartosz Golaszewski wrote:
> > You say net but it doesn't apply:
> >
> > Applying: net: phy: aquantia: add missing include guards
> > error: patch failed: drivers/net/phy/aquantia/aquantia.h:198
> > error: drivers/net/phy/aquantia/aquantia.h: patch does not apply
> > Patch failed at 0001 net: phy: aquantia: add missing include guards
> > --
> > pw-bot: cr  
> 
> I resent a rebased version. However I noticed I forgot the 'net'
> prefix as I split it out of the previous series. Sorry, I hope it can
> still be applied without resending again.

The bot wasn't clever enough, sorry :( Third time is the charm..

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-06-28 23:59 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-27 10:58 [RESEND PATCH net] net: phy: aquantia: add missing include guards Bartosz Golaszewski
2024-06-27 21:59 ` Jakub Kicinski
2024-06-28  7:56   ` Bartosz Golaszewski
2024-06-28 23:59     ` Jakub Kicinski

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).