From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932651Ab3BKVd4 (ORCPT ); Mon, 11 Feb 2013 16:33:56 -0500 Received: from mho-03-ewr.mailhop.org ([204.13.248.66]:62521 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1759394Ab3BKVdy (ORCPT ); Mon, 11 Feb 2013 16:33:54 -0500 X-Mail-Handler: Dyn Standard SMTP by Dyn X-Originating-IP: 50.131.214.131 X-Report-Abuse-To: abuse@dyndns.com (see http://www.dyndns.com/services/sendlabs/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX1+1TfJ+ziiFUHnMPOGuHCvK Date: Mon, 11 Feb 2013 13:33:49 -0800 From: Tony Lindgren To: Stephen Warren Cc: Linus Walleij , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Stephen Warren , Anmar Oueja , Laurent Meunier , Linus Walleij Subject: Re: [PATCH] pinctrl/pinconfig: add debug interface Message-ID: <20130211213348.GY4801@atomide.com> References: <1360527070-18430-1-git-send-email-linus.walleij@stericsson.com> <51195A63.7080706@wwwdotorg.org> <20130211210041.GW4801@atomide.com> <51195F00.4020309@wwwdotorg.org> <20130211212105.GX4801@atomide.com> <51196161.7020901@wwwdotorg.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <51196161.7020901@wwwdotorg.org> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Stephen Warren [130211 13:27]: > On 02/11/2013 02:21 PM, Tony Lindgren wrote: > > * Stephen Warren [130211 13:17]: > >> On 02/11/2013 02:00 PM, Tony Lindgren wrote: > >>> * Stephen Warren [130211 12:57]: > >>>> On 02/10/2013 01:11 PM, Linus Walleij wrote: > >>>>> From: Laurent Meunier > >>>>> > >>>>> This update adds a debugfs interface to modify a pin configuration > >>>>> for a given state in the pinctrl map. This allows to modify the > >>>>> configuration for a non-active state, typically sleep state. > >>>>> This configuration is not applied right away, but only when the state > >>>>> will be entered. > >>>>> > >>>>> This solution is mandated for us by HW validation: in order > >>>>> to test and verify several pin configurations during sleep without > >>>>> recompiling the software. > >>>> > >>>> I never understood why HW engineers can't just recompile the kernel. > >>>> Besides, it's just a device tree change these days - no recompile even > >>>> required, right? > >>> > >>> Typically when bringing up a new board you do not have the driver > >>> specific mux settings verified. For developers, it's easiest to tweak > >>> the muxing during runtime do the drivers as a loadable module, then > >>> export the verified mux configuration into a .dts file. > >> > >> Well HW engineers typically just write to the HW registers directly > >> rather than screwing around with the Linux pinctrl tables and > >> unloading/reloading drivers. > > > > I've seen cases where most of the mux register configuration are > > done correctly by HW engineers.. But there's always been some pieces > > of the pin configuration wrong initially and that needs to be fixed by > > the driver people to get things working. > > > > Being able to set the mux configuration via debugfs is also very > > useful for the case of adding external devices to your proto boards > > like beagle etc. > > This patch won't work for that, since it doesn't support adding new > entries to the mux table, just editing existing entries. Being able to change the existing mux entries still helps developers to debug things like pull values needed for i2c device impedance etc. Regards, Tony