* [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
@ 2019-10-22 10:31 Laurent Pinchart
2019-11-05 19:49 ` Laurent Pinchart
0 siblings, 1 reply; 11+ messages in thread
From: Laurent Pinchart @ 2019-10-22 10:31 UTC (permalink / raw)
To: Dave Airlie, dri-devel
Hi Dave,
Second attempt.
The following changes since commit f1b4a9217efd61d0b84c6dc404596c8519ff6f59:
Merge tag 'du-next-20191016' of git://linuxtv.org/pinchartl/media into drm-next (2019-10-22 15:04:07 +1000)
are available in the Git repository at:
git://linuxtv.org/pinchartl/media.git tags/du-next-20191022
for you to fetch changes up to aad1552f1defd3a5334cd4b2573fae9963d4db57:
drm: rcar-du: crtc: Register GAMMA_LUT properties (2019-10-22 13:21:18 +0300)
----------------------------------------------------------------
- R-Car DU Color Management Module support
----------------------------------------------------------------
Jacopo Mondi (6):
dt-bindings: display: renesas,cmm: Add R-Car CMM documentation
dt-bindings: display, renesas,du: Document cmms property
drm: rcar-du: Add support for CMM
drm: rcar-du: kms: Initialize CMM instances
drm: rcar-du: crtc: Control CMM operations
drm: rcar-du: crtc: Register GAMMA_LUT properties
.../devicetree/bindings/display/renesas,cmm.yaml | 67 +++++++
.../devicetree/bindings/display/renesas,du.txt | 5 +
drivers/gpu/drm/rcar-du/Kconfig | 7 +
drivers/gpu/drm/rcar-du/Makefile | 1 +
drivers/gpu/drm/rcar-du/rcar_cmm.c | 217 +++++++++++++++++++++
drivers/gpu/drm/rcar-du/rcar_cmm.h | 58 ++++++
drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 71 +++++++
drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 2 +
drivers/gpu/drm/rcar-du/rcar_du_drv.h | 2 +
drivers/gpu/drm/rcar-du/rcar_du_group.c | 10 +
drivers/gpu/drm/rcar-du/rcar_du_group.h | 2 +
drivers/gpu/drm/rcar-du/rcar_du_kms.c | 76 ++++++++
drivers/gpu/drm/rcar-du/rcar_du_regs.h | 5 +
13 files changed, 523 insertions(+)
create mode 100644 Documentation/devicetree/bindings/display/renesas,cmm.yaml
create mode 100644 drivers/gpu/drm/rcar-du/rcar_cmm.c
create mode 100644 drivers/gpu/drm/rcar-du/rcar_cmm.h
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-10-22 10:31 [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support Laurent Pinchart
@ 2019-11-05 19:49 ` Laurent Pinchart
2019-11-05 19:56 ` Dave Airlie
0 siblings, 1 reply; 11+ messages in thread
From: Laurent Pinchart @ 2019-11-05 19:49 UTC (permalink / raw)
To: Dave Airlie, dri-devel
Hi Dave,
Has this pull request fallen through the cracks ?
On Tue, Oct 22, 2019 at 01:31:35PM +0300, Laurent Pinchart wrote:
> Hi Dave,
>
> Second attempt.
>
> The following changes since commit f1b4a9217efd61d0b84c6dc404596c8519ff6f59:
>
> Merge tag 'du-next-20191016' of git://linuxtv.org/pinchartl/media into drm-next (2019-10-22 15:04:07 +1000)
>
> are available in the Git repository at:
>
> git://linuxtv.org/pinchartl/media.git tags/du-next-20191022
>
> for you to fetch changes up to aad1552f1defd3a5334cd4b2573fae9963d4db57:
>
> drm: rcar-du: crtc: Register GAMMA_LUT properties (2019-10-22 13:21:18 +0300)
>
> ----------------------------------------------------------------
> - R-Car DU Color Management Module support
>
> ----------------------------------------------------------------
> Jacopo Mondi (6):
> dt-bindings: display: renesas,cmm: Add R-Car CMM documentation
> dt-bindings: display, renesas,du: Document cmms property
> drm: rcar-du: Add support for CMM
> drm: rcar-du: kms: Initialize CMM instances
> drm: rcar-du: crtc: Control CMM operations
> drm: rcar-du: crtc: Register GAMMA_LUT properties
>
> .../devicetree/bindings/display/renesas,cmm.yaml | 67 +++++++
> .../devicetree/bindings/display/renesas,du.txt | 5 +
> drivers/gpu/drm/rcar-du/Kconfig | 7 +
> drivers/gpu/drm/rcar-du/Makefile | 1 +
> drivers/gpu/drm/rcar-du/rcar_cmm.c | 217 +++++++++++++++++++++
> drivers/gpu/drm/rcar-du/rcar_cmm.h | 58 ++++++
> drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 71 +++++++
> drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 2 +
> drivers/gpu/drm/rcar-du/rcar_du_drv.h | 2 +
> drivers/gpu/drm/rcar-du/rcar_du_group.c | 10 +
> drivers/gpu/drm/rcar-du/rcar_du_group.h | 2 +
> drivers/gpu/drm/rcar-du/rcar_du_kms.c | 76 ++++++++
> drivers/gpu/drm/rcar-du/rcar_du_regs.h | 5 +
> 13 files changed, 523 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/display/renesas,cmm.yaml
> create mode 100644 drivers/gpu/drm/rcar-du/rcar_cmm.c
> create mode 100644 drivers/gpu/drm/rcar-du/rcar_cmm.h
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-11-05 19:49 ` Laurent Pinchart
@ 2019-11-05 19:56 ` Dave Airlie
2019-11-06 3:40 ` Dave Airlie
0 siblings, 1 reply; 11+ messages in thread
From: Dave Airlie @ 2019-11-05 19:56 UTC (permalink / raw)
To: Laurent Pinchart; +Cc: dri-devel
On Wed, 6 Nov 2019 at 05:49, Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
>
> Hi Dave,
>
> Has this pull request fallen through the cracks ?
It fell down a different crack than usual, it made it from patchwork
onto my hard drive, but then I didn't execute the pull.
I've pulled it down, thanks for reminder.
Dave.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-11-05 19:56 ` Dave Airlie
@ 2019-11-06 3:40 ` Dave Airlie
2019-11-06 10:00 ` Laurent Pinchart
0 siblings, 1 reply; 11+ messages in thread
From: Dave Airlie @ 2019-11-06 3:40 UTC (permalink / raw)
To: Laurent Pinchart; +Cc: dri-devel
On Wed, 6 Nov 2019 at 05:56, Dave Airlie <airlied@gmail.com> wrote:
>
> On Wed, 6 Nov 2019 at 05:49, Laurent Pinchart
> <laurent.pinchart@ideasonboard.com> wrote:
> >
> > Hi Dave,
> >
> > Has this pull request fallen through the cracks ?
>
> It fell down a different crack than usual, it made it from patchwork
> onto my hard drive, but then I didn't execute the pull.
>
> I've pulled it down, thanks for reminder.
Now it failed as I mentioned on irc.
I think the new kconfig option needs to be a tristate, otherwise
setting it to Y and having rcar-du as M fails to link
Dave.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-11-06 3:40 ` Dave Airlie
@ 2019-11-06 10:00 ` Laurent Pinchart
2019-11-06 10:02 ` Laurent Pinchart
0 siblings, 1 reply; 11+ messages in thread
From: Laurent Pinchart @ 2019-11-06 10:00 UTC (permalink / raw)
To: Dave Airlie; +Cc: jmondi, dri-devel
Hi Dave,
(CC'ing Jacopo)
On Wed, Nov 06, 2019 at 01:40:13PM +1000, Dave Airlie wrote:
> On Wed, 6 Nov 2019 at 05:56, Dave Airlie wrote:
> > On Wed, 6 Nov 2019 at 05:49, Laurent Pinchart wrote:
> > >
> > > Hi Dave,
> > >
> > > Has this pull request fallen through the cracks ?
> >
> > It fell down a different crack than usual, it made it from patchwork
> > onto my hard drive, but then I didn't execute the pull.
> >
> > I've pulled it down, thanks for reminder.
>
> Now it failed as I mentioned on irc.
>
> I think the new kconfig option needs to be a tristate, otherwise
> setting it to Y and having rcar-du as M fails to link
Sorry about that :-S Both I and the 0-day bot failed to catch it.
Jacopo, could you please have a look ? I'm afraid this likely mean a new
version of the series, and thus missing the merge window, as we're
already at -rc6.
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-11-06 10:00 ` Laurent Pinchart
@ 2019-11-06 10:02 ` Laurent Pinchart
2019-11-06 10:46 ` Jacopo Mondi
0 siblings, 1 reply; 11+ messages in thread
From: Laurent Pinchart @ 2019-11-06 10:02 UTC (permalink / raw)
To: Dave Airlie; +Cc: Jacopo Mondi, dri-devel
This time with Jacopo correctly CC'ed.
On Wed, Nov 06, 2019 at 12:00:59PM +0200, Laurent Pinchart wrote:
> Hi Dave,
>
> (CC'ing Jacopo)
>
> On Wed, Nov 06, 2019 at 01:40:13PM +1000, Dave Airlie wrote:
> > On Wed, 6 Nov 2019 at 05:56, Dave Airlie wrote:
> > > On Wed, 6 Nov 2019 at 05:49, Laurent Pinchart wrote:
> > > >
> > > > Hi Dave,
> > > >
> > > > Has this pull request fallen through the cracks ?
> > >
> > > It fell down a different crack than usual, it made it from patchwork
> > > onto my hard drive, but then I didn't execute the pull.
> > >
> > > I've pulled it down, thanks for reminder.
> >
> > Now it failed as I mentioned on irc.
> >
> > I think the new kconfig option needs to be a tristate, otherwise
> > setting it to Y and having rcar-du as M fails to link
>
> Sorry about that :-S Both I and the 0-day bot failed to catch it.
> Jacopo, could you please have a look ? I'm afraid this likely mean a new
> version of the series, and thus missing the merge window, as we're
> already at -rc6.
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-11-06 10:02 ` Laurent Pinchart
@ 2019-11-06 10:46 ` Jacopo Mondi
2019-11-06 12:52 ` Laurent Pinchart
0 siblings, 1 reply; 11+ messages in thread
From: Jacopo Mondi @ 2019-11-06 10:46 UTC (permalink / raw)
To: Laurent Pinchart; +Cc: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 1935 bytes --]
Hi Laurent, Dave,
On Wed, Nov 06, 2019 at 12:02:05PM +0200, Laurent Pinchart wrote:
> This time with Jacopo correctly CC'ed.
>
> On Wed, Nov 06, 2019 at 12:00:59PM +0200, Laurent Pinchart wrote:
> > Hi Dave,
> >
> > (CC'ing Jacopo)
> >
> > On Wed, Nov 06, 2019 at 01:40:13PM +1000, Dave Airlie wrote:
> > > On Wed, 6 Nov 2019 at 05:56, Dave Airlie wrote:
> > > > On Wed, 6 Nov 2019 at 05:49, Laurent Pinchart wrote:
> > > > >
> > > > > Hi Dave,
> > > > >
> > > > > Has this pull request fallen through the cracks ?
> > > >
> > > > It fell down a different crack than usual, it made it from patchwork
> > > > onto my hard drive, but then I didn't execute the pull.
> > > >
> > > > I've pulled it down, thanks for reminder.
> > >
> > > Now it failed as I mentioned on irc.
> > >
> > > I think the new kconfig option needs to be a tristate, otherwise
> > > setting it to Y and having rcar-du as M fails to link
> >
> > Sorry about that :-S Both I and the 0-day bot failed to catch it.
> > Jacopo, could you please have a look ? I'm afraid this likely mean a new
> > version of the series, and thus missing the merge window, as we're
> > already at -rc6.
I managed to reproduce it by setting DRM=m.
RCAR_CMM stays indeed =y as it's a bool. Simply setting it to tristate
is enough to have it set to =m when DRM=m.
Could this be changed when applying the series ?
diff --git a/drivers/gpu/drm/rcar-du/Kconfig b/drivers/gpu/drm/rcar-du/Kconfig
index 539d232790d1..e6607b5c8bb3 100644
--- a/drivers/gpu/drm/rcar-du/Kconfig
+++ b/drivers/gpu/drm/rcar-du/Kconfig
@@ -14,7 +14,7 @@ config DRM_RCAR_DU
If M is selected the module will be called rcar-du-drm.
config DRM_RCAR_CMM
- bool "R-Car DU Color Management Module (CMM) Support"
+ tristate "R-Car DU Color Management Module (CMM) Support"
depends on DRM && OF
depends on DRM_RCAR_DU
help
Thanks
j
>
> --
> Regards,
>
> Laurent Pinchart
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-11-06 10:46 ` Jacopo Mondi
@ 2019-11-06 12:52 ` Laurent Pinchart
2019-11-06 12:56 ` Laurent Pinchart
0 siblings, 1 reply; 11+ messages in thread
From: Laurent Pinchart @ 2019-11-06 12:52 UTC (permalink / raw)
To: Jacopo Mondi; +Cc: dri-devel
Hello,
On Wed, Nov 06, 2019 at 11:46:28AM +0100, Jacopo Mondi wrote:
> On Wed, Nov 06, 2019 at 12:02:05PM +0200, Laurent Pinchart wrote:
> > On Wed, Nov 06, 2019 at 12:00:59PM +0200, Laurent Pinchart wrote:
> > > On Wed, Nov 06, 2019 at 01:40:13PM +1000, Dave Airlie wrote:
> > > > On Wed, 6 Nov 2019 at 05:56, Dave Airlie wrote:
> > > > > On Wed, 6 Nov 2019 at 05:49, Laurent Pinchart wrote:
> > > > > >
> > > > > > Hi Dave,
> > > > > >
> > > > > > Has this pull request fallen through the cracks ?
> > > > >
> > > > > It fell down a different crack than usual, it made it from patchwork
> > > > > onto my hard drive, but then I didn't execute the pull.
> > > > >
> > > > > I've pulled it down, thanks for reminder.
> > > >
> > > > Now it failed as I mentioned on irc.
> > > >
> > > > I think the new kconfig option needs to be a tristate, otherwise
> > > > setting it to Y and having rcar-du as M fails to link
> > >
> > > Sorry about that :-S Both I and the 0-day bot failed to catch it.
> > > Jacopo, could you please have a look ? I'm afraid this likely mean a new
> > > version of the series, and thus missing the merge window, as we're
> > > already at -rc6.
>
> I managed to reproduce it by setting DRM=m.
> RCAR_CMM stays indeed =y as it's a bool. Simply setting it to tristate
> is enough to have it set to =m when DRM=m.
>
> Could this be changed when applying the series ?
>
> diff --git a/drivers/gpu/drm/rcar-du/Kconfig b/drivers/gpu/drm/rcar-du/Kconfig
> index 539d232790d1..e6607b5c8bb3 100644
> --- a/drivers/gpu/drm/rcar-du/Kconfig
> +++ b/drivers/gpu/drm/rcar-du/Kconfig
> @@ -14,7 +14,7 @@ config DRM_RCAR_DU
> If M is selected the module will be called rcar-du-drm.
>
> config DRM_RCAR_CMM
> - bool "R-Car DU Color Management Module (CMM) Support"
> + tristate "R-Car DU Color Management Module (CMM) Support"
> depends on DRM && OF
> depends on DRM_RCAR_DU
> help
This looks reasonable to me. I'll let Dave decide if he wants to apply
this change himself, or if I should send a new pull request for v5.6.
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-11-06 12:52 ` Laurent Pinchart
@ 2019-11-06 12:56 ` Laurent Pinchart
2019-11-07 11:13 ` Jacopo Mondi
0 siblings, 1 reply; 11+ messages in thread
From: Laurent Pinchart @ 2019-11-06 12:56 UTC (permalink / raw)
To: Jacopo Mondi; +Cc: dri-devel
Hi Jacopo,
On Wed, Nov 06, 2019 at 02:52:25PM +0200, Laurent Pinchart wrote:
> On Wed, Nov 06, 2019 at 11:46:28AM +0100, Jacopo Mondi wrote:
> > On Wed, Nov 06, 2019 at 12:02:05PM +0200, Laurent Pinchart wrote:
> > > On Wed, Nov 06, 2019 at 12:00:59PM +0200, Laurent Pinchart wrote:
> > > > On Wed, Nov 06, 2019 at 01:40:13PM +1000, Dave Airlie wrote:
> > > > > On Wed, 6 Nov 2019 at 05:56, Dave Airlie wrote:
> > > > > > On Wed, 6 Nov 2019 at 05:49, Laurent Pinchart wrote:
> > > > > > >
> > > > > > > Hi Dave,
> > > > > > >
> > > > > > > Has this pull request fallen through the cracks ?
> > > > > >
> > > > > > It fell down a different crack than usual, it made it from patchwork
> > > > > > onto my hard drive, but then I didn't execute the pull.
> > > > > >
> > > > > > I've pulled it down, thanks for reminder.
> > > > >
> > > > > Now it failed as I mentioned on irc.
> > > > >
> > > > > I think the new kconfig option needs to be a tristate, otherwise
> > > > > setting it to Y and having rcar-du as M fails to link
> > > >
> > > > Sorry about that :-S Both I and the 0-day bot failed to catch it.
> > > > Jacopo, could you please have a look ? I'm afraid this likely mean a new
> > > > version of the series, and thus missing the merge window, as we're
> > > > already at -rc6.
> >
> > I managed to reproduce it by setting DRM=m.
> > RCAR_CMM stays indeed =y as it's a bool. Simply setting it to tristate
> > is enough to have it set to =m when DRM=m.
> >
> > Could this be changed when applying the series ?
> >
> > diff --git a/drivers/gpu/drm/rcar-du/Kconfig b/drivers/gpu/drm/rcar-du/Kconfig
> > index 539d232790d1..e6607b5c8bb3 100644
> > --- a/drivers/gpu/drm/rcar-du/Kconfig
> > +++ b/drivers/gpu/drm/rcar-du/Kconfig
> > @@ -14,7 +14,7 @@ config DRM_RCAR_DU
> > If M is selected the module will be called rcar-du-drm.
> >
> > config DRM_RCAR_CMM
> > - bool "R-Car DU Color Management Module (CMM) Support"
> > + tristate "R-Car DU Color Management Module (CMM) Support"
> > depends on DRM && OF
> > depends on DRM_RCAR_DU
> > help
>
> This looks reasonable to me. I'll let Dave decide if he wants to apply
> this change himself, or if I should send a new pull request for v5.6.
Actually this won't work. With DRM_RCAR_DU=y and DRM_RCAR_CMM=m you'll
have a link failure.
Let's fix this properly for v5.6. Could you please propose a fix that
will work with all combinations of DRM, DRM_RCAR_DU and DRM_RCAR_CMM ?
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-11-06 12:56 ` Laurent Pinchart
@ 2019-11-07 11:13 ` Jacopo Mondi
2019-11-07 13:17 ` Laurent Pinchart
0 siblings, 1 reply; 11+ messages in thread
From: Jacopo Mondi @ 2019-11-07 11:13 UTC (permalink / raw)
To: Laurent Pinchart; +Cc: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 3058 bytes --]
Hi Laurent,
On Wed, Nov 06, 2019 at 02:56:53PM +0200, Laurent Pinchart wrote:
> Hi Jacopo,
>
> On Wed, Nov 06, 2019 at 02:52:25PM +0200, Laurent Pinchart wrote:
> > On Wed, Nov 06, 2019 at 11:46:28AM +0100, Jacopo Mondi wrote:
> > > On Wed, Nov 06, 2019 at 12:02:05PM +0200, Laurent Pinchart wrote:
> > > > On Wed, Nov 06, 2019 at 12:00:59PM +0200, Laurent Pinchart wrote:
> > > > > On Wed, Nov 06, 2019 at 01:40:13PM +1000, Dave Airlie wrote:
> > > > > > On Wed, 6 Nov 2019 at 05:56, Dave Airlie wrote:
> > > > > > > On Wed, 6 Nov 2019 at 05:49, Laurent Pinchart wrote:
> > > > > > > >
> > > > > > > > Hi Dave,
> > > > > > > >
> > > > > > > > Has this pull request fallen through the cracks ?
> > > > > > >
> > > > > > > It fell down a different crack than usual, it made it from patchwork
> > > > > > > onto my hard drive, but then I didn't execute the pull.
> > > > > > >
> > > > > > > I've pulled it down, thanks for reminder.
> > > > > >
> > > > > > Now it failed as I mentioned on irc.
> > > > > >
> > > > > > I think the new kconfig option needs to be a tristate, otherwise
> > > > > > setting it to Y and having rcar-du as M fails to link
> > > > >
> > > > > Sorry about that :-S Both I and the 0-day bot failed to catch it.
> > > > > Jacopo, could you please have a look ? I'm afraid this likely mean a new
> > > > > version of the series, and thus missing the merge window, as we're
> > > > > already at -rc6.
> > >
> > > I managed to reproduce it by setting DRM=m.
> > > RCAR_CMM stays indeed =y as it's a bool. Simply setting it to tristate
> > > is enough to have it set to =m when DRM=m.
> > >
> > > Could this be changed when applying the series ?
> > >
> > > diff --git a/drivers/gpu/drm/rcar-du/Kconfig b/drivers/gpu/drm/rcar-du/Kconfig
> > > index 539d232790d1..e6607b5c8bb3 100644
> > > --- a/drivers/gpu/drm/rcar-du/Kconfig
> > > +++ b/drivers/gpu/drm/rcar-du/Kconfig
> > > @@ -14,7 +14,7 @@ config DRM_RCAR_DU
> > > If M is selected the module will be called rcar-du-drm.
> > >
> > > config DRM_RCAR_CMM
> > > - bool "R-Car DU Color Management Module (CMM) Support"
> > > + tristate "R-Car DU Color Management Module (CMM) Support"
> > > depends on DRM && OF
> > > depends on DRM_RCAR_DU
> > > help
> >
> > This looks reasonable to me. I'll let Dave decide if he wants to apply
> > this change himself, or if I should send a new pull request for v5.6.
>
> Actually this won't work. With DRM_RCAR_DU=y and DRM_RCAR_CMM=m you'll
> have a link failure.
>
You're right.. Actually just changing the RCAR_CMM option to tristate
won't be enough. Adding "imply DRM_RCAR_CMM" to the DU option instead
makes it impossible to select CMM=m when DU=y and viceversa, so it
should be enough.
> Let's fix this properly for v5.6. Could you please propose a fix that
> will work with all combinations of DRM, DRM_RCAR_DU and DRM_RCAR_CMM ?
Can I send a v6.1 version of the CMM patch only, or would you prefer a
v7 ? we're anyway late for 5.5 I presume
Thanks
j
>
> --
> Regards,
>
> Laurent Pinchart
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support
2019-11-07 11:13 ` Jacopo Mondi
@ 2019-11-07 13:17 ` Laurent Pinchart
0 siblings, 0 replies; 11+ messages in thread
From: Laurent Pinchart @ 2019-11-07 13:17 UTC (permalink / raw)
To: Jacopo Mondi; +Cc: dri-devel
Hi Jacopo,
On Thu, Nov 07, 2019 at 12:13:35PM +0100, Jacopo Mondi wrote:
> On Wed, Nov 06, 2019 at 02:56:53PM +0200, Laurent Pinchart wrote:
> > On Wed, Nov 06, 2019 at 02:52:25PM +0200, Laurent Pinchart wrote:
> >> On Wed, Nov 06, 2019 at 11:46:28AM +0100, Jacopo Mondi wrote:
> >>> On Wed, Nov 06, 2019 at 12:02:05PM +0200, Laurent Pinchart wrote:
> >>>> On Wed, Nov 06, 2019 at 12:00:59PM +0200, Laurent Pinchart wrote:
> >>>>> On Wed, Nov 06, 2019 at 01:40:13PM +1000, Dave Airlie wrote:
> >>>>>> On Wed, 6 Nov 2019 at 05:56, Dave Airlie wrote:
> >>>>>>> On Wed, 6 Nov 2019 at 05:49, Laurent Pinchart wrote:
> >>>>>>>>
> >>>>>>>> Hi Dave,
> >>>>>>>>
> >>>>>>>> Has this pull request fallen through the cracks ?
> >>>>>>>
> >>>>>>> It fell down a different crack than usual, it made it from patchwork
> >>>>>>> onto my hard drive, but then I didn't execute the pull.
> >>>>>>>
> >>>>>>> I've pulled it down, thanks for reminder.
> >>>>>>
> >>>>>> Now it failed as I mentioned on irc.
> >>>>>>
> >>>>>> I think the new kconfig option needs to be a tristate, otherwise
> >>>>>> setting it to Y and having rcar-du as M fails to link
> >>>>>
> >>>>> Sorry about that :-S Both I and the 0-day bot failed to catch it.
> >>>>> Jacopo, could you please have a look ? I'm afraid this likely mean a new
> >>>>> version of the series, and thus missing the merge window, as we're
> >>>>> already at -rc6.
> >>>
> >>> I managed to reproduce it by setting DRM=m.
> >>> RCAR_CMM stays indeed =y as it's a bool. Simply setting it to tristate
> >>> is enough to have it set to =m when DRM=m.
> >>>
> >>> Could this be changed when applying the series ?
> >>>
> >>> diff --git a/drivers/gpu/drm/rcar-du/Kconfig b/drivers/gpu/drm/rcar-du/Kconfig
> >>> index 539d232790d1..e6607b5c8bb3 100644
> >>> --- a/drivers/gpu/drm/rcar-du/Kconfig
> >>> +++ b/drivers/gpu/drm/rcar-du/Kconfig
> >>> @@ -14,7 +14,7 @@ config DRM_RCAR_DU
> >>> If M is selected the module will be called rcar-du-drm.
> >>>
> >>> config DRM_RCAR_CMM
> >>> - bool "R-Car DU Color Management Module (CMM) Support"
> >>> + tristate "R-Car DU Color Management Module (CMM) Support"
> >>> depends on DRM && OF
> >>> depends on DRM_RCAR_DU
> >>> help
> >>
> >> This looks reasonable to me. I'll let Dave decide if he wants to apply
> >> this change himself, or if I should send a new pull request for v5.6.
> >
> > Actually this won't work. With DRM_RCAR_DU=y and DRM_RCAR_CMM=m you'll
> > have a link failure.
>
> You're right.. Actually just changing the RCAR_CMM option to tristate
> won't be enough. Adding "imply DRM_RCAR_CMM" to the DU option instead
> makes it impossible to select CMM=m when DU=y and viceversa, so it
> should be enough.
>
> > Let's fix this properly for v5.6. Could you please propose a fix that
> > will work with all combinations of DRM, DRM_RCAR_DU and DRM_RCAR_CMM ?
>
> Can I send a v6.1 version of the CMM patch only, or would you prefer a
> v7 ? we're anyway late for 5.5 I presume
v6.1 should be enough. Please make sure you test all combinations of
CONFIG_DRM_RCAR_DU and CONFIG_DRM_RCAR_CMM :-)
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2019-11-07 13:17 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-22 10:31 [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support Laurent Pinchart
2019-11-05 19:49 ` Laurent Pinchart
2019-11-05 19:56 ` Dave Airlie
2019-11-06 3:40 ` Dave Airlie
2019-11-06 10:00 ` Laurent Pinchart
2019-11-06 10:02 ` Laurent Pinchart
2019-11-06 10:46 ` Jacopo Mondi
2019-11-06 12:52 ` Laurent Pinchart
2019-11-06 12:56 ` Laurent Pinchart
2019-11-07 11:13 ` Jacopo Mondi
2019-11-07 13:17 ` Laurent Pinchart
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.