linux-kernel - Re: [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Mon, 18 Aug 2014 11:48:03 -0400 (EDT)
From:	kiran.padwal@...rtplayin.com
To:	"Bjorn Andersson" <bjorn@...o.se>
Cc:	"Rob Herring" <robh+dt@...nel.org>,
	"Pawel Moll" <pawel.moll@....com>,
	"Mark Rutland" <mark.rutland@....com>,
	"Ian Campbell" <ijc+devicetree@...lion.org.uk>,
	"Kumar Gala" <galak@...eaurora.org>,
	"Russell King" <linux@....linux.org.uk>,
	"David Brown" <davidb@...eaurora.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-arm-msm" <linux-arm-msm@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] ARM: apq8064: Add pinmux and i2c pinctrl nodes


On Friday, August 15, 2014 3:08pm, kiran.padwal@...rtplayin.com said:

> 
> On Thursday, August 14, 2014 6:37pm, "Bjorn Andersson" <bjorn@...o.se> said:
> 
>> On Thu, Aug 14, 2014 at 12:20 AM, Kiran Padwal
>> <kiran.padwal@...rtplayin.com> wrote:
>>> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> b/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> index 92bf793..fbebf5c 100644
>>> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
>>> @@ -70,6 +70,17 @@
>>>                 ranges;
>>>                 compatible = "simple-bus";
>>>
>>> +               qcom_pinmux: pinmux@...000 {
>>
>> There are (at least) three different pinmuxes in these platforms: TLMM, PMIC
>> GPIO, PMIC MPP. Also this is the phandle that is used to reference the gpio
>> chip throughout the board.
>>
>> So I would like to suggest that we name it "tlmm" or like in the downstream
>> kernel "msmgpio".
> 
> ok. How about "qcomgpio" or rename it to msmgpio?

Thanks for suggestion, I will rename it to "tlmm".

> 
>>
>>> +                       compatible = "qcom,apq8064-pinctrl";
>>> +                       reg = <0x800000 0x4000>;
>>> +
>>> +                       gpio-controller;
>>> +                       #gpio-cells = <2>;
>>> +                       interrupt-controller;
>>> +                       #interrupt-cells = <2>;
>>> +                       interrupts = <0 32 0x4>;
>>
>> I must have gotten this wrong in the dt binding example, sorry about that.
>> interrupts should be <0 16 0x4>.
> 
> Thanks for information. I will update that.
> 
>>
>>> +               };
>>
>> Regards,
>> Bjorn
>>
> 
> 
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/