[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <56A0AF6D.6070200@hisilicon.com>
Date: Thu, 21 Jan 2016 18:14:05 +0800
From: chenfeng <puck.chen@...ilicon.com>
To: Mark Brown <broonie@...nel.org>
CC: <lee.jones@...aro.org>, <lgirdwood@...il.com>,
<linux-kernel@...r.kernel.org>, <yudongbin@...ilicon.com>,
<saberlily.xia@...ilicon.com>, <suzhuangluan@...ilicon.com>,
<kong.kongxinwei@...ilicon.com>, <xuyiping@...ilicon.com>,
<z.liuxinliang@...ilicon.com>, <weidong2@...ilicon.com>,
<w.f@...wei.com>, <puck.chen@...mail.com>,
<shimingxing@...ilicon.com>, <oliver.fu@...ilicon.com>,
<albert.lubing@...ilicon.com>, <chenxiang9@...wei.com>,
<liuzixing@...wei.com>, <haojian.zhuang@...aro.org>,
<qijiwen@...ilicon.com>, <peter.panshilin@...ilicon.com>,
<dan.zhao@...ilicon.com>, <linuxarm@...wei.com>,
<dev@...ts.96boards.org>
Subject: Re: [PATCH v6 0/5] Add Support for Hi6220 PMIC Hi6553 MFD Core
On 2016/1/20 19:46, Mark Brown wrote:
> On Wed, Jan 20, 2016 at 11:51:27AM +0800, chenfeng wrote:
>
>> I will change it like this:
>> bindings: regulator: hi655x: Document for hi655x regulator driver
>
> As we discussed previously I'd expected something more like:
>
> regulator: hi655x: Document for hi655x regulator driver
>
ok
>> arm64: dts: hisilicon: Add hi655x pmic dts node
>
> That's what I'd expect.
>
>> Have you also reviewed the regulator driver code? I changed it to
>> of_match and regulators_node without opening code.
>
> Not yet.
>
Powered by blists - more mailing lists