Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759014Ab2KWJXh (ORCPT ); Fri, 23 Nov 2012 04:23:37 -0500 Received: from mail-bk0-f46.google.com ([209.85.214.46]:36110 "EHLO mail-bk0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758442Ab2KWJXd (ORCPT ); Fri, 23 Nov 2012 04:23:33 -0500 Date: Fri, 23 Nov 2012 09:23:28 +0000 From: Lee Jones To: Viresh Kumar Cc: sameo@linux.intel.com, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, spear-devel@list.st.com, Vipul Kumar Samar Subject: Re: [PATCH V2 2/2] mfd: stmpe: Extend DT support in stmpe driver Message-ID: <20121123092328.GB17471@gmail.com> References: <20121122112451.GE4328@gmail.com> <20121122154612.GC10986@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3358 Lines: 84 On Thu, 22 Nov 2012, Viresh Kumar wrote: > On 22 November 2012 21:16, Lee Jones wrote: > >> >> diff --git a/Documentation/devicetree/bindings/mfd/stmpe.txt b/Documentation/devicetree/bindings/mfd/stmpe.txt > >> >> +- irq-over-gpio: bool, true if gpio is used to get irq > >> >> +- irq-gpios: gpio number over which irq will be requested (significant only if > >> >> + irq-over-gpio is true) > >> > > >> > You don't need these. Use gpio_to_irq() instead. > >> > >> I am passing gpio numbers here and am doing gpio_to_irq() in driver. > >> Didn't get this one :( > > > > For a start you have 'irq-over-gpio' in the binding document and Device Tree > > and 'irq_over_gpio' in the code. Has it even been tested? > > > > GPIOs are used as IRQ lines in many other previously DT:ed drivers. Take a > > look to see how they are handled without adding unnecessary DT bindings. > > I already knew it, should have picked that up. :( > > >> stmpe is an interrupt controller for the IP's which are present inside > >> it: gpio, adc. > >> But interrupt lines for them are managed by stmpe driver internally. So should > >> we really add interrupt-controller for it? > > > > You can't manage IRQ lines internally, you have to go through > > the IRQ subsystem. When you request an IRQ via device tree you > > will do so like this: > > By that i meant, there is no external node which would have stmpe as > interrupt controller. Because all of them would be its child node. > > This is guaranteed because stmpe is an external device is present on board. > So, it will have its entry in board dts file, and so wouldn't be > scattered in different > files. It doesn't matter how it's wired up. If another node references it as it's IRQ controller you have to declare it as one using the interrupt-controller binding. > > The STMPE GPIO controller can't be used by Device Tree yet in any case, > > because it doesn't have an IRQ domain. This is compulsory, or it won't > > work. Have you tried to test this functionality yet? > > I don't have SPEAr board to test it anymore. I have moved out of ST now and > working in linaro as ARM asignee. Just pushing these as an part time activity. Surely you can't push patches which haven't been tested?! Try to get yourself some hardware. Does anyone near you have an HREF? In the mean-time, I will write you an IRQ domain. > Though ST guys would have tested stmpe, but stmpe-gpio, i am not sure about. It needs to be tested before being accpeted. > > I didn't go through them, but are you sure that: > > > > 1. Can I do without them? > > 1.1 Can I derive the configuration from other things? > > 2.2 Are they _really_ required, or am I just blindly copying platform data? > > 2. Does a similar binding already exist? > > 3. Can other drivers make use of them? > > 3.1 If so, create a generic binding > > 3.2 If not, prepend the binding with "," > > I will go through them again. Thanks. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/