Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759182Ab3CZJh6 (ORCPT ); Tue, 26 Mar 2013 05:37:58 -0400 Received: from hqemgate04.nvidia.com ([216.228.121.35]:9372 "EHLO hqemgate04.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753147Ab3CZJh5 (ORCPT ); Tue, 26 Mar 2013 05:37:57 -0400 X-PGP-Universal: processed; by hqnvupgp08.nvidia.com on Tue, 26 Mar 2013 02:37:31 -0700 Message-ID: <51516BC3.7030707@nvidia.com> Date: Tue, 26 Mar 2013 15:04:59 +0530 From: Laxman Dewangan User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121028 Thunderbird/16.0.2 MIME-Version: 1.0 To: Graeme Gregory CC: Kishon Vijay Abraham I , "balbi@ti.com" , Rajendra Nayak , "grant.likely@secretlab.ca" , "rob.herring@calxeda.com" , "rob@landley.net" , "gregkh@linuxfoundation.org" , "s-guiriec@ti.com" , "sameo@linux.intel.com" , "broonie@opensource.wolfsonmicro.com" , "devicetree-discuss@lists.ozlabs.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-usb@vger.kernel.org" Subject: Re: [PATCH v3] USB: PHY: Palmas USB Transceiver Driver References: <1362662506-14823-4-git-send-email-kishon@ti.com> <1364203926-24488-1-git-send-email-kishon@ti.com> <51501CFB.4020905@nvidia.com> <51513A40.7080905@ti.com> <515163F6.3010400@slimlogic.co.uk> <51516695.8020808@nvidia.com> <51516A10.40704@slimlogic.co.uk> In-Reply-To: <51516A10.40704@slimlogic.co.uk> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3367 Lines: 80 On Tuesday 26 March 2013 02:57 PM, Graeme Gregory wrote: > On 26/03/13 09:12, Laxman Dewangan wrote: >> On Tuesday 26 March 2013 02:31 PM, Graeme Gregory wrote: >> >> But still you are using the PALMAS macro here and indirectly it is >> tied up. It is not completely independent. >> If need to be independent then regmap pointer with address need to be >> passed as independt header and no where on code whould refer the PALMA. >> I think as per current code, it is not possible although it is your >> big plan what I understand from some time back in one of patch >> discussion. >> > It is actually almost possible, but it is something I gave up looking > at. You can get the regmap of your parent i2c device without having > knowledge of the type of parent. There is multiple regmap of parent and hence getting correct regmap is really issue. May be RTC require regmap[0] and gpio require regmap[1]. > >>>>>> + palmas_usb->dev = &pdev->dev; >>>>>> + >>>>>> + palmas_usb->irq1 = regmap_irq_get_virq(palmas->irq_data, >>>>>> + PALMAS_ID_OTG_IRQ); >>>>>> + palmas_usb->irq2 = regmap_irq_get_virq(palmas->irq_data, >>>>>> + PALMAS_ID_IRQ); >>>>>> + palmas_usb->irq3 = regmap_irq_get_virq(palmas->irq_data, >>>>>> + PALMAS_VBUS_OTG_IRQ); >>>>>> + palmas_usb->irq4 = regmap_irq_get_virq(palmas->irq_data, >>>>>> + PALMAS_VBUS_IRQ); >>>>> Should be come from platform_get_irq() through platform driver. >>>> No. It can be obtained from regmap too. >> Kishon, >> I think it is very much possible. You can pass the interrupt throough >> IRQ_RESOURCE and populate it from DT. If you provide proper interrupt >> parent and irq number then irq framework take care of every thing. >> already tested this with RTC interrupt of plama and it worked very well. >> > If we are tightly coupling as above then using platform_irq is an extra > inefficiency. You both have to populate this then parse it afterwards. > Why not just use the regmap helper? Ill admit this code is like this as > there was a period where platform irqs in DT just was not working right! > > We should really agree now if we are going for loose or tight coupling > now rather than keep switching? Here we are hardcoding for PALMAS_ID_OTG_IRQ and so on. If we take data from platform then it need not and it will be completely independent of palma atleast on this front. We need to populate just as: palmas: palmas { ::::::: palams_usb_phy { compatile = ... interrupt-parent = <& palmas>; interrupt = < 10, 0, 21, 0, 22, 0, 23, 0>; } and in code, we just need to do irq1 = platform_get_irq(pdev, 0); irq2 = platform_get_irq(pdev, 1); etc.. So here, actually we do not need to use palmas one and it is completely independent. Also the way you define the DT od palmas, the above one looks more appropriate. -- 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/