From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1033080AbbKFJnT (ORCPT ); Fri, 6 Nov 2015 04:43:19 -0500 Received: from mx0b-001ae601.pphosted.com ([67.231.152.168]:36212 "EHLO mx0b-001ae601.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756411AbbKFJnP (ORCPT ); Fri, 6 Nov 2015 04:43:15 -0500 Date: Fri, 6 Nov 2015 09:27:33 +0000 From: Charles Keepax To: Lee Jones CC: Robert Jarzmik , Mark Brown , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , , , Subject: Re: [PATCH] ASoC: wm9713: add gpio chip Message-ID: <20151106092733.GG2549@ck-lbox> References: <1446657164-25012-1-git-send-email-robert.jarzmik@free.fr> <20151104183322.GD2549@ck-lbox> <87oaf9bls9.fsf@belgarion.home> <20151106092913.GC3378@x1> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20151106092913.GC3378@x1> User-Agent: Mutt/1.5.20 (2009-06-14) X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2015-11-06_07:,, signatures=0 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1507310000 definitions=main-1511060174 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Nov 06, 2015 at 09:29:13AM +0000, Lee Jones wrote: > On Wed, 04 Nov 2015, Robert Jarzmik wrote: > > > Charles Keepax 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 > > >> --- > > > > > > 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. > > I have no idea what you're talking about. Context please? Apologies Lee, we are discussing a patch that adds a GPIO driver into an AC97 CODEC. I had suggested that perhaps we should put the GPIO driver as a seperate driver under GPIO and link the two with an MFD. But Mark has already replied in the thread to say that he doesn't think that will be necessary. Although he did raise some concerns that perhaps it could be done more generally as it should apply to other AC97 CODECs as well. So I think you can probably safely ignore this for now, sorry for the noise. Thanks, Charles