On 6/22/20 2:37 AM, Bartosz Golaszewski wrote: > From: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx> > > 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 <bgolaszewski@xxxxxxxxxxxx> > --- > 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
