diff mbox series

[-next] media: dvb/frontend.h: fix kernel-doc warnings

Message ID 20221128054303.3124-1-rdunlap@infradead.org (mailing list archive)
State New, archived
Headers show
Series [-next] media: dvb/frontend.h: fix kernel-doc warnings | expand

Commit Message

Randy Dunlap Nov. 28, 2022, 5:43 a.m. UTC
scripts/kernel-doc spouts multiple warnings, so fix them:

include/uapi/linux/dvb/frontend.h:399: warning: Enum value 'QAM_1024' not described in enum 'fe_modulation'
include/uapi/linux/dvb/frontend.h:399: warning: Enum value 'QAM_4096' not described in enum 'fe_modulation'
frontend.h:286: warning: contents before sections
frontend.h:780: warning: missing initial short description on line:
 * enum atscmh_rs_code_mode

Fixes: 8220ead805b6 ("media: dvb/frontend.h: document the uAPI file")
Fixes: 6508a50fe84f ("media: dvb: add DVB-C2 and DVB-S2X parameter values")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: linux-media@vger.kernel.org
Cc: Robert Schlabbach <robert_s@gmx.net>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
---
 include/uapi/linux/dvb/frontend.h |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

Comments

Robert Schlabbach Nov. 28, 2022, 7:54 a.m. UTC | #1
> From: "Randy Dunlap" <rdunlap@infradead.org>
> Subject: [PATCH -next] media: dvb/frontend.h: fix kernel-doc warnings
> 
> scripts/kernel-doc spouts multiple warnings, so fix them:

Wow, sorry for introducing those, and thanks for promptly providing a fix!

Reviewed-by: Robert Schlabbach <robert_s@gmx.net>

Best Regards,
Bagas Sanjaya Dec. 6, 2022, 9:49 a.m. UTC | #2
On Sun, Nov 27, 2022 at 09:43:03PM -0800, Randy Dunlap wrote:
> scripts/kernel-doc spouts multiple warnings, so fix them:
> 
> include/uapi/linux/dvb/frontend.h:399: warning: Enum value 'QAM_1024' not described in enum 'fe_modulation'
> include/uapi/linux/dvb/frontend.h:399: warning: Enum value 'QAM_4096' not described in enum 'fe_modulation'
> frontend.h:286: warning: contents before sections
> frontend.h:780: warning: missing initial short description on line:
>  * enum atscmh_rs_code_mode
> 

These warnings are gone, thanks!

Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>
diff mbox series

Patch

diff -- a/include/uapi/linux/dvb/frontend.h b/include/uapi/linux/dvb/frontend.h
--- a/include/uapi/linux/dvb/frontend.h
+++ b/include/uapi/linux/dvb/frontend.h
@@ -282,7 +282,6 @@  enum fe_spectral_inversion {
 /**
  * enum fe_code_rate - Type of Forward Error Correction (FEC)
  *
- *
  * @FEC_NONE: No Forward Error Correction Code
  * @FEC_1_2:  Forward Error Correction Code 1/2
  * @FEC_2_3:  Forward Error Correction Code 2/3
@@ -363,8 +362,8 @@  enum fe_code_rate {
  * @APSK_32:	32-APSK modulation
  * @DQPSK:	DQPSK modulation
  * @QAM_4_NR:	4-QAM-NR modulation
- * @QAM-1024:	1024-QAM modulation
- * @QAM-4096:	4096-QAM modulation
+ * @QAM_1024:	1024-QAM modulation
+ * @QAM_4096:	4096-QAM modulation
  * @APSK_8_L:	8APSK-L modulation
  * @APSK_16_L:	16APSK-L modulation
  * @APSK_32_L:	32APSK-L modulation
@@ -777,7 +776,7 @@  enum atscmh_rs_frame_mode {
 };
 
 /**
- * enum atscmh_rs_code_mode
+ * enum atscmh_rs_code_mode - ATSC-M/H Reed Solomon modes
  * @ATSCMH_RSCODE_211_187:	Reed Solomon code (211,187).
  * @ATSCMH_RSCODE_223_187:	Reed Solomon code (223,187).
  * @ATSCMH_RSCODE_235_187:	Reed Solomon code (235,187).