RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display Unit support
Hi Laurent,
> -Original Message-
> From: Laurent Pinchart
> Sent: Friday, June 2, 2023 12:15 PM
> To: Biju Das
> Cc: David Airlie ; Daniel Vetter ;
> Philipp Zabel ; Geert Uytterhoeven
> ; Kieran Bingham
> ; dri-
> de...@lists.freedesktop.org; linux-renesas-...@vger.kernel.org; Fabrizio
> Castro ; Prabhakar Mahadev Lad
> ; linux-ker...@vger.kernel.org
> Subject: Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display
> Unit support
>
> Hi Biju,
>
> On Thu, Jun 01, 2023 at 12:12:59PM +, Biju Das wrote:
> > > Subject: Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > Display Unit support
> > >
> > > On Mon, May 29, 2023 at 02:22:06PM +0000, Biju Das wrote:
> > > > > Subject: Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > > > Display Unit support On Thu, May 25, 2023 at 02:30:10PM +,
> > > > > Biju Das wrote:
> > > > > > Hi DRM maintainers,
> > > > > >
> > > > > > Gentle ping.
> > > > >
> > > > > Sorry, I was on holidays the last two weeks.
> > > > >
> > > > > > Are we happy with moving all Renesas drm drivers to Renesas
> > > > > > specific directory or preference is for separate one??
> > > > >
> > > > > This works for me.
> > > > >
> > > > > > If it is later, I can send RZ/G2L drm driver separate.
> > > > > >
> > > > > > Otherwise, I need to rebase and resend.
> > > > >
> > > > > Your series applies cleanly on top of the latest drm-next
> > > > > branch. Is there a specific need to rebase and resend ?
> > > >
> > > > Nope. After my patch series there were some patches from Geert for
> > > > drm/shmobile merged to drm-misc-next by Thomas.
> > > >
> > > > Maybe git managed this automatically??
> > >
> > > Probably, git is nice :-)
> > >
> > > > > I haven't had time to review patch 4/5 (the driver) yet. All the
> > > > > rest looks good to me. Should I already include 1/5 in my next
> > > > > pull
> > > request ?
> > > >
> > > > Yes, please.
> > >
> > > OK, I will do so. I've reviewed 4/5 in the meantime, but changes are
> > > needed, so I won't wait for v10 before applying 1/5.
> >
> > I have incorporated review comments for v9. I need to rebase my
> changes.
> >
> > Is the pull request being done to drm-misc-next?
>
> I've just sent the pull request for drm-next, and have CC'ed you.
Thank you.
Cheers,
Biju
Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display Unit support
Hi Biju,
On Thu, Jun 01, 2023 at 12:12:59PM +, Biju Das wrote:
> > Subject: Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display
> > Unit support
> >
> > On Mon, May 29, 2023 at 02:22:06PM +, Biju Das wrote:
> > > > Subject: Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > > Display Unit support
> > > > On Thu, May 25, 2023 at 02:30:10PM +, Biju Das wrote:
> > > > > Hi DRM maintainers,
> > > > >
> > > > > Gentle ping.
> > > >
> > > > Sorry, I was on holidays the last two weeks.
> > > >
> > > > > Are we happy with moving all Renesas drm drivers to Renesas
> > > > > specific directory or preference is for separate one??
> > > >
> > > > This works for me.
> > > >
> > > > > If it is later, I can send RZ/G2L drm driver separate.
> > > > >
> > > > > Otherwise, I need to rebase and resend.
> > > >
> > > > Your series applies cleanly on top of the latest drm-next branch. Is
> > > > there a specific need to rebase and resend ?
> > >
> > > Nope. After my patch series there were some patches from Geert for
> > > drm/shmobile merged to drm-misc-next by Thomas.
> > >
> > > Maybe git managed this automatically??
> >
> > Probably, git is nice :-)
> >
> > > > I haven't had time to review patch 4/5 (the driver) yet. All the
> > > > rest looks good to me. Should I already include 1/5 in my next pull
> > request ?
> > >
> > > Yes, please.
> >
> > OK, I will do so. I've reviewed 4/5 in the meantime, but changes are
> > needed, so I won't wait for v10 before applying 1/5.
>
> I have incorporated review comments for v9. I need to rebase my changes.
>
> Is the pull request being done to drm-misc-next?
I've just sent the pull request for drm-next, and have CC'ed you.
> > > > > Please let me know your preference.
> > > > >
> > > > > Cheers,
> > > > > Biju
> > > > >
> > > > >
> > > > > > -Original Message-
> > > > > > From: Biju Das
> > > > > > Sent: Monday, May 15, 2023 8:58 AM
> > > > > > To: David Airlie ; Daniel Vetter
> > > > > > ; Philipp Zabel ; Geert
> > > > > > Uytterhoeven ; Laurent Pinchart
> > > > > > ; Kieran Bingham
> > > > > >
> > > > > > Cc: dri-devel@lists.freedesktop.org;
> > > > > > linux-renesas-...@vger.kernel.org;
> > > > > > Fabrizio Castro ; Prabhakar
> > > > > > Mahadev Lad
> > > > > > Subject: RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > > > > Display Unit support
> > > > > >
> > > > > > Hi All,
> > > > > >
> > > > > > Gentle ping. Are we happy with this patch series?
> > > > > >
> > > > > > Cheers,
> > > > > > Biju
> > > > > >
> > > > > > > Subject: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > > > > > Display Unit support
> > > > > > >
> > > > > > > RZ/G2L LCD controller composed of Frame compression
> > > > > > > Processor(FCPVD), Video signal processor (VSPD) and Display
> > > > > > > unit(DU). The output of LCDC is connected to Display parallel
> > > > > > > interface and MIPI link video interface.
> > > > > > >
> > > > > > > The output from DSI is connected to ADV7535.
> > > > > > >
> > > > > > > Created a vendor specific directory renesas and moved all
> > > > > > > renesas drm drivers to it (rcar-du and shmobile). Then added
> > > > > > > support for RZ/G2L DU DRM driver by creating rz_du directory.
> > > > > > >
> > > > > > > Ref:
> > > > > > >
> > > > > > >
> > > > > > > v8->v9:
> > > > > > > * Added Rb tag from Laurent and Acked-by tag from Kieran for
> > > > patch#1.
> > > > > > > * Added Rb tag from Laurent and Geert for patch#3.
> > > > > > > * Dropped reset_control_assert() from error patch for
> > > > > > > rzg2l_du_crtc_get() as
> > &
RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display Unit support
Hi Laurent,
Thanks for the feedback.
> Subject: Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display
> Unit support
>
> On Mon, May 29, 2023 at 02:22:06PM +, Biju Das wrote:
> > HI Laurent,
> >
> > Thanks for the feedback.
> >
> > > Subject: Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > Display Unit support
> > >
> > > Hi Biju,
> > >
> > > On Thu, May 25, 2023 at 02:30:10PM +, Biju Das wrote:
> > > > Hi DRM maintainers,
> > > >
> > > > Gentle ping.
> > >
> > > Sorry, I was on holidays the last two weeks.
> > >
> > > > Are we happy with moving all Renesas drm drivers to Renesas
> > > > specific directory or preference is for separate one??
> > >
> > > This works for me.
> > >
> > > > If it is later, I can send RZ/G2L drm driver separate.
> > > >
> > > > Otherwise, I need to rebase and resend.
> > >
> > > Your series applies cleanly on top of the latest drm-next branch. Is
> > > there a specific need to rebase and resend ?
> >
> > Nope. After my patch series there were some patches from Geert for
> > drm/shmobile merged to drm-misc-next by Thomas.
> >
> > Maybe git managed this automatically??
>
> Probably, git is nice :-)
>
> > > I haven't had time to review patch 4/5 (the driver) yet. All the
> > > rest looks good to me. Should I already include 1/5 in my next pull
> request ?
> >
> > Yes, please.
>
> OK, I will do so. I've reviewed 4/5 in the meantime, but changes are
> needed, so I won't wait for v10 before applying 1/5.
>
I have incorporated review comments for v9. I need to rebase my changes.
Is the pull request being done to drm-misc-next?
Cheers,
Biju
> > > > Please let me know your preference.
> > > >
> > > > Cheers,
> > > > Biju
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: Biju Das
> > > > > Sent: Monday, May 15, 2023 8:58 AM
> > > > > To: David Airlie ; Daniel Vetter
> > > > > ; Philipp Zabel ; Geert
> > > > > Uytterhoeven ; Laurent Pinchart
> > > > > ; Kieran Bingham
> > > > >
> > > > > Cc: dri-devel@lists.freedesktop.org;
> > > > > linux-renesas-...@vger.kernel.org;
> > > > > Fabrizio Castro ; Prabhakar
> > > > > Mahadev Lad
> > > > > Subject: RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > > > Display Unit support
> > > > >
> > > > > Hi All,
> > > > >
> > > > > Gentle ping. Are we happy with this patch series?
> > > > >
> > > > > Cheers,
> > > > > Biju
> > > > >
> > > > > > Subject: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > > > > Display Unit support
> > > > > >
> > > > > > RZ/G2L LCD controller composed of Frame compression
> > > > > > Processor(FCPVD), Video signal processor (VSPD) and Display
> > > > > > unit(DU). The output of LCDC is connected to Display parallel
> > > > > > interface and MIPI link video interface.
> > > > > >
> > > > > > The output from DSI is connected to ADV7535.
> > > > > >
> > > > > > Created a vendor specific directory renesas and moved all
> > > > > > renesas drm drivers to it (rcar-du and shmobile). Then added
> > > > > > support for RZ/G2L DU DRM driver by creating rz_du directory.
> > > > > >
> > > > > > Ref:
> > > > > >
> > > > > >
> > > > > > v8->v9:
> > > > > > * Added Rb tag from Laurent and Acked-by tag from Kieran for
> > > patch#1.
> > > > > > * Added Rb tag from Laurent and Geert for patch#3.
> > > > > > * Dropped reset_control_assert() from error patch for
> > > > > > rzg2l_du_crtc_get() as
> > > > > >suggested by Philipp Zabel.
> > > > > > * Added Rb tag from Laurent oatch#5.
> > > > > > * Updated MAINTAINERS entries for common parts(Makefile and
> > > Kconfig).
> > > > > > v7->v8:
> > > > > > * Moved rcar-du and shmobile DRM drivers to renesas specific
> > > > > > vendor directory.
&
Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display Unit support
On Mon, May 29, 2023 at 02:22:06PM +, Biju Das wrote:
> HI Laurent,
>
> Thanks for the feedback.
>
> > Subject: Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display
> > Unit support
> >
> > Hi Biju,
> >
> > On Thu, May 25, 2023 at 02:30:10PM +, Biju Das wrote:
> > > Hi DRM maintainers,
> > >
> > > Gentle ping.
> >
> > Sorry, I was on holidays the last two weeks.
> >
> > > Are we happy with moving all Renesas drm drivers to Renesas specific
> > > directory or preference is for separate one??
> >
> > This works for me.
> >
> > > If it is later, I can send RZ/G2L drm driver separate.
> > >
> > > Otherwise, I need to rebase and resend.
> >
> > Your series applies cleanly on top of the latest drm-next branch. Is
> > there a specific need to rebase and resend ?
>
> Nope. After my patch series there were
> some patches from Geert for drm/shmobile merged to drm-misc-next by Thomas.
>
> Maybe git managed this automatically??
Probably, git is nice :-)
> > I haven't had time to review patch 4/5 (the driver) yet. All the rest
> > looks good to me. Should I already include 1/5 in my next pull request ?
>
> Yes, please.
OK, I will do so. I've reviewed 4/5 in the meantime, but changes are
needed, so I won't wait for v10 before applying 1/5.
> > > Please let me know your preference.
> > >
> > > Cheers,
> > > Biju
> > >
> > >
> > > > -Original Message-
> > > > From: Biju Das
> > > > Sent: Monday, May 15, 2023 8:58 AM
> > > > To: David Airlie ; Daniel Vetter
> > > > ; Philipp Zabel ; Geert
> > > > Uytterhoeven ; Laurent Pinchart
> > > > ; Kieran Bingham
> > > >
> > > > Cc: dri-devel@lists.freedesktop.org;
> > > > linux-renesas-...@vger.kernel.org;
> > > > Fabrizio Castro ; Prabhakar Mahadev
> > > > Lad
> > > > Subject: RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > > Display Unit support
> > > >
> > > > Hi All,
> > > >
> > > > Gentle ping. Are we happy with this patch series?
> > > >
> > > > Cheers,
> > > > Biju
> > > >
> > > > > Subject: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > > > Display Unit support
> > > > >
> > > > > RZ/G2L LCD controller composed of Frame compression
> > > > > Processor(FCPVD), Video signal processor (VSPD) and Display
> > > > > unit(DU). The output of LCDC is connected to Display parallel
> > > > > interface and MIPI link video interface.
> > > > >
> > > > > The output from DSI is connected to ADV7535.
> > > > >
> > > > > Created a vendor specific directory renesas and moved all renesas
> > > > > drm drivers to it (rcar-du and shmobile). Then added support for
> > > > > RZ/G2L DU DRM driver by creating rz_du directory.
> > > > >
> > > > > Ref:
> > > > >
> > > > >
> > > > > v8->v9:
> > > > > * Added Rb tag from Laurent and Acked-by tag from Kieran for
> > patch#1.
> > > > > * Added Rb tag from Laurent and Geert for patch#3.
> > > > > * Dropped reset_control_assert() from error patch for
> > > > > rzg2l_du_crtc_get() as
> > > > >suggested by Philipp Zabel.
> > > > > * Added Rb tag from Laurent oatch#5.
> > > > > * Updated MAINTAINERS entries for common parts(Makefile and
> > Kconfig).
> > > > > v7->v8:
> > > > > * Moved rcar-du and shmobile DRM drivers to renesas specific
> > > > > vendor directory.
> > > > > * Fixed the typo vsp2->du in RZ/V2L DU bindings patch.
> > > > > * Added Rb tag from Rob for RZ/V2L DU bindings patch.
> > > > > * Dropped RCar du lib and created RZ/G2L DU DRM driver by
> > > > > creating rz_du folder.
> > > > > * Updated MAINTAINERS entries.
> > > > > v6->v7:
> > > > > * Split DU lib and RZ/G2L du driver as separate patch series as
> > > > >DU support added to more platforms based on RZ/G2L alike SoCs.
> > > > > * Rebased to latest drm-tip.
> > > > > * Added patch #2 for binding support for RZ/V2L DU
> > > > > * Added patch #4 for driver support for RZ/V2L DU
>
RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display Unit support
HI Laurent,
Thanks for the feedback.
> Subject: Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display
> Unit support
>
> Hi Biju,
>
> On Thu, May 25, 2023 at 02:30:10PM +, Biju Das wrote:
> > Hi DRM maintainers,
> >
> > Gentle ping.
>
> Sorry, I was on holidays the last two weeks.
>
> > Are we happy with moving all Renesas drm drivers to Renesas specific
> > directory or preference is for separate one??
>
> This works for me.
>
> > If it is later, I can send RZ/G2L drm driver separate.
> >
> > Otherwise, I need to rebase and resend.
>
> Your series applies cleanly on top of the latest drm-next branch. Is
> there a specific need to rebase and resend ?
Nope. After my patch series there were
some patches from Geert for drm/shmobile merged to drm-misc-next by Thomas.
Maybe git managed this automatically??
>
> I haven't had time to review patch 4/5 (the driver) yet. All the rest
> looks good to me. Should I already include 1/5 in my next pull request ?
Yes, please.
Cheers,
Biju
>
> > Please let me know your preference.
> >
> > Cheers,
> > Biju
> >
> >
> > > -Original Message-
> > > From: Biju Das
> > > Sent: Monday, May 15, 2023 8:58 AM
> > > To: David Airlie ; Daniel Vetter
> > > ; Philipp Zabel ; Geert
> > > Uytterhoeven ; Laurent Pinchart
> > > ; Kieran Bingham
> > >
> > > Cc: dri-devel@lists.freedesktop.org;
> > > linux-renesas-...@vger.kernel.org;
> > > Fabrizio Castro ; Prabhakar Mahadev
> > > Lad
> > > Subject: RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > Display Unit support
> > >
> > > Hi All,
> > >
> > > Gentle ping. Are we happy with this patch series?
> > >
> > > Cheers,
> > > Biju
> > >
> > > > Subject: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L
> > > > Display Unit support
> > > >
> > > > RZ/G2L LCD controller composed of Frame compression
> > > > Processor(FCPVD), Video signal processor (VSPD) and Display
> > > > unit(DU). The output of LCDC is connected to Display parallel
> > > > interface and MIPI link video interface.
> > > >
> > > > The output from DSI is connected to ADV7535.
> > > >
> > > > Created a vendor specific directory renesas and moved all renesas
> > > > drm drivers to it (rcar-du and shmobile). Then added support for
> > > > RZ/G2L DU DRM driver by creating rz_du directory.
> > > >
> > > > Ref:
> > > >
> > > >
> > > > v8->v9:
> > > > * Added Rb tag from Laurent and Acked-by tag from Kieran for
> patch#1.
> > > > * Added Rb tag from Laurent and Geert for patch#3.
> > > > * Dropped reset_control_assert() from error patch for
> > > > rzg2l_du_crtc_get() as
> > > >suggested by Philipp Zabel.
> > > > * Added Rb tag from Laurent oatch#5.
> > > > * Updated MAINTAINERS entries for common parts(Makefile and
> Kconfig).
> > > > v7->v8:
> > > > * Moved rcar-du and shmobile DRM drivers to renesas specific
> > > > vendor directory.
> > > > * Fixed the typo vsp2->du in RZ/V2L DU bindings patch.
> > > > * Added Rb tag from Rob for RZ/V2L DU bindings patch.
> > > > * Dropped RCar du lib and created RZ/G2L DU DRM driver by
> > > > creating rz_du folder.
> > > > * Updated MAINTAINERS entries.
> > > > v6->v7:
> > > > * Split DU lib and RZ/G2L du driver as separate patch series as
> > > >DU support added to more platforms based on RZ/G2L alike SoCs.
> > > > * Rebased to latest drm-tip.
> > > > * Added patch #2 for binding support for RZ/V2L DU
> > > > * Added patch #4 for driver support for RZ/V2L DU
> > > > * Added patch #5 for SoC DTSI support for RZ/G2L DU
> > > > * Added patch #6 for SoC DTSI support for RZ/V2L DU
> > > > * Added patch #7 for Enabling DU on SMARC EVK based on
> > > > RZ/{G2L,V2L} SoCs.
> > > > * Added patch #8 for Enabling DU on SMARC EVK based on RZ/G2LC
> SoC.
> > > > v5->v6:
> > > > * Merged DU lib and RZ/G2L du driver in same patch series
> > > > * Rebased to latest drm-misc.
> > > > * Merged patch#1 to RZ/G2L Driver patch.
> > > > * Updated KConfig dependency from ARCH_RENESAS->ARCH_RZG2L.
> > > > * Optimized rzg2l_du_output_nam
Re: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display Unit support
Hi Biju,
On Thu, May 25, 2023 at 02:30:10PM +, Biju Das wrote:
> Hi DRM maintainers,
>
> Gentle ping.
Sorry, I was on holidays the last two weeks.
> Are we happy with moving all Renesas drm drivers to Renesas specific
> directory or preference is for separate one??
This works for me.
> If it is later, I can send RZ/G2L drm driver separate.
>
> Otherwise, I need to rebase and resend.
Your series applies cleanly on top of the latest drm-next branch. Is
there a specific need to rebase and resend ?
I haven't had time to review patch 4/5 (the driver) yet. All the rest
looks good to me. Should I already include 1/5 in my next pull request ?
> Please let me know your preference.
>
> Cheers,
> Biju
>
>
> > -Original Message-
> > From: Biju Das
> > Sent: Monday, May 15, 2023 8:58 AM
> > To: David Airlie ; Daniel Vetter ;
> > Philipp Zabel ; Geert Uytterhoeven
> > ; Laurent Pinchart
> > ; Kieran Bingham
> >
> > Cc: dri-devel@lists.freedesktop.org; linux-renesas-...@vger.kernel.org;
> > Fabrizio Castro ; Prabhakar Mahadev Lad
> >
> > Subject: RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display
> > Unit support
> >
> > Hi All,
> >
> > Gentle ping. Are we happy with this patch series?
> >
> > Cheers,
> > Biju
> >
> > > Subject: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display
> > > Unit support
> > >
> > > RZ/G2L LCD controller composed of Frame compression Processor(FCPVD),
> > > Video signal processor (VSPD) and Display unit(DU). The output of LCDC
> > > is connected to Display parallel interface and MIPI link video
> > > interface.
> > >
> > > The output from DSI is connected to ADV7535.
> > >
> > > Created a vendor specific directory renesas and moved all renesas drm
> > > drivers to it (rcar-du and shmobile). Then added support for RZ/G2L DU
> > > DRM driver by creating rz_du directory.
> > >
> > > Ref:
> > >
> > >
> > > v8->v9:
> > > * Added Rb tag from Laurent and Acked-by tag from Kieran for patch#1.
> > > * Added Rb tag from Laurent and Geert for patch#3.
> > > * Dropped reset_control_assert() from error patch for
> > > rzg2l_du_crtc_get() as
> > >suggested by Philipp Zabel.
> > > * Added Rb tag from Laurent oatch#5.
> > > * Updated MAINTAINERS entries for common parts(Makefile and Kconfig).
> > > v7->v8:
> > > * Moved rcar-du and shmobile DRM drivers to renesas specific vendor
> > > directory.
> > > * Fixed the typo vsp2->du in RZ/V2L DU bindings patch.
> > > * Added Rb tag from Rob for RZ/V2L DU bindings patch.
> > > * Dropped RCar du lib and created RZ/G2L DU DRM driver by creating
> > > rz_du folder.
> > > * Updated MAINTAINERS entries.
> > > v6->v7:
> > > * Split DU lib and RZ/G2L du driver as separate patch series as
> > >DU support added to more platforms based on RZ/G2L alike SoCs.
> > > * Rebased to latest drm-tip.
> > > * Added patch #2 for binding support for RZ/V2L DU
> > > * Added patch #4 for driver support for RZ/V2L DU
> > > * Added patch #5 for SoC DTSI support for RZ/G2L DU
> > > * Added patch #6 for SoC DTSI support for RZ/V2L DU
> > > * Added patch #7 for Enabling DU on SMARC EVK based on RZ/{G2L,V2L}
> > > SoCs.
> > > * Added patch #8 for Enabling DU on SMARC EVK based on RZ/G2LC SoC.
> > > v5->v6:
> > > * Merged DU lib and RZ/G2L du driver in same patch series
> > > * Rebased to latest drm-misc.
> > > * Merged patch#1 to RZ/G2L Driver patch.
> > > * Updated KConfig dependency from ARCH_RENESAS->ARCH_RZG2L.
> > > * Optimized rzg2l_du_output_name() by removing unsupported outputs.
> > >
> > > v4->v5:
> > > * Added Rb tag from Rob for binding patch.
> > > * Started using RCar DU libs(kms, vsp and encoder)
> > > * Started using rcar_du_device, rcar_du_write, rcar_du_crtc,
> > >rcar_du_format_info and rcar_du_encoder.
> > > v3->v4:
> > > * Changed compatible name from
> > > renesas,du-r9a07g044->renesas,r9a07g044-
> > > du
> > > * started using same compatible for RZ/G2{L,LC}
> > > * Removed rzg2l_du_group.h and struct rzg2l_du_group
> > > * Renamed __rzg2l_du_group_start_stop->rzg2l_du_start_stop
> > > * Removed rzg2l_du_group_restart
> > > * Updated rzg2l_du_crtc_set_display_timing
> > > * Removed mode_valid call
RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display Unit support
Hi DRM maintainers,
Gentle ping.
Are we happy with moving all Renesas drm drivers to Renesas specific
directory or preference is for separate one??
If it is later, I can send RZ/G2L drm driver separate.
Otherwise, I need to rebase and resend.
Please let me know your preference.
Cheers,
Biju
> -Original Message-
> From: Biju Das
> Sent: Monday, May 15, 2023 8:58 AM
> To: David Airlie ; Daniel Vetter ;
> Philipp Zabel ; Geert Uytterhoeven
> ; Laurent Pinchart
> ; Kieran Bingham
>
> Cc: dri-devel@lists.freedesktop.org; linux-renesas-...@vger.kernel.org;
> Fabrizio Castro ; Prabhakar Mahadev Lad
>
> Subject: RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display
> Unit support
>
> Hi All,
>
> Gentle ping. Are we happy with this patch series?
>
> Cheers,
> Biju
>
> > Subject: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display
> > Unit support
> >
> > RZ/G2L LCD controller composed of Frame compression Processor(FCPVD),
> > Video signal processor (VSPD) and Display unit(DU). The output of LCDC
> > is connected to Display parallel interface and MIPI link video
> > interface.
> >
> > The output from DSI is connected to ADV7535.
> >
> > Created a vendor specific directory renesas and moved all renesas drm
> > drivers to it (rcar-du and shmobile). Then added support for RZ/G2L DU
> > DRM driver by creating rz_du directory.
> >
> > Ref:
> >
> >
> > v8->v9:
> > * Added Rb tag from Laurent and Acked-by tag from Kieran for patch#1.
> > * Added Rb tag from Laurent and Geert for patch#3.
> > * Dropped reset_control_assert() from error patch for
> > rzg2l_du_crtc_get() as
> >suggested by Philipp Zabel.
> > * Added Rb tag from Laurent oatch#5.
> > * Updated MAINTAINERS entries for common parts(Makefile and Kconfig).
> > v7->v8:
> > * Moved rcar-du and shmobile DRM drivers to renesas specific vendor
> > directory.
> > * Fixed the typo vsp2->du in RZ/V2L DU bindings patch.
> > * Added Rb tag from Rob for RZ/V2L DU bindings patch.
> > * Dropped RCar du lib and created RZ/G2L DU DRM driver by creating
> > rz_du folder.
> > * Updated MAINTAINERS entries.
> > v6->v7:
> > * Split DU lib and RZ/G2L du driver as separate patch series as
> >DU support added to more platforms based on RZ/G2L alike SoCs.
> > * Rebased to latest drm-tip.
> > * Added patch #2 for binding support for RZ/V2L DU
> > * Added patch #4 for driver support for RZ/V2L DU
> > * Added patch #5 for SoC DTSI support for RZ/G2L DU
> > * Added patch #6 for SoC DTSI support for RZ/V2L DU
> > * Added patch #7 for Enabling DU on SMARC EVK based on RZ/{G2L,V2L}
> > SoCs.
> > * Added patch #8 for Enabling DU on SMARC EVK based on RZ/G2LC SoC.
> > v5->v6:
> > * Merged DU lib and RZ/G2L du driver in same patch series
> > * Rebased to latest drm-misc.
> > * Merged patch#1 to RZ/G2L Driver patch.
> > * Updated KConfig dependency from ARCH_RENESAS->ARCH_RZG2L.
> > * Optimized rzg2l_du_output_name() by removing unsupported outputs.
> >
> > v4->v5:
> > * Added Rb tag from Rob for binding patch.
> > * Started using RCar DU libs(kms, vsp and encoder)
> > * Started using rcar_du_device, rcar_du_write, rcar_du_crtc,
> >rcar_du_format_info and rcar_du_encoder.
> > v3->v4:
> > * Changed compatible name from
> > renesas,du-r9a07g044->renesas,r9a07g044-
> > du
> > * started using same compatible for RZ/G2{L,LC}
> > * Removed rzg2l_du_group.h and struct rzg2l_du_group
> > * Renamed __rzg2l_du_group_start_stop->rzg2l_du_start_stop
> > * Removed rzg2l_du_group_restart
> > * Updated rzg2l_du_crtc_set_display_timing
> > * Removed mode_valid callback.
> > * Updated rzg2l_du_crtc_create() parameters
> > * Updated compatible
> > * Removed RZG2L_DU_MAX_GROUPS
> > V2->v3:
> > * Added new bindings for RZ/G2L DU
> > * Removed indirection and created new DRM driver based on R-Car DU
> > v1->v2:
> > * Based on [1], all references to 'rzg2l_lcdc' replaced with
> 'rzg2l_du'
> > * Updated commit description for bindings
> > * Removed LCDC references from bindings
> > * Changed clock name from du.0->aclk from bindings
> > * Changed reset name from du.0->du from bindings
> > * Replaced crtc_helper_funcs->rcar_crtc_helper_funcs
> > * Updated macro DRM_RZG2L_LCDC->DRM_RZG2L_DU
> > * Replaced rzg2l-lcdc-drm->rzg2l-du-drm
> > * Added forward declaration for struct reset_control
> >
> > [1]
> &g
RE: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display Unit support
Hi All,
Gentle ping. Are we happy with this patch series?
Cheers,
Biju
> Subject: [PATCH v9 RESEND 0/5] Add RZ/{G2L,G2LC} and RZ/V2L Display Unit
> support
>
> RZ/G2L LCD controller composed of Frame compression Processor(FCPVD),
> Video signal processor (VSPD) and Display unit(DU). The output of LCDC
> is connected to Display parallel interface and MIPI link video
> interface.
>
> The output from DSI is connected to ADV7535.
>
> Created a vendor specific directory renesas and moved all renesas drm
> drivers to it (rcar-du and shmobile). Then added support for RZ/G2L DU
> DRM driver by creating rz_du directory.
>
> Ref:
>
>
> v8->v9:
> * Added Rb tag from Laurent and Acked-by tag from Kieran for patch#1.
> * Added Rb tag from Laurent and Geert for patch#3.
> * Dropped reset_control_assert() from error patch for
> rzg2l_du_crtc_get() as
>suggested by Philipp Zabel.
> * Added Rb tag from Laurent oatch#5.
> * Updated MAINTAINERS entries for common parts(Makefile and Kconfig).
> v7->v8:
> * Moved rcar-du and shmobile DRM drivers to renesas specific vendor
> directory.
> * Fixed the typo vsp2->du in RZ/V2L DU bindings patch.
> * Added Rb tag from Rob for RZ/V2L DU bindings patch.
> * Dropped RCar du lib and created RZ/G2L DU DRM driver by creating
> rz_du folder.
> * Updated MAINTAINERS entries.
> v6->v7:
> * Split DU lib and RZ/G2L du driver as separate patch series as
>DU support added to more platforms based on RZ/G2L alike SoCs.
> * Rebased to latest drm-tip.
> * Added patch #2 for binding support for RZ/V2L DU
> * Added patch #4 for driver support for RZ/V2L DU
> * Added patch #5 for SoC DTSI support for RZ/G2L DU
> * Added patch #6 for SoC DTSI support for RZ/V2L DU
> * Added patch #7 for Enabling DU on SMARC EVK based on RZ/{G2L,V2L}
> SoCs.
> * Added patch #8 for Enabling DU on SMARC EVK based on RZ/G2LC SoC.
> v5->v6:
> * Merged DU lib and RZ/G2L du driver in same patch series
> * Rebased to latest drm-misc.
> * Merged patch#1 to RZ/G2L Driver patch.
> * Updated KConfig dependency from ARCH_RENESAS->ARCH_RZG2L.
> * Optimized rzg2l_du_output_name() by removing unsupported outputs.
>
> v4->v5:
> * Added Rb tag from Rob for binding patch.
> * Started using RCar DU libs(kms, vsp and encoder)
> * Started using rcar_du_device, rcar_du_write, rcar_du_crtc,
>rcar_du_format_info and rcar_du_encoder.
> v3->v4:
> * Changed compatible name from renesas,du-r9a07g044->renesas,r9a07g044-
> du
> * started using same compatible for RZ/G2{L,LC}
> * Removed rzg2l_du_group.h and struct rzg2l_du_group
> * Renamed __rzg2l_du_group_start_stop->rzg2l_du_start_stop
> * Removed rzg2l_du_group_restart
> * Updated rzg2l_du_crtc_set_display_timing
> * Removed mode_valid callback.
> * Updated rzg2l_du_crtc_create() parameters
> * Updated compatible
> * Removed RZG2L_DU_MAX_GROUPS
> V2->v3:
> * Added new bindings for RZ/G2L DU
> * Removed indirection and created new DRM driver based on R-Car DU
> v1->v2:
> * Based on [1], all references to 'rzg2l_lcdc' replaced with 'rzg2l_du'
> * Updated commit description for bindings
> * Removed LCDC references from bindings
> * Changed clock name from du.0->aclk from bindings
> * Changed reset name from du.0->du from bindings
> * Replaced crtc_helper_funcs->rcar_crtc_helper_funcs
> * Updated macro DRM_RZG2L_LCDC->DRM_RZG2L_DU
> * Replaced rzg2l-lcdc-drm->rzg2l-du-drm
> * Added forward declaration for struct reset_control
>
> [1]
>
> Biju Das (5):
> drm: Place Renesas drivers in a separate dir
> dt-bindings: display: Document Renesas RZ/G2L DU bindings
> dt-bindings: display: renesas,rzg2l-du: Document RZ/V2L DU bindings
> drm: Add RZ/G2L DU Support
> MAINTAINERS: Add maintainer for RZ DU drivers
>
> .../bindings/display/renesas,rzg2l-du.yaml| 129 +++
> MAINTAINERS | 16 +-
> drivers/gpu/drm/Kconfig | 4 +-
> drivers/gpu/drm/Makefile | 3 +-
> drivers/gpu/drm/renesas/Kconfig | 5 +
> drivers/gpu/drm/renesas/Makefile | 5 +
> drivers/gpu/drm/{ => renesas}/rcar-du/Kconfig | 0
> .../gpu/drm/{ => renesas}/rcar-du/Makefile| 0
> .../gpu/drm/{ => renesas}/rcar-du/rcar_cmm.c | 0
> .../gpu/drm/{ => renesas}/rcar-du/rcar_cmm.h | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_crtc.c | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_crtc.h | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_drv.c | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_drv.h | 0
> .../{ => renesas}/rcar-du/rcar_du_encoder.c | 0
> .../{ => renesas}/rcar-du/rcar_du_encoder.h | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_group.c | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_group.h | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_kms.c | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_kms.h | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_plane.c | 0
> .../drm/{ => renesas}/rcar-du/rcar_du_plane.h | 0
> .../drm/{
