Re: [PATCH v8 1/2] mfd: add atmel-hlcdc driver

[Lee Jones <lee.jones@linaro.org>]

> >> 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@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