Re: ld.lld: error: undefined symbol: gpmc_omap_onenand_set_timings

[Roger Quadros <rogerq@kernel.org>]

On 07/11/2022 10:42, Krzysztof Kozlowski wrote:
> On 07/11/2022 09:41, Roger Quadros wrote:
>>
>>
>> On 04/11/2022 21:36, Krzysztof Kozlowski wrote:
>>> On 04/11/2022 15:29, kernel test robot wrote:
>>>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>>>> head:   ee6050c8af96bba2f81e8b0793a1fc2f998fcd20
>>>> commit: 854fd9209b20837ab48c2e6714e5e44dd8ea33a2 memory: omap-gpmc: Allow building as a module
>>>> date:   6 months ago
>>>> config: arm-buildonly-randconfig-r004-20221104
>>>> compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 2bbafe04fe785a9469bea5a3737f8d7d3ce4aca2)
>>>> reproduce (this is a W=1 build):
>>>>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>>>         chmod +x ~/bin/make.cross
>>>>         # install arm cross compiling tool for clang build
>>>>         # apt-get install binutils-arm-linux-gnueabi
>>>>         # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=854fd9209b20837ab48c2e6714e5e44dd8ea33a2
>>>>         git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
>>>>         git fetch --no-tags linus master
>>>>         git checkout 854fd9209b20837ab48c2e6714e5e44dd8ea33a2
>>>>         # save the config file
>>>>         mkdir build_dir && cp config build_dir/.config
>>>>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash
>>>>
>>>> If you fix the issue, kindly add following tag where applicable
>>>> | Reported-by: kernel test robot <lkp@intel.com>
>>>
>>> +Cc Miquel,
>>>
>>> Hm, I am pretty sure we already fixed it... but it seems not. I think
>>> this needs dependency in MTD_ONENAND_OMAP2 - one is module, second
>>> cannot be built-in.
>>
>> We fixed MTD_NAND_OMAP2 but missed this MTD_ONENAND_OMAP2.
>>
>> I'll send out a patch soon.
> 
> I already sent a patch for this. Can you review it?

Just saw it after I replied to this one :)
Thanks! I'll review it.

cheers,
-roger