Commit cca5ff94 authored by Lee Jones's avatar Lee Jones Committed by Rob Clark

drm/msm/disp/dpu1/dpu_encoder: Fix a few parameter/member formatting issues

Fixes the following W=1 kernel build warning(s):

 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:207: warning: Function parameter or member 'cur_slave' not described in 'dpu_encoder_virt'
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:207: warning: Function parameter or member 'hw_pp' not described in 'dpu_encoder_virt'
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:207: warning: Function parameter or member 'intfs_swapped' not described in 'dpu_encoder_virt'
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1445: warning: Function parameter or member 'drm_enc' not described in '_dpu_encoder_trigger_flush'
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1445: warning: Function parameter or member 'phys' not described in '_dpu_encoder_trigger_flush'
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1445: warning: Function parameter or member 'extra_flush_bits' not described in '_dpu_encoder_trigger_flush'
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1481: warning: Function parameter or member 'phys' not described in '_dpu_encoder_trigger_start'
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:1564: warning: Function parameter or member 'dpu_enc' not described in '_dpu_encoder_kickoff_phys'

Cc: Rob Clark <robdclark@gmail.com>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Reviewed-by: default avatarAbhinav Kumar <abhinavk@codeaurora.org>
Signed-off-by: default avatarRob Clark <robdclark@chromium.org>
parent 09c7e370
...@@ -132,9 +132,10 @@ enum dpu_enc_rc_states { ...@@ -132,9 +132,10 @@ enum dpu_enc_rc_states {
* @phys_encs: Container of physical encoders managed. * @phys_encs: Container of physical encoders managed.
* @cur_master: Pointer to the current master in this mode. Optimization * @cur_master: Pointer to the current master in this mode. Optimization
* Only valid after enable. Cleared as disable. * Only valid after enable. Cleared as disable.
* @hw_pp Handle to the pingpong blocks used for the display. No. * @cur_slave: As above but for the slave encoder.
* @hw_pp: Handle to the pingpong blocks used for the display. No.
* pingpong blocks can be different than num_phys_encs. * pingpong blocks can be different than num_phys_encs.
* @intfs_swapped Whether or not the phys_enc interfaces have been swapped * @intfs_swapped: Whether or not the phys_enc interfaces have been swapped
* for partial update right-only cases, such as pingpong * for partial update right-only cases, such as pingpong
* split where virtual pingpong does not generate IRQs * split where virtual pingpong does not generate IRQs
* @crtc: Pointer to the currently assigned crtc. Normally you * @crtc: Pointer to the currently assigned crtc. Normally you
...@@ -1436,9 +1437,9 @@ static void dpu_encoder_off_work(struct work_struct *work) ...@@ -1436,9 +1437,9 @@ static void dpu_encoder_off_work(struct work_struct *work)
/** /**
* _dpu_encoder_trigger_flush - trigger flush for a physical encoder * _dpu_encoder_trigger_flush - trigger flush for a physical encoder
* drm_enc: Pointer to drm encoder structure * @drm_enc: Pointer to drm encoder structure
* phys: Pointer to physical encoder structure * @phys: Pointer to physical encoder structure
* extra_flush_bits: Additional bit mask to include in flush trigger * @extra_flush_bits: Additional bit mask to include in flush trigger
*/ */
static void _dpu_encoder_trigger_flush(struct drm_encoder *drm_enc, static void _dpu_encoder_trigger_flush(struct drm_encoder *drm_enc,
struct dpu_encoder_phys *phys, uint32_t extra_flush_bits) struct dpu_encoder_phys *phys, uint32_t extra_flush_bits)
...@@ -1475,7 +1476,7 @@ static void _dpu_encoder_trigger_flush(struct drm_encoder *drm_enc, ...@@ -1475,7 +1476,7 @@ static void _dpu_encoder_trigger_flush(struct drm_encoder *drm_enc,
/** /**
* _dpu_encoder_trigger_start - trigger start for a physical encoder * _dpu_encoder_trigger_start - trigger start for a physical encoder
* phys: Pointer to physical encoder structure * @phys: Pointer to physical encoder structure
*/ */
static void _dpu_encoder_trigger_start(struct dpu_encoder_phys *phys) static void _dpu_encoder_trigger_start(struct dpu_encoder_phys *phys)
{ {
...@@ -1558,7 +1559,7 @@ static void dpu_encoder_helper_hw_reset(struct dpu_encoder_phys *phys_enc) ...@@ -1558,7 +1559,7 @@ static void dpu_encoder_helper_hw_reset(struct dpu_encoder_phys *phys_enc)
* encoder rather than the individual physical ones in order to handle * encoder rather than the individual physical ones in order to handle
* use cases that require visibility into multiple physical encoders at * use cases that require visibility into multiple physical encoders at
* a time. * a time.
* dpu_enc: Pointer to virtual encoder structure * @dpu_enc: Pointer to virtual encoder structure
*/ */
static void _dpu_encoder_kickoff_phys(struct dpu_encoder_virt *dpu_enc) static void _dpu_encoder_kickoff_phys(struct dpu_encoder_virt *dpu_enc)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment