Re: [PATCH v4 3/3] drm/msm/dp: replace DRM_DEBUG_DP marco with drm_dbg_dp

[Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx>]

Hi Stephen,

Are you still has concern on this patch?

Thanks,


On 2/17/2022 12:01 PM, Kuogee Hsieh wrote:
> On 2/17/2022 11:36 AM, Stephen Boyd wrote:
> > Quoting Kuogee Hsieh (2022-02-17 10:35:30)
> > > Since DRM_DEBUG_DP is deprecated in favor of drm_dbg_dp(NULL, ...),
> > > this patch replace all DRM_DEBUG_DP with drm_dbg_dp().
> > >
> > > Changes in v4:
> > > -- replace (strucr drm_dev *)NULL with drm_dev
> > Why can't the platform device be used?
> #define drm_dbg_dp(drm, fmt, ...)                                       \
>
>         drm_dev_dbg((drm) ? (drm)->dev : NULL, DRM_UT_DP, fmt, 
> ##__VA_ARGS__)
>
> it looks for (drm)->dev (pointer)
>
> struct platform_device {
>         const char      *name;
>         int             id;
>         bool            id_auto;
>         struct device   dev          <== not an pointer here