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