On Thu, Apr 08, 2021 at 10:21:12AM -0500, Rob Herring wrote: > On Mon, Apr 05, 2021 at 01:18:15PM -0700, Matthias Kaehlcke wrote: > > Code for platform_device_create() and of_platform_device_create() is > > platform_device_create()? ack, should be of_platform_device_destroy() > > only generated if CONFIG_OF_ADDRESS=y. Add stubs to avoid unresolved > > symbols when CONFIG_OF_ADDRESS is not set. > > > > Signed-off-by: Matthias Kaehlcke <mka@xxxxxxxxxxxx> > > --- > > > > Changes in v7: > > - none > > > > Changes in v6: > > - patch added to the series > > > > include/linux/of_platform.h | 22 ++++++++++++++++++---- > > 1 file changed, 18 insertions(+), 4 deletions(-) > > > > diff --git a/include/linux/of_platform.h b/include/linux/of_platform.h > > index 84a966623e78..d15b6cd5e1c3 100644 > > --- a/include/linux/of_platform.h > > +++ b/include/linux/of_platform.h > > @@ -61,16 +61,18 @@ static inline struct platform_device *of_find_device_by_node(struct device_node > > } > > #endif > > > > +extern int of_platform_bus_probe(struct device_node *root, > > + const struct of_device_id *matches, > > + struct device *parent); > > This is also only built for CONFIG_OF_ADDRESS. But there's no need for > an empty function as it is powerpc only and should never have a new > user. ok