Re: [GIT PULL FOR v5.5 - 2nd try] R-Car DU CMM support

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

 



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@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[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