On Thu, Feb 28, 2013 at 05:13:09PM -0600, Nishanth Menon wrote:
Some basic cleanups for kernel-doc errors or missing documentation parameters.
Warnings generated by ./scripts/kernel-doc drivers/gpu/drm/drm_edid.c >Kerr
Warning(drivers/gpu/drm/drm_edid.c:997): No description found for parameter 'adapter' Warning(drivers/gpu/drm/drm_edid.c:997): No description found for parameter 'buf' Warning(drivers/gpu/drm/drm_edid.c:997): No description found for parameter 'block' Warning(drivers/gpu/drm/drm_edid.c:997): No description found for parameter 'len' Warning(drivers/gpu/drm/drm_edid.c:1138): No description found for parameter 'adapter' Warning(drivers/gpu/drm/drm_edid.c:1467): No description found for parameter 'connector' Warning(drivers/gpu/drm/drm_edid.c:1467): No description found for parameter 'edid' Warning(drivers/gpu/drm/drm_edid.c:1467): No description found for parameter 'revision' Warning(drivers/gpu/drm/drm_edid.c:1467): Excess function parameter 'timing_level' description in 'drm_mode_std' Warning(drivers/gpu/drm/drm_edid.c:2010): No description found for parameter 'connector' Warning(drivers/gpu/drm/drm_edid.c:2072): No description found for parameter 'connector' Warning(drivers/gpu/drm/drm_edid.c:2237): No description found for parameter 'edid' Warning(drivers/gpu/drm/drm_edid.c:2616): No description found for parameter 'edid' Warning(drivers/gpu/drm/drm_edid.c:2658): No description found for parameter 'edid'
Cc: David Airlie airlied@linux.ie Cc: Dave Airlie airlied@redhat.com Cc: Adam Jackson ajax@redhat.com Cc: Rodrigo Vivi rodrigo.vivi@intel.com Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Nishanth Menon nm@ti.com
It does seem the drm_edid.c could do with some amount of documentation cleanup, but I have stayed with the bare minimum. the above error is based off: master de1a226 Merge tag 'writeback-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/wfg/linux from: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git
defconfig used: omap2plus_defconfig
Disclaimer: I am no drm expert, and I tried to use my best judgement about what the meaning of the parameters might be, hope it helps.
drivers/gpu/drm/drm_edid.c | 31 +++++++++++++++++++------------ 1 file changed, 19 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index c194f4e..f604c5f 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -982,14 +982,14 @@ EXPORT_SYMBOL(drm_edid_is_valid);
#define DDC_SEGMENT_ADDR 0x30 /**
- Get EDID information via I2C.
- \param adapter : i2c device adaptor
- \param buf : EDID data buffer to be filled
- \param len : EDID data buffer length
- \return 0 on success or -1 on failure.
- drm_do_probe_ddc_edid() - Get EDID information via I2C.
- @adapter: i2c device adaptor
nit: adapter vs. adaptor
- @buf: EDID data buffer to be filled
- @block: EDID block to get
- @len: EDID data buffer length
- Try to fetch EDID information by calling i2c driver function.
*/
- Return 0 on success or -1 on failure.
static int drm_do_probe_ddc_edid(struct i2c_adapter *adapter, unsigned char *buf, @@ -1128,10 +1128,10 @@ out: }
/**
- Probe DDC presence.
- drm_probe_ddc() - Probe DDC presence
- @adapter: i2c device adaptor
- \param adapter : i2c device adaptor
- \return 1 on success
*/
- returns 1 on success
bool drm_probe_ddc(struct i2c_adapter *adapter) @@ -1455,8 +1455,10 @@ bad_std_timing(u8 a, u8 b)
/**
- drm_mode_std - convert standard mode info (width, height, refresh) into mode
- @connector: connector corresponding to the HDMI/DP sink
EDID isn't limited to HDMI/DP.
- @edid: EDID data
- @t: standard timing params
- @timing_level: standard timing level
- @revision: EDID revision
- Take the standard timing params (in this case width, aspect, and refresh)
- and convert them into a real mode using CVT/GTF/DMT.
@@ -2000,6 +2002,7 @@ do_established_modes(struct detailed_timing *timing, void *c)
/**
- add_established_modes - get est. modes from EDID and add them
- @connector: connector corresponding to the HDMI/DP sink
- @edid: EDID block to scan
- Each EDID block contains a bitmap of the supported "established modes" list
@@ -2062,6 +2065,7 @@ do_standard_modes(struct detailed_timing *timing, void *c)
/**
- add_standard_modes - get std. modes from EDID and add them
- @connector: connector corresponding to the HDMI/DP sink
- @edid: EDID block to scan
- Standard modes can be calculated using the appropriate standard (DMT,
@@ -2192,7 +2196,7 @@ do_detailed_mode(struct detailed_timing *timing, void *c) } }
-/* +/**
- add_detailed_modes - Add modes from detailed timings
- @connector: attached connector
- @edid: EDID block to scan
@@ -2231,7 +2235,8 @@ add_detailed_modes(struct drm_connector *connector, struct edid *edid, #define EDID_CEA_VCDB_QS (1 << 6)
/**
- Search EDID for CEA extension block.
- drm_find_cea_extension() - Search EDID for CEA extension block.
*/
- @edid: EDID block to scan
u8 *drm_find_cea_extension(struct edid *edid) { @@ -2604,6 +2609,7 @@ EXPORT_SYMBOL(drm_detect_hdmi_monitor);
/**
- drm_detect_monitor_audio - check monitor audio capability
- @edid: monitor EDID information
- Monitor should have CEA extension block.
- If monitor has 'basic audio', but no CEA audio blocks, it's 'basic
@@ -2649,6 +2655,7 @@ EXPORT_SYMBOL(drm_detect_monitor_audio);
/**
- drm_rgb_quant_range_selectable - is RGB quantization range selectable?
- @edid: EDID data
nit: Different description for the same thing between this and the previous hunk.
- Check whether the monitor reports the RGB quantization range selection
- as supported. The AVI infoframe can then be used to inform the monitor
-- 1.7.9.5
dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel