[v2,0/4] dev_err_probe usage for imx8qxp DPU pipeline

Message

Alexander Stein Jan. 7, 2025, 2:38 p.m. UTC

Hi everyone,

this is a v2 of an old series still in my queue.

Changes in v2:
* Rebase to next-20250107
* Remove 'imx' prefix for commit subject in patch 2 & 3

Best regards,
Alexander

Alexander Stein (4):
  drm/bridge: imx8qxp-pxl2dpi: Use dev_err_probe
  drm/bridge: imx8qxp-ldb: Use dev_err_probe
  drm/bridge: imx-ldb-helper: Use dev_err_probe
  drm/bridge: imx8qxp-pixel-link: Use dev_err_probe

 drivers/gpu/drm/bridge/imx/imx-ldb-helper.c   | 13 ++-----
 drivers/gpu/drm/bridge/imx/imx8qxp-ldb.c      | 37 ++++++-------------
 .../gpu/drm/bridge/imx/imx8qxp-pixel-link.c   | 19 +++-------
 drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c  | 26 ++++---------
 4 files changed, 29 insertions(+), 66 deletions(-)

Comments

Liu Ying Jan. 8, 2025, 3:13 a.m. UTC | #1

On 01/07/2025, Alexander Stein wrote:
> Hi everyone,

Hi Alexander,

> 
> this is a v2 of an old series still in my queue.
> 
> Changes in v2:
> * Rebase to next-20250107
> * Remove 'imx' prefix for commit subject in patch 2 & 3
> 
> Best regards,
> Alexander
> 
> Alexander Stein (4):
>   drm/bridge: imx8qxp-pxl2dpi: Use dev_err_probe
>   drm/bridge: imx8qxp-ldb: Use dev_err_probe
>   drm/bridge: imx-ldb-helper: Use dev_err_probe
>   drm/bridge: imx8qxp-pixel-link: Use dev_err_probe

i.MX8QXP Pixel Combiner is also in the pipeline, so imx8qxp-pixel-combiner.c
needs to be changed.

To achieve consistent log upon probe deferral, i.MX8QM LDB bridge driver
imx8qm-ldb.c needs to be changed too, but it can be done with a separate patch.

> 
>  drivers/gpu/drm/bridge/imx/imx-ldb-helper.c   | 13 ++-----
>  drivers/gpu/drm/bridge/imx/imx8qxp-ldb.c      | 37 ++++++-------------
>  .../gpu/drm/bridge/imx/imx8qxp-pixel-link.c   | 19 +++-------
>  drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c  | 26 ++++---------
>  4 files changed, 29 insertions(+), 66 deletions(-)
>

Alexander Stein Jan. 8, 2025, 9:46 a.m. UTC | #2

Hi,

Am Mittwoch, 8. Januar 2025, 04:13:51 CET schrieb Liu Ying:
> On 01/07/2025, Alexander Stein wrote:
> > Hi everyone,
> 
> Hi Alexander,
> 
> > 
> > this is a v2 of an old series still in my queue.
> > 
> > Changes in v2:
> > * Rebase to next-20250107
> > * Remove 'imx' prefix for commit subject in patch 2 & 3
> > 
> > Best regards,
> > Alexander
> > 
> > Alexander Stein (4):
> >   drm/bridge: imx8qxp-pxl2dpi: Use dev_err_probe
> >   drm/bridge: imx8qxp-ldb: Use dev_err_probe
> >   drm/bridge: imx-ldb-helper: Use dev_err_probe
> >   drm/bridge: imx8qxp-pixel-link: Use dev_err_probe
> 
> i.MX8QXP Pixel Combiner is also in the pipeline, so imx8qxp-pixel-combiner.c
> needs to be changed.
> 
> To achieve consistent log upon probe deferral, i.MX8QM LDB bridge driver
> imx8qm-ldb.c needs to be changed too, but it can be done with a separate patch.

Thanks for spotting. I've added both for v2 of this series.

Best regards,
Alexander

> > 
> >  drivers/gpu/drm/bridge/imx/imx-ldb-helper.c   | 13 ++-----
> >  drivers/gpu/drm/bridge/imx/imx8qxp-ldb.c      | 37 ++++++-------------
> >  .../gpu/drm/bridge/imx/imx8qxp-pixel-link.c   | 19 +++-------
> >  drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c  | 26 ++++---------
> >  4 files changed, 29 insertions(+), 66 deletions(-)
> > 
> 
>