Re: [PATCH] ASoC: wm9713: add gpio chip

Robert Jarzmik
2015-11-04T19:42:13Z

Charles Keepax <ckeepax@opensource.wolfsonmicro.com> writes:

> On Wed, Nov 04, 2015 at 06:12:44PM +0100, Robert Jarzmik wrote:
>> The Wolfson WM9713 provides 8 GPIOs. If the gpiolib is compiled in the
>> kernel, declare a gpio chip.
>>
>> Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
>> ---
>
> You should probably make a seperate driver within GPIO for this
> and then tie the two together, using an MFD. I appreciate that is
> more work but it is likely a nicer solution overall.

I'd like to first have a confirmation from :
- Mark (Brown)
- and Lee (Jones)

The confirmation I'm looking for states that :
- the wm9713 should have a part in the mfd tree
- the gpio part should be in drivers/gpio
- the sound soc codecs will remain as is
- if the future driver/mfd/wm9713.c is technically sound, it will be accepted

I remember at least one example where the MFD approach was rejected from mfd
tree for pxa gpios, so I won't work unless I have a confirmation from both
maintainers.

Thanks.

--
Robert