[PATCH 02/13] ltq-vdsl-app: fix -Wundef warnings

Andre Heider a.heider at gmail.com
Fri Dec 11 14:03:26 EST 2020


On 11/12/2020 19:44, Andre Heider wrote:
> On 11/12/2020 18:59, Mathias Kresin wrote:
>> 12/11/20 1:05 PM, Andre Heider:
>>> Shut up these warnings:
>>>
>>> /usr/include/vdsl/cmv_message_format.h:33:6: warning: 
>>> "MEI_SUPPORT_DEBUG_STREAMS" is not defined, evaluates to 0 [-Wundef]
>>>   #if (MEI_SUPPORT_DEBUG_STREAMS == 1)
>>>        ^~~~~~~~~~~~~~~~~~~~~~~~~
>>> /usr/include/vdsl/drv_mei_cpe_interface.h:2256:6: warning: 
>>> "MEI_SUPPORT_OPTIMIZED_FW_DL" is not defined, evaluates to 0 [-Wundef]
>>>   #if (MEI_SUPPORT_OPTIMIZED_FW_DL == 1)
>>>        ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>>>
>>> Signed-off-by: Andre Heider <a.heider at gmail.com>
>>> ---
>>>   package/network/config/ltq-vdsl-app/Makefile | 7 ++++++-
>>>   1 file changed, 6 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/package/network/config/ltq-vdsl-app/Makefile 
>>> b/package/network/config/ltq-vdsl-app/Makefile
>>> index 5e2db1d5c6..ef81f7c8d7 100644
>>> --- a/package/network/config/ltq-vdsl-app/Makefile
>>> +++ b/package/network/config/ltq-vdsl-app/Makefile
>>> @@ -35,6 +35,11 @@ define Package/ltq-vdsl-app/description
>>>     Userland tool needed to control Lantiq VDSL CPE
>>>   endef
>>> +APPL_CFLAGS= \
>>> +    -DMAX_CLI_PIPES=1 \
>>> +    -DMEI_SUPPORT_DEBUG_STREAMS=0 \
>>> +    -DMEI_SUPPORT_OPTIMIZED_FW_DL=0
>>
>> Hey Andre,
>>
>> optimized firmware download is enabled in the MEI driver by default:
>>
>> drv_mei_cpe-1.5.17.6/src/drv_mei_cpe_config.h
>>
>> #ifndef MEI_SUPPORT_OPTIMIZED_FW_DL
>> #  define MEI_SUPPORT_OPTIMIZED_FW_DL    1
>> #else
>>
>> cat /proc/driver/mei_cpe/status/00 shows the FwDownlOpt counter, which 
>> confirms the above.
>>
>> MEI_SUPPORT_OPTIMIZED_FW_DL should be rather defined as 1 to be on 
>> safe side, even if the IOCTL_MEI_statistic_t struct isn't used at all 
>> in the vdsl-app.
> 
> fishy...
> 
> Maybe drv_mei_cpe_config.h should be included before 
> drv_mei_cpe_interface.h then?

Hm nope, that fails to build. Looks like that's an internal header of 
the kernel driver.

All this looks so fragile, I'll just drop this patch from the series.

Thanks for spotting this!

> 
>>
>> regards
>> Mathias
> 




More information about the openwrt-devel mailing list