Re: [PATCH DRM] drm: msm: disp: dpu1: Use DRM_DEV_* instead of DPU_*

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 09, 2018 at 07:35:33PM +0530, Mamta Shukla wrote:
> On Wed, Nov 7, 2018 at 12:25 AM Sean Paul <sean@xxxxxxxxxx> wrote:
> >
> > On Sat, Oct 20, 2018 at 11:33:54PM +0530, Mamta Shukla wrote:
> > > Use DRM_DEV_ERROR/DEBUG instead of DPU_ERROR/DEBUG  to generate
> > > drm-formatted specific log messages in the Kernel log in case of
> > > multiple instances.
> > >
> > > Signed-off-by: Mamta Shukla <mamtashukla555@xxxxxxxxx>
> >
> > Hi Mamta,
> > Thanks for sending this patch, but I don't think it's going to even compile.
> > DRM_DEV_* takes a struct device * pointer as the first argument, and it's not
> > being passed in anywhere. Make sure you compile test with CONFIG_DRM_MSM enabled
> >
> Thank you for pointing this out. Definitely DRM_DEV_* requires pointer
> to function or the device pointer along with
> message string. I had compiled this before submitting the patch. But I
> missed out enabling the DRM_MSM.
> Is it good to write v2 of above patch with mentioning/stating pointer
> to function so that in DEBUG message function
> name is also available with error message?

Yes, to do this right, you'd pass the device pointer where applicable (most
everwhere). You'll also want to convert the dev_*/pr_*/DRM_* instances to
DRM_DEV_*.

Sean

> 
> 
> > > ---
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c       |  58 +++----
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c      |  60 +++----
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c           | 176 ++++++++++-----------
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c        | 134 ++++++++--------
> > >  .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c   |  40 ++---
> > >  .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_vid.c   |  42 ++---
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c        |  10 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c     |   2 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.c         |   2 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c         |   2 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c        |   2 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c          |   2 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.c    |   2 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c        |   2 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c         |   2 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_irq.c            |   4 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c            |  90 +++++------
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c           |  10 +-
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c          | 134 ++++++++--------
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c             |  96 +++++------
> > >  drivers/gpu/drm/msm/disp/dpu1/dpu_vbif.c           |  32 ++--
> > >  21 files changed, 451 insertions(+), 451 deletions(-)
> > >
> >
> > /snip
> >
> > --
> > Sean Paul, Software Engineer, Google / Chromium OS
> 
> 
> 
> -- 
> Mamta Shukla

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux