Thread (1 message) 1 message, 1 author, 2020-06-23

Re: [PATCH 04/15] net: mdio: add a forward declaration for reset_control to mdio.h

From: Florian Fainelli <f.fainelli@gmail.com>
Date: 2020-06-23 18:59:59
Also in: linux-devicetree, linux-mediatek, lkml, netdev

On 6/22/20 2:37 AM, Bartosz Golaszewski wrote:
quoted hunk
From: Bartosz Golaszewski <redacted>

This header refers to struct reset_control but doesn't include any reset
header. The structure definition is probably somehow indirectly pulled in
since no warnings are reported but for the sake of correctness add the
forward declaration for struct reset_control.

Signed-off-by: Bartosz Golaszewski <redacted>
---
 include/linux/mdio.h | 1 +
 1 file changed, 1 insertion(+)
diff --git a/include/linux/mdio.h b/include/linux/mdio.h
index 36d2e0673d03..9ac5e7ff6156 100644
--- a/include/linux/mdio.h
+++ b/include/linux/mdio.h
@@ -17,6 +17,7 @@
 #define MII_REGADDR_C45_MASK	GENMASK(15, 0)
 
 struct gpio_desc;
+struct reset_control;
 struct mii_bus;
You wrote 3 patches to sort the headers alphabetically, do you mind
doing the same thing for forward declarations as well?
-- 
Florian

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help