[PATCH v8 1/2] mfd: add atmel-hlcdc driver
Lee Jones
lee.jones at linaro.org
Tue Oct 7 06:28:41 PDT 2014
> >> That said, Nicolas Ferre (Cc'ing) at some point requested this to become
> >> a select (or at least for the DRM driver, but I guess the same applies
> >> to PWM) on the grounds that a depends on will make it more difficult to
> >> enable the driver.
> >
> > It's not that much more difficult. It just entails enabling 3 instead
> > of 2 config options.
>
> Yes it is more difficult. Believe me, it's a mess, but...
>
> > Once all of the required components are merged,
> > feel free to drop back to 'select'. This is easier than sharing round
> > immutable branches all over the place.
>
> .. I agree with this option of moving to an easier-to-merge solution and
> then dealing with the ease of use.
>
> >> So we have two options here: 1) turn the select into a depends on here
> >> and allow the dependency to be resolved that way, or 2) solve the
> >> dependency by making sure the MFD part is merged first (either by
> >> pulling the MFD tree into the PWM and DRM trees or waiting for a full
> >> cycle for the MFD changes to land).
> >>
> >> I don't mind either way.
> >
> > I'll go with either of the two suggestions above.
>
> So, Lee and Thierry, you can both take your part in your respective
> trees with the change (1) described above and with my:
>
> Acked-by: Nicolas Ferre <nicolas.ferre at atmel.com>
Works for me. The MFD part has already been applied.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
More information about the dri-devel
mailing list