| Message ID | 1526386840-15368-3-git-send-email-ulrich.hecht+renesas@gmail.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
On Tue, May 15, 2018 at 02:20:37PM +0200, Ulrich Hecht wrote: > Add support for the R-Car D3 (R8A77995) SoC to the LVDS encoder driver. > > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com> > --- > drivers/gpu/drm/rcar-du/rcar_lvds.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c > index 3d2d3bb..58fb9f8 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c > +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c > @@ -511,6 +511,11 @@ static const struct rcar_lvds_device_info rcar_lvds_r8a77970_info = { > .quirks = RCAR_LVDS_QUIRK_GEN2_PLLCR | RCAR_LVDS_QUIRK_GEN3_LVEN, > }; > > +static const struct rcar_lvds_device_info rcar_lvds_r8a77995_info = { > + .gen = 3, > + .quirks = RCAR_LVDS_QUIRK_GEN3_LVEN, > +}; This new structure seems to be identical to rcar_lvds_r8a77970_info. Could we consolidate somehow? > + > static const struct of_device_id rcar_lvds_of_table[] = { > { .compatible = "renesas,r8a7743-lvds", .data = &rcar_lvds_gen2_info }, > { .compatible = "renesas,r8a7790-lvds", .data = &rcar_lvds_r8a7790_info }, > @@ -519,6 +524,7 @@ static const struct of_device_id rcar_lvds_of_table[] = { > { .compatible = "renesas,r8a7795-lvds", .data = &rcar_lvds_gen3_info }, > { .compatible = "renesas,r8a7796-lvds", .data = &rcar_lvds_gen3_info }, > { .compatible = "renesas,r8a77970-lvds", .data = &rcar_lvds_r8a77970_info }, > + { .compatible = "renesas,r8a77995-lvds", .data = &rcar_lvds_r8a77995_info }, > { } > }; > > -- > 2.7.4 >
Hello! On 5/16/2018 10:54 AM, Simon Horman wrote: >> Add support for the R-Car D3 (R8A77995) SoC to the LVDS encoder driver. >> >> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com> >> --- >> drivers/gpu/drm/rcar-du/rcar_lvds.c | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c >> index 3d2d3bb..58fb9f8 100644 >> --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c >> +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c >> @@ -511,6 +511,11 @@ static const struct rcar_lvds_device_info rcar_lvds_r8a77970_info = { >> .quirks = RCAR_LVDS_QUIRK_GEN2_PLLCR | RCAR_LVDS_QUIRK_GEN3_LVEN, >> }; >> >> +static const struct rcar_lvds_device_info rcar_lvds_r8a77995_info = { >> + .gen = 3, >> + .quirks = RCAR_LVDS_QUIRK_GEN3_LVEN, >> +}; > > This new structure seems to be identical to rcar_lvds_r8a77970_info. > Could we consolidate somehow? In my book, that one has 2 quirk flags. What tree are you looking at? [...] MBR, Sergei
On Wed, May 16, 2018 at 10:59 AM, Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> wrote: > Hello! > > On 5/16/2018 10:54 AM, Simon Horman wrote: > >>> Add support for the R-Car D3 (R8A77995) SoC to the LVDS encoder driver. >>> >>> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com> >>> --- >>> drivers/gpu/drm/rcar-du/rcar_lvds.c | 6 ++++++ >>> 1 file changed, 6 insertions(+) >>> >>> diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c >>> b/drivers/gpu/drm/rcar-du/rcar_lvds.c >>> index 3d2d3bb..58fb9f8 100644 >>> --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c >>> +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c >>> @@ -511,6 +511,11 @@ static const struct rcar_lvds_device_info >>> rcar_lvds_r8a77970_info = { >>> .quirks = RCAR_LVDS_QUIRK_GEN2_PLLCR | RCAR_LVDS_QUIRK_GEN3_LVEN, >>> }; >>> +static const struct rcar_lvds_device_info rcar_lvds_r8a77995_info = { >>> + .gen = 3, >>> + .quirks = RCAR_LVDS_QUIRK_GEN3_LVEN, >>> +}; >> >> >> This new structure seems to be identical to rcar_lvds_r8a77970_info. > >> Could we consolidate somehow? > > In my book, that one has 2 quirk flags. What tree are you looking at? IIRC the other one was for the PLL, which is not supported yet. CU Uli
On 05/16/2018 04:06 PM, Ulrich Hecht wrote: >>>> Add support for the R-Car D3 (R8A77995) SoC to the LVDS encoder driver. >>>> >>>> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com> >>>> --- >>>> drivers/gpu/drm/rcar-du/rcar_lvds.c | 6 ++++++ >>>> 1 file changed, 6 insertions(+) >>>> >>>> diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c >>>> b/drivers/gpu/drm/rcar-du/rcar_lvds.c >>>> index 3d2d3bb..58fb9f8 100644 >>>> --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c >>>> +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c >>>> @@ -511,6 +511,11 @@ static const struct rcar_lvds_device_info >>>> rcar_lvds_r8a77970_info = { >>>> .quirks = RCAR_LVDS_QUIRK_GEN2_PLLCR | RCAR_LVDS_QUIRK_GEN3_LVEN, >>>> }; >>>> +static const struct rcar_lvds_device_info rcar_lvds_r8a77995_info = { >>>> + .gen = 3, >>>> + .quirks = RCAR_LVDS_QUIRK_GEN3_LVEN, >>>> +}; >>> >>> >>> This new structure seems to be identical to rcar_lvds_r8a77970_info. >> >>> Could we consolidate somehow? >> >> In my book, that one has 2 quirk flags. What tree are you looking at? > > IIRC the other one was for the PLL, which is not supported yet. V3M PLL is surely supported and it's surely different from D3 one. My point was that r8a77970 data can't be reused... > CU > Uli MBR, Sergei
On Wed, May 16, 2018 at 11:59:32AM +0300, Sergei Shtylyov wrote: > Hello! > > On 5/16/2018 10:54 AM, Simon Horman wrote: > > > > Add support for the R-Car D3 (R8A77995) SoC to the LVDS encoder driver. > > > > > > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com> > > > --- > > > drivers/gpu/drm/rcar-du/rcar_lvds.c | 6 ++++++ > > > 1 file changed, 6 insertions(+) > > > > > > diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c > > > index 3d2d3bb..58fb9f8 100644 > > > --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c > > > +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c > > > @@ -511,6 +511,11 @@ static const struct rcar_lvds_device_info rcar_lvds_r8a77970_info = { > > > .quirks = RCAR_LVDS_QUIRK_GEN2_PLLCR | RCAR_LVDS_QUIRK_GEN3_LVEN, > > > }; > > > +static const struct rcar_lvds_device_info rcar_lvds_r8a77995_info = { > > > + .gen = 3, > > > + .quirks = RCAR_LVDS_QUIRK_GEN3_LVEN, > > > +}; > > > > This new structure seems to be identical to rcar_lvds_r8a77970_info. > > Could we consolidate somehow? > > In my book, that one has 2 quirk flags. What tree are you looking at? True, my eyes seem to have deceived me.
Hi Ulrich, Thank you for the patch. On Tuesday, 15 May 2018 15:20:37 EEST Ulrich Hecht wrote: > Add support for the R-Car D3 (R8A77995) SoC to the LVDS encoder driver. > > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com> > --- > drivers/gpu/drm/rcar-du/rcar_lvds.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c > b/drivers/gpu/drm/rcar-du/rcar_lvds.c index 3d2d3bb..58fb9f8 100644 > --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c > +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c > @@ -511,6 +511,11 @@ static const struct rcar_lvds_device_info > rcar_lvds_r8a77970_info = { .quirks = RCAR_LVDS_QUIRK_GEN2_PLLCR | > RCAR_LVDS_QUIRK_GEN3_LVEN, }; > > +static const struct rcar_lvds_device_info rcar_lvds_r8a77995_info = { > + .gen = 3, > + .quirks = RCAR_LVDS_QUIRK_GEN3_LVEN, I'm afraid this won't be enough. With this patch the driver will call rcar_lvds_lvdpllcr_gen3(), which writes values to the LVDPLLCR register that don't match the register layout for D3. While I'm fine with an initial version that doesn't support fine-grained control of the LVDS PLL to achieve the HDMI clock accuracy requirements, we need the LVDS encoder to be at least functional for the patches to get merged. There are also other registers not related to the PLL that need to be set (such as the LVDSTRIPE register), and other differences in register layouts (for instance the D3 doesn't have a PLLON bit in register LVDC0). Even the LVEN bit seems to need special handling on D3. According to version 1.00 of the datasheet it should be set to 1 at the same time as bit LVRES. Could you please study the datasheet in details and update the code accordingly ? > +}; > + > static const struct of_device_id rcar_lvds_of_table[] = { > { .compatible = "renesas,r8a7743-lvds", .data = &rcar_lvds_gen2_info }, > { .compatible = "renesas,r8a7790-lvds", .data = &rcar_lvds_r8a7790_info }, > @@ -519,6 +524,7 @@ static const struct of_device_id rcar_lvds_of_table[] = > { { .compatible = "renesas,r8a7795-lvds", .data = &rcar_lvds_gen3_info }, { > .compatible = "renesas,r8a7796-lvds", .data = &rcar_lvds_gen3_info }, { > .compatible = "renesas,r8a77970-lvds", .data = &rcar_lvds_r8a77970_info }, > + { .compatible = "renesas,r8a77995-lvds", .data = &rcar_lvds_r8a77995_info > }, { } > };
diff --git a/drivers/gpu/drm/rcar-du/rcar_lvds.c b/drivers/gpu/drm/rcar-du/rcar_lvds.c index 3d2d3bb..58fb9f8 100644 --- a/drivers/gpu/drm/rcar-du/rcar_lvds.c +++ b/drivers/gpu/drm/rcar-du/rcar_lvds.c @@ -511,6 +511,11 @@ static const struct rcar_lvds_device_info rcar_lvds_r8a77970_info = { .quirks = RCAR_LVDS_QUIRK_GEN2_PLLCR | RCAR_LVDS_QUIRK_GEN3_LVEN, }; +static const struct rcar_lvds_device_info rcar_lvds_r8a77995_info = { + .gen = 3, + .quirks = RCAR_LVDS_QUIRK_GEN3_LVEN, +}; + static const struct of_device_id rcar_lvds_of_table[] = { { .compatible = "renesas,r8a7743-lvds", .data = &rcar_lvds_gen2_info }, { .compatible = "renesas,r8a7790-lvds", .data = &rcar_lvds_r8a7790_info }, @@ -519,6 +524,7 @@ static const struct of_device_id rcar_lvds_of_table[] = { { .compatible = "renesas,r8a7795-lvds", .data = &rcar_lvds_gen3_info }, { .compatible = "renesas,r8a7796-lvds", .data = &rcar_lvds_gen3_info }, { .compatible = "renesas,r8a77970-lvds", .data = &rcar_lvds_r8a77970_info }, + { .compatible = "renesas,r8a77995-lvds", .data = &rcar_lvds_r8a77995_info }, { } };
Add support for the R-Car D3 (R8A77995) SoC to the LVDS encoder driver. Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com> --- drivers/gpu/drm/rcar-du/rcar_lvds.c | 6 ++++++ 1 file changed, 6 insertions(+)