Re: [PATCH] Re: linux-next: Tree for Feb 2 (media/radio/wl128x)

[Randy Dunlap <rdunlap@xenotime.net>]

On 02/02/2012 10:20 AM, Manjunatha Halli wrote:
> On 02/02/2012 12:40 PM, Randy Dunlap wrote:
>> On 02/02/2012 09:20 AM, Manjunatha Halli wrote:
>>> Hi Randy Dunlap,
>>>
>>> In config file you are missing the CONFIG_TI_ST config which builds the TI's shared transport driver upon which the FM driver works.
>>>
>>> Please select this config in drivers/misc/ti-st/Kconfig which will solve the problem.
>> Wrong answer.
>>
>> The problem seems to be that GPIOLIB is not enabled, but wl128x Kconfig says:
>>
>> config RADIO_WL128X
>>     tristate "Texas Instruments WL128x FM Radio"
>>     depends on VIDEO_V4L2&&  RFKILL
>>     select TI_ST if NET&&  GPIOLIB
>>
>> so TI_ST is not selected here.
>>
>> The Kconfig files should handle this properly.
>>
>> Here is one possible fix for you to consider.
>>
>> ---
>> From: Randy Dunlap<rdunlap@xenotime.net>
>>
>> Fix build errors when GPIOLIB is not enabled.
>> Fix wl128x Kconfig to depend on GPIOLIB since TI_ST also
>> depends on GPIOLIB.
>>
>> (.text+0xe6d60): undefined reference to `st_register'
>> (.text+0xe7016): undefined reference to `st_unregister'
>> (.text+0xe70ce): undefined reference to `st_unregister'
>>
>> Signed-off-by: Randy Dunlap<rdunlap@xenotime.net>
>> Cc: Manjunatha Halli<manjunatha_halli@ti.com>
>> ---
>>   drivers/media/radio/wl128x/Kconfig |    4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> --- linux-next-20120202.orig/drivers/media/radio/wl128x/Kconfig
>> +++ linux-next-20120202/drivers/media/radio/wl128x/Kconfig
>> @@ -4,8 +4,8 @@
>>   menu "Texas Instruments WL128x FM driver (ST based)"
>>   config RADIO_WL128X
>>       tristate "Texas Instruments WL128x FM Radio"
>> -    depends on VIDEO_V4L2&&  RFKILL
>> -    select TI_ST if NET&&  GPIOLIB
>> +    depends on VIDEO_V4L2&&  RFKILL&&  GPIOLIB
>> +    select TI_ST if NET
>>       help
>>       Choose Y here if you have this FM radio chip.
>>
>>
>>> Regards
>>> Manju
>>>
>>> On 02/02/2012 12:07 PM, Randy Dunlap wrote:
>>>> On 02/01/2012 07:45 PM, Stephen Rothwell wrote:
>>>>> Hi all,
>>>>>
>>>>> Changes since 20120201:
>>>> drivers/built-in.o: In function `fmc_prepare':
>>>> (.text+0xe6d60): undefined reference to `st_register'
>>>> drivers/built-in.o: In function `fmc_prepare':
>>>> (.text+0xe7016): undefined reference to `st_unregister'
>>>> drivers/built-in.o: In function `fmc_release':
>>>> (.text+0xe70ce): undefined reference to `st_unregister'
>>>>
>>>>
>>>> Full randconfig file is attached.
>>
> 
> This solutions seems fine for me...
> 
> My only concern is since TI_ST is already have GPIOLIB in its dependency list is it OK to have the same thing in FM driver also?.

Sure, it's OK.


-- 
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***