devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: wangfei <w.f-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
To: Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	catalin.marinas-5wv7dgnIgG8@public.gmane.org,
	will.deacon-5wv7dgnIgG8@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	pawel.moll-5wv7dgnIgG8@public.gmane.org,
	mark.rutland-5wv7dgnIgG8@public.gmane.org,
	ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org,
	khilman-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org,
	broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	bintian.wang-hv44wF8Li93QT0dZR+AlfA@public.gmane.org,
	xuwei5-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org,
	haojian.zhuang-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	zhangfei.gao-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	guodong.xu-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	jorge.ramirez-ortiz-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	puck.chen-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org,
	xuyiping-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org,
	kong.kongxinwei-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org,
	z.liuxinliang-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org,
	william.wfei-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	zhongkaihua-hv44wF8Li93QT0dZR+AlfA@public.gmane.org
Subject: Re: [PATCH 1/8] dt-bindings: pmic: Document Hi655x pmic driver
Date: Thu, 8 Oct 2015 15:03:17 +0800	[thread overview]
Message-ID: <56161535.1030801@huawei.com> (raw)
In-Reply-To: <20151001075844.GL3214@x1>

On 2015/10/1 15:58, Lee Jones wrote:
> I just noticed that you Cc'ed all of the Core ARM people too.  Please
> do not do that.  I'm sure they have enough of their own mail to trawl
> though.
> 
> Only mail relevant parties i.e. those who show up when you use:
> 
>   ./scripts/get_maintainer.pl.
> 
OK,i see,thank you!
>> Document the new compatible for Hisilicon Hi655x pmic driver.
>>
>> Signed-off-by: Fei Wang <w.f-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
>> ---
>>  .../devicetree/bindings/mfd/hisilicon,hi655x.txt   |   80 ++++++++++++++++++++
>>  1 file changed, 80 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt
>>
>> diff --git a/Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt b/Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt
>> new file mode 100644
>> index 0000000..17bd8ca
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt
>> @@ -0,0 +1,80 @@
>> +Hisilicon hi655x Power Management Integrated Circuit (PMIC)
>> +
>> +hi655x consists of a large and varied group of sub-devices:
>> +
>> +Device			 Supply Names	 Description
>> +------			 ------------	 -----------
>> +hi655x-powerkey		:		: Powerkey
>> +hi655x-regulator-pmic	:		: Regulators
>> +hi655x-usbvbus		:		: USB plug detection
>> +hi655x-pmu-rtc		:		: RTC
>> +hi655x-coul		:		: Coulomb
>> +
>> +Required properties:
>> +- compatible : Should be "hisilicon,hi655x-pmic-driver"
>> +- reg: Base address of PMIC on hi6220 soc
>> +- #interrupt-cells: Should be 2, is the local IRQ number for hi655x.
>> +- interrupt-controller: hi655x has internal IRQs (has own IRQ domain).
>> +- pmu_irq_gpio: should be &gpio_pmu_irq_n, is the IRQ gpio of hi655x.
>> +
>> +Example:
>> +	pmic: pmic@F8000000 {
>> +                compatible = "hisilicon,hi655x-pmic-driver";
>> +                reg = <0x0 0xF8000000 0x0 0x1000>;
>> +                #interrupt-cells = <2>;
>> +                interrupt-controller;
>> +                pmu_irq_gpio = <&gpio_pmu_irq_n>;
>> +                status = "ok";
>> +
>> +                ponkey:ponkey@b1{
>> +                        compatible = "hisilicon,hi655x-powerkey";
>> +                        interrupt-parent = <&pmic>;
>> +                        interrupts = <6 0>, <5 0>, <4 0>;
>> +                        interrupt-names = "down", "up", "hold 1s";
>> +                };
>> +
>> +                coul: coul@1 {
>> +                        compatible = "hisilicon,hi655x-coul";
>> +                        interrupt-parent = <&pmic>;
>> +                        interrupts = <24 0>, <25 0>, <26 0>, <27 0>;
>> +                        interrupt-names = "cl_int_i", "cl_out_i", "cl_in_i", "vbat_int_i";
>> +                        battery_product_index = <0>;
>> +                        status = "ok";
>> +                };
>> +
>> +                rtc: rtc@1 {
>> +                        compatible = "hisilicon,hi655x-pmu-rtc";
>> +                        interrupt-parent = <&pmic>;
>> +                        interrupts = <20 0>;
>> +                        interrupt-names = "hi655x_pmu_rtc";
>> +                        board_id = <1>;
>> +                };
>> +
>> +                usbvbus:usbvbus@b2{
>> +                        compatible = "hisilicon,hi655x-usbvbus";
>> +                        interrupt-parent = <&pmic>;
>> +                        interrupts = <9 0>, <8 0>;
>> +                        interrupt-names = "connect", "disconnect";
>> +                };
>> +
>> +		ldo2: regulator@a21 {
>> +                        compatible = "hisilicon,hi655x-regulator-pmic";
>> +                        regulator-name = "ldo2";
>> +                        regulator-min-microvolt = <2500000>;
>> +                        regulator-max-microvolt = <3200000>;
>> +                        hisilicon,valid-modes-mask = <0x02>;
>> +                        hisilicon,valid-ops-mask = <0x01d>;
>> +                        hisilicon,initial-mode = <0x02>;
>> +                        hisilicon,regulator-type = <0x01>;
>> +
>> +                        hisilicon,off-on-delay = <120>;
>> +                        hisilicon,ctrl-regs = <0x029 0x02a 0x02b>;
>> +                        hisilicon,ctrl-data = <0x1 0x1>;
>> +                        hisilicon,vset-regs = <0x072>;
>> +                        hisilicon,vset-data = <0 0x3>;
>> +                        hisilicon,regulator-n-vol = <8>;
>> +                        hisilicon,vset-table = <2500000>,<2600000>,<2700000>,<2800000>,<2900000>,<3000000>,<3100000>,<3200000>;
>> +                        hisilicon,num_consumer_supplies = <1>;
>> +                        hisilicon,consumer-supplies = "sensor_analog";
>> +                };
>> +	};
> 


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2015-10-08  7:03 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-30 11:05 [PATCH 0/8] Add Support for Hi6220 PMIC Hi6553 MFD Core and Regulator Fei Wang
2015-09-30 11:05 ` [PATCH 1/8] dt-bindings: pmic: Document Hi655x pmic driver Fei Wang
     [not found]   ` <1443611111-3196-2-git-send-email-w.f-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2015-09-30 17:39     ` Mark Brown
2015-10-08  6:33       ` wangfei
2015-10-12 16:49         ` Mark Brown
2015-10-01  7:56     ` Lee Jones
2015-10-01  7:58   ` Lee Jones
2015-10-08  7:03     ` wangfei [this message]
2015-09-30 11:05 ` [PATCH 2/8] mfd: Hi655x: Add support for PMIC Hi655x MFD Fei Wang
2015-09-30 11:05 ` [PATCH 4/8] regulator: Hi655x: Add support for Hi655x regulator Fei Wang
     [not found]   ` <1443611111-3196-5-git-send-email-w.f-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2015-09-30 17:58     ` Mark Brown
2015-10-08  7:38       ` wangfei
2015-10-08  7:47   ` Javier Martinez Canillas
2015-09-30 11:05 ` [PATCH 5/8] arm64: dts: Add Hi655x regulator config node Fei Wang
2015-09-30 11:05 ` [PATCH 6/8] dt-bindings: mtcmos: Document Hi6220 mtcmos driver Fei Wang
     [not found] ` <1443611111-3196-1-git-send-email-w.f-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2015-09-30 11:05   ` [PATCH 3/8] arm64: dts: add Hi655x pmic node Fei Wang
2015-09-30 11:05   ` [PATCH 7/8] mtcmos: Hi6220: Add Hi6220 mtcmos regulator driver Fei Wang
2015-09-30 11:05   ` [PATCH 8/8] arm64: dts: Add Hi6220 mtcmos regulator node Fei Wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56161535.1030801@huawei.com \
    --to=w.f-hv44wf8li93qt0dzr+alfa@public.gmane.org \
    --cc=bintian.wang-hv44wF8Li93QT0dZR+AlfA@public.gmane.org \
    --cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=catalin.marinas-5wv7dgnIgG8@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=guodong.xu-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=haojian.zhuang-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
    --cc=jorge.ramirez-ortiz-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=khilman-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=kong.kongxinwei-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org \
    --cc=lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
    --cc=puck.chen-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org \
    --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=will.deacon-5wv7dgnIgG8@public.gmane.org \
    --cc=william.wfei-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=xuwei5-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org \
    --cc=xuyiping-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org \
    --cc=z.liuxinliang-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org \
    --cc=zhangfei.gao-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=zhongkaihua-hv44wF8Li93QT0dZR+AlfA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).