On Thu, Dec 05, 2024 at 10:12:43AM +0200, Jarkko Nikula wrote: > On 12/5/24 9:53 AM, Andy Shevchenko wrote: > > On Wed, Dec 04, 2024 at 11:25:40AM +0100, Uwe Kleine-König wrote: > > > On Wed, Dec 04, 2024 at 03:23:52AM +0200, Andy Shevchenko wrote: > > > > On Tue, Dec 03, 2024 at 11:46:07PM +0100, Uwe Kleine-König wrote: > > > > > On Tue, Dec 03, 2024 at 09:27:35PM +0200, Andy Shevchenko wrote: > > > > > > On Tue, Dec 03, 2024 at 06:36:40PM +0100, Uwe Kleine-König wrote: > > > > > > > DEFAULT_SYMBOL_NAMESPACE must already be defined when <linux/export.h> > > > > > > > is included. So move the define above the include block. > > > > > > > > > > > > As in the other email I pointed out the doc says that we need to undef the > > > > > > symbol. No need to move it around. > > > > > > > > > > > > The only requirement is to place that before any EXPORT_SYMBOL*() we want to > > > > > > add it to. [...] > > > > Perhaps we need to update the documentation first. > > > > > > I addressed that in https://lore.kernel.org/all/3dd7ff6fa0a636de86e091286016be8c90e03631.1733305665.git.ukleinek@xxxxxxxxxx/ > > > > Thank you! > Andy: is this your reviewed by? If so then > > Acked-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxxxxxxxx> I would like to have a clarifications from Documentation to be settled down first. When it's done, depending on the outcome it may or may not be my Rb tag. -- With Best Regards, Andy Shevchenko
