media: frontend.h.rst.exceptions: add more dvb define exceptions

Message ID 20230518234735.20289-1-rdunlap@infradead.org
State New
Headers
Series media: frontend.h.rst.exceptions: add more dvb define exceptions |

Commit Message

Randy Dunlap May 18, 2023, 11:47 p.m. UTC
  Building documentation reports multiple warnings for undefined DVB
frontend labels:

Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-11-45'
Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-4-15'
Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-14-45'
Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-7-15'
Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-11-45'
Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-4-15'
Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-14-45'
Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-7-15'

so add those symbols to the ignore list to prevent the build warnings.

Fixes: 1825788e2a96 ("media: dvb: add missing DVB-S2X FEC parameter values")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Reported-by: kernel test robot <lkp@intel.com>
Link: https://lore.kernel.org/linux-media/202305162245.wtaLIXf3-lkp@intel.com/
Cc: Athanasios Oikonomou <athoik@gmail.com>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: linux-media@vger.kernel.org
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: linux-doc@vger.kernel.org
---
 Documentation/userspace-api/media/frontend.h.rst.exceptions |    4 ++++
 1 file changed, 4 insertions(+)
  

Comments

Mauro Carvalho Chehab May 19, 2023, 8:03 a.m. UTC | #1
Hi Randy,

Em Thu, 18 May 2023 16:47:35 -0700
Randy Dunlap <rdunlap@infradead.org> escreveu:

> Building documentation reports multiple warnings for undefined DVB
> frontend labels:
> 
> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-11-45'
> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-4-15'
> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-14-45'
> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-7-15'
> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-11-45'
> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-4-15'
> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-14-45'
> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-7-15'

Thanks for the patch. FYI, I already merged yesterday a fix identical
to your patch:

	https://git.linuxtv.org/media_stage.git/commit/?id=8bc27fa5d7763d376a992a1638475987ed4807e7

Regards,
Mauro


> 
> so add those symbols to the ignore list to prevent the build warnings.
> 
> Fixes: 1825788e2a96 ("media: dvb: add missing DVB-S2X FEC parameter values")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Reported-by: kernel test robot <lkp@intel.com>
> Link: https://lore.kernel.org/linux-media/202305162245.wtaLIXf3-lkp@intel.com/
> Cc: Athanasios Oikonomou <athoik@gmail.com>
> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
> Cc: linux-media@vger.kernel.org
> Cc: Jonathan Corbet <corbet@lwn.net>
> Cc: linux-doc@vger.kernel.org
> ---
>  Documentation/userspace-api/media/frontend.h.rst.exceptions |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff -- a/Documentation/userspace-api/media/frontend.h.rst.exceptions b/Documentation/userspace-api/media/frontend.h.rst.exceptions
> --- a/Documentation/userspace-api/media/frontend.h.rst.exceptions
> +++ b/Documentation/userspace-api/media/frontend.h.rst.exceptions
> @@ -142,6 +142,10 @@ ignore symbol FEC_26_45
>  ignore symbol FEC_28_45
>  ignore symbol FEC_32_45
>  ignore symbol FEC_77_90
> +ignore symbol FEC_11_45
> +ignore symbol FEC_4_15
> +ignore symbol FEC_14_45
> +ignore symbol FEC_7_15
>  
>  ignore symbol TRANSMISSION_MODE_AUTO
>  ignore symbol TRANSMISSION_MODE_1K
  
Randy Dunlap May 19, 2023, 3:01 p.m. UTC | #2
On 5/19/23 01:03, Mauro Carvalho Chehab wrote:
> Hi Randy,
> 
> Em Thu, 18 May 2023 16:47:35 -0700
> Randy Dunlap <rdunlap@infradead.org> escreveu:
> 
>> Building documentation reports multiple warnings for undefined DVB
>> frontend labels:
>>
>> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-11-45'
>> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-4-15'
>> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-14-45'
>> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-7-15'
>> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-11-45'
>> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-4-15'
>> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-14-45'
>> Documentation/output/frontend.h.rst:6: WARNING: undefined label: 'fec-7-15'
> 
> Thanks for the patch. FYI, I already merged yesterday a fix identical
> to your patch:
> 
> 	https://git.linuxtv.org/media_stage.git/commit/?id=8bc27fa5d7763d376a992a1638475987ed4807e7
> 
> Regards,
> Mauro


Hi Mauro,

Did you post that patch to the linux-media mailing list?
I searched that list for such a patch before I made the patch.

Thanks.

> 
>>
>> so add those symbols to the ignore list to prevent the build warnings.
>>
>> Fixes: 1825788e2a96 ("media: dvb: add missing DVB-S2X FEC parameter values")
>> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
>> Reported-by: kernel test robot <lkp@intel.com>
>> Link: https://lore.kernel.org/linux-media/202305162245.wtaLIXf3-lkp@intel.com/
>> Cc: Athanasios Oikonomou <athoik@gmail.com>
>> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
>> Cc: linux-media@vger.kernel.org
>> Cc: Jonathan Corbet <corbet@lwn.net>
>> Cc: linux-doc@vger.kernel.org
>> ---
>>  Documentation/userspace-api/media/frontend.h.rst.exceptions |    4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff -- a/Documentation/userspace-api/media/frontend.h.rst.exceptions b/Documentation/userspace-api/media/frontend.h.rst.exceptions
>> --- a/Documentation/userspace-api/media/frontend.h.rst.exceptions
>> +++ b/Documentation/userspace-api/media/frontend.h.rst.exceptions
>> @@ -142,6 +142,10 @@ ignore symbol FEC_26_45
>>  ignore symbol FEC_28_45
>>  ignore symbol FEC_32_45
>>  ignore symbol FEC_77_90
>> +ignore symbol FEC_11_45
>> +ignore symbol FEC_4_15
>> +ignore symbol FEC_14_45
>> +ignore symbol FEC_7_15
>>  
>>  ignore symbol TRANSMISSION_MODE_AUTO
>>  ignore symbol TRANSMISSION_MODE_1K
  

Patch

diff -- a/Documentation/userspace-api/media/frontend.h.rst.exceptions b/Documentation/userspace-api/media/frontend.h.rst.exceptions
--- a/Documentation/userspace-api/media/frontend.h.rst.exceptions
+++ b/Documentation/userspace-api/media/frontend.h.rst.exceptions
@@ -142,6 +142,10 @@  ignore symbol FEC_26_45
 ignore symbol FEC_28_45
 ignore symbol FEC_32_45
 ignore symbol FEC_77_90
+ignore symbol FEC_11_45
+ignore symbol FEC_4_15
+ignore symbol FEC_14_45
+ignore symbol FEC_7_15
 
 ignore symbol TRANSMISSION_MODE_AUTO
 ignore symbol TRANSMISSION_MODE_1K