[PATCH] ltq-vdsl-app: fix -Wundef warnings
Adrian Schmutzler
mail at adrianschmutzler.de
Thu Feb 18 12:07:15 EST 2021
> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-bounces at lists.openwrt.org]
> On Behalf Of Mathias Kresin
> Sent: Mittwoch, 17. Februar 2021 21:20
> To: Adrian Schmutzler <mail at adrianschmutzler.de>; openwrt-
> devel at lists.openwrt.org
> Subject: Re: [PATCH] ltq-vdsl-app: fix -Wundef warnings
>
> 2/16/21 10:54 PM, Adrian Schmutzler:
> > Hi,
> >
> >> -----Original Message-----
> >> From: openwrt-devel [mailto:openwrt-devel-
> bounces at lists.openwrt.org]
> >> On Behalf Of Mathias Kresin
> >> Sent: Dienstag, 16. Februar 2021 19:35
> >> To: openwrt-devel at lists.openwrt.org
> >> Subject: [PATCH] ltq-vdsl-app: fix -Wundef warnings
> >>
> >> The following warnings are shown during build:
> >>
> >> /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)
> >> ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> >>
> >> The headers are provided by the MEI driver, but the defines are never
> >> set by the vdsl app. While the struct with the
> >> MEI_SUPPORT_OPTIMIZED_FW_DL conditional isn't used by the vdsl app,
> >> however CMV_USED_PAYLOAD_8BIT_SIZE which value depends on
> >> MEI_SUPPORT_DEBUG_STREAMS is.
> >>
> >> Since the MEI driver doesn't provide an autogenerated header with
> >> compile flags, the flags are hardcoded for the vdsl app.
> >>
> >> Set them for the MEI driver as well, to indicate a relation to the
> >> values used for the vdsl app and to be not surprised by a changed
> >> default in case the MEI driver gets updated. Use the current default
> >> values defined in the MEI driver.
> >
> > does this need PKG_RELEASE bump or is it really limited to altering
> compilation parameters?
>
> The change is limited to compile parameters without an intended change.
>
> But due to
>
> > ... isn't used by the vdsl app, however CMV_USED_PAYLOAD_8BIT_SIZE >
> which value depends on MEI_SUPPORT_DEBUG_STREAMS is
>
> a different binary is produced.
>
> I still tend to not bump the PKG_RELEASE but let me hear what you think
> about it.
>
Maybe the reproducible people will care.
Personally, I can live with both.
Best
Adrian
> Mathias
>
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel
More information about the openwrt-devel
mailing list