Re: [PATCH] clk: imx: scu: Fix compile error with module build of clk-scu.o

From: Randy Dunlap
Date: Mon Nov 02 2020 - 16:32:11 EST


On 11/2/20 1:01 PM, Valdis Klētnieks wrote:
> On Mon, 02 Nov 2020 09:15:20 -0800, Randy Dunlap said:
>
>> also
>> Reported-by: kernel test robot <lkp@xxxxxxxxx>
>>
>> However, this driver does not directly use <linux/module.h>.
>
> Just my luck - I looked at 3 or 4 other things that include of_platform.h
> and they all *did* include module.h.
>
>> platform_device.h #includes <linux/device.h>, which is where the
>> problem lies:
>>
>> <linux/device.h> uses macros that are provided by <linux/module.h>
>> so <linux/device.h> should #include <linux/module.h>.
>>
>> and that fixes this commit:
>>
>> commit 4c002c978b7f2f2306d53de051c054504af920a9
>> Author: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
>> Date: Mon Dec 9 20:33:03 2019 +0100
>>
>> device.h: move 'struct driver' stuff out to device/driver.h
>
> OK.. who's going to do that? Me, or Randy, or Greg?

You could go ahead... I began on it yesterday but didn't finish
testing, although I did see the same build error that the 0day
bot reported, so I don't know what it's going to take to fix that.


--
~Randy