[v2] retrieve DSI DSC through DRM bridge

[PATCH v2 0/2] retrieve DSI DSC through DRM bridge

Kuogee Hsieh posted 2 patches 2 years, 8 months ago

Diff against v1
Download series mbox

drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 18 +++++++++++++-----
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h |  1 -
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c     |  2 --
3 files changed, 13 insertions(+), 8 deletions(-)

Expand all Fold all

[PATCH v2 0/2] retrieve DSI DSC through DRM bridge

Posted by Kuogee Hsieh 2 years, 8 months ago

move retrieving DSC from setup_display to atomic_check() and delete struct drm_dsc_config
from struct msm_display_info.

Kuogee Hsieh (2):
  drm/msm/dpu: retrieve DSI DSC struct at atomic_check()
  drm/msm/dpu: remove struct drm_dsc_config from struct msm_display_info

 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 18 +++++++++++++-----
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h |  1 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c     |  2 --
 3 files changed, 13 insertions(+), 8 deletions(-)

-- 
2.7.4

Re: [PATCH v2 0/2] retrieve DSI DSC through DRM bridge

Posted by Dmitry Baryshkov 2 years, 8 months ago

On 02/06/2023 01:08, Kuogee Hsieh wrote:
> move retrieving DSC from setup_display to atomic_check() and delete struct drm_dsc_config
> from struct msm_display_info.

This is obvious from the patches themselves. You should be describing 
_why_ the changes are necessary, not what is changed.


What was changed between v1 and v2?

> 
> Kuogee Hsieh (2):
>    drm/msm/dpu: retrieve DSI DSC struct at atomic_check()
>    drm/msm/dpu: remove struct drm_dsc_config from struct msm_display_info
> 
>   drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 18 +++++++++++++-----
>   drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h |  1 -
>   drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c     |  2 --
>   3 files changed, 13 insertions(+), 8 deletions(-)
> 

-- 
With best wishes
Dmitry