On Mon, Mar 31, 2025 at 6:32 PM Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > > Add a note to the fwnode.h that the header should not be used > directly in the leaf drivers, they all should use the higher > level APIs and the respective headers. This sounds like a solution to a problem, but the problem statement is missing. What's your motivation? > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > --- > include/linux/fwnode.h | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/include/linux/fwnode.h b/include/linux/fwnode.h > index 6fa0a268d538..da537c7c30dc 100644 > --- a/include/linux/fwnode.h > +++ b/include/linux/fwnode.h > @@ -2,6 +2,12 @@ > /* > * fwnode.h - Firmware device node object handle type definition. > * > + * Note, this header is not meant to be used by the leaf drivers. > + * It provides the low level data types and definitions for the firmware > + * and device property providers. The respective API headers should > + * guarantee all the required data types and definitions without including > + * this header directly. > + * > * Copyright (C) 2015, Intel Corporation > * Author: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > */ > -- > 2.47.2 > >