RE: [PATCH 0/6] Samsung Exynos 7870 DECON driver support

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

 



Hi Rob Herring,

> -----Original Message-----
> From: Rob Herring <robh@xxxxxxxxxx>
> Sent: Wednesday, November 6, 2024 5:11 AM
> To: 대인기/Tizen Platform Lab(SR)/삼성전자 <inki.dae@xxxxxxxxxxx>
> Cc: Kaustabh Chakraborty <kauschluss@xxxxxxxxxxx>; Seung-Woo Kim
> <sw0312.kim@xxxxxxxxxxx>; Kyungmin Park <kyungmin.park@xxxxxxxxxxx>; David
> Airlie <airlied@xxxxxxxxx>; Simona Vetter <simona@xxxxxxxx>; Krzysztof
> Kozlowski <krzk@xxxxxxxxxx>; Alim Akhtar <alim.akhtar@xxxxxxxxxxx>;
> Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>; Maxime Ripard
> <mripard@xxxxxxxxxx>; Thomas Zimmermann <tzimmermann@xxxxxxx>; Conor
> Dooley <conor@xxxxxxxxxx>; dri-devel@xxxxxxxxxxxxxxxxxxxxx; linux-arm-
> kernel@xxxxxxxxxxxxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 0/6] Samsung Exynos 7870 DECON driver support
> 
> On Fri, Nov 1, 2024 at 12:08 AM 대인기/Tizen Platform Lab(SR)/삼성전자
> <inki.dae@xxxxxxxxxxx> wrote:
> >
> > Hi Kaustabh Chakraborty,
> >
> > Sorry for late.
> >
> > > -----Original Message-----
> > > From: Kaustabh Chakraborty <kauschluss@xxxxxxxxxxx>
> > > Sent: Friday, September 20, 2024 12:11 AM
> > > To: Inki Dae <inki.dae@xxxxxxxxxxx>; Seung-Woo Kim
> > > <sw0312.kim@xxxxxxxxxxx>; Kyungmin Park <kyungmin.park@xxxxxxxxxxx>;
> David
> > > Airlie <airlied@xxxxxxxxx>; Simona Vetter <simona@xxxxxxxx>; Krzysztof
> > > Kozlowski <krzk@xxxxxxxxxx>; Alim Akhtar <alim.akhtar@xxxxxxxxxxx>;
> > > Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>; Maxime Ripard
> > > <mripard@xxxxxxxxxx>; Thomas Zimmermann <tzimmermann@xxxxxxx>; Rob
> Herring
> > > <robh@xxxxxxxxxx>; Conor Dooley <conor@xxxxxxxxxx>
> > > Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx; linux-arm-
> kernel@xxxxxxxxxxxxxxxxxxx;
> > > linux-samsung-soc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> > > devicetree@xxxxxxxxxxxxxxx; Kaustabh Chakraborty
> <kauschluss@xxxxxxxxxxx>
> > > Subject: [PATCH 0/6] Samsung Exynos 7870 DECON driver support
> > >
> > > This patch series aims at adding support for Exynos7870's DECON in the
> > > Exynos7 DECON driver. It introduces a driver data struct so that
> support
> > > for DECON on other SoCs can be added to it in the future.
> > >
> > > It also fixes a few bugs in the driver, such as functions recieving
> bad
> > > pointers.
> > >
> > > Tested on Samsung Galaxy J7 Prime and Samsung Galaxy A2 Core.
> > >
> > > Signed-off-by: Kaustabh Chakraborty <kauschluss@xxxxxxxxxxx>
> > > ---
> > > Kaustabh Chakraborty (6):
> > >       drm/exynos: exynos7_drm_decon: fix uninitialized crtc reference
> in
> > > functions
> > >       drm/exynos: exynos7_drm_decon: fix suspended condition in
> > > decon_commit()
> > >       drm/exynos: exynos7_drm_decon: fix ideal_clk by converting it to
> Hz
> > >       drm/exynos: exynos7_drm_decon: properly clear channels during
> bind
> > >       drm/exynos: exynos7_drm_decon: add driver data and support for
> > > Exynos7870
> > >       dt-bindings: display: samsung,exynos7-decon: add exynos7870
> > > compatible
> >
> > I will apply all except for the two patches below,
> > [PATCH 2/6] drm/exynos: exynos7_drm_decon: fix suspended condition in
> decon_commit()
> > [PATCH 6/6] dt-bindings: display: samsung,exynos7-decon: add exynos7870
> compatible
> 
> Now we have a warning in linux-next that samsung,exynos7870-decon is
> not documented.
> 
> Please apply the binding patch. Or let me know if it missed 6.13 for
> DRM tree and I'll apply it.
> 

Ah... sorry for this. I didn't check the warning. Will apply the binding patch. I was awaiting the submission of DTS.

Thanks,
Inki Dae

> Rob







[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