linux-kernel - Re: [PATCH v4] mfd: lp87565: Add lp87565 PMIC support

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <373e82d1-39fb-9cbb-300a-aa8a99544cdc@ti.com>
Date:   Mon, 12 Jun 2017 14:47:03 +0530
From:   Keerthy <j-keerthy@...com>
To:     Lee Jones <lee.jones@...aro.org>
CC:     Rob Herring <robh@...nel.org>, <broonie@...nel.org>,
        <linux-omap@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <mark.rutland@....com>
Subject: Re: [PATCH v4] mfd: lp87565: Add lp87565 PMIC support



On Monday 12 June 2017 02:41 PM, Lee Jones wrote:
> On Sun, 11 Jun 2017, Keerthy wrote:
> 
>>
>>
>> On Sunday 11 June 2017 10:36 AM, Keerthy wrote:
>>>
>>>
>>> On Friday 09 June 2017 07:58 PM, Rob Herring wrote:
>>>> On Thu, Jun 08, 2017 at 09:38:14AM +0530, Keerthy wrote:
>>>>> The LP87565 chip is a power management IC for Portable Navigation Systems
>>>>> and Tablet Computing devices. It contains the following components:
>>>>>
>>>>>         - Configurable Bucks(Single and multi-phase).
>>>>>         - Configurable General Purpose Output Signals (GPO).
>>>>>
>>>>> The LP87565-Q1 variant device uses two 2-phase outputs configuration,
>>>>> Buck0 is master for Buck0/1 output and Buck2 is master for Buck2/3
>>>>> output.
>>>>>
>>
>> Lee Jones,
>>
>> Shall i add back i2c_device_id as pointed here:
>> http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1417316.html
> 
> Hasn't that been fixed yet?
> 
> I guess so then. :(

Okay. So with that i assume i should reintroduce probe instead of probe_new.

> 
>>>>> Signed-off-by: Keerthy <j-keerthy@...com>
>>>>> ---
>>>>>
>>>>> The other patches are already pulled by Mark. Hence posting
>>>>> the remaining patch.
>>>>>
>>>>> Changes in v4:
>>>>>
>>>>>   * Fixed device tree comments from Rob on the pmic node name.
>>>>>   * removed regulators from properties list as it is a node.
>>>>>  
>>>>> Changes in v3:
>>>>>
>>>>>   * Fixed License to GPL v2.
>>>>>   * Fixed an indentation issue.
>>>>>
>>>>> Changes in v2:
>>>>>
>>>>>   * Fixed a bunch of whitespace errors.
>>>>>   * Changed the License to short form.
>>>>>   * Added the generic compatible lp87565
>>>>>   * Removed i2c_device_id table.
>>>>>   * Introduced probe_new function in place of probe.
>>>>>
>>>>>  Documentation/devicetree/bindings/mfd/lp87565.txt |  43 ++++
>>>>
>>>> Acked-by: Rob Herring <robh@...nel.org>
>>>
>>> Thanks Rob.
>>>
>>>>
>>>>>  drivers/mfd/Kconfig                               |  14 ++
>>>>>  drivers/mfd/Makefile                              |   1 +
>>>>>  drivers/mfd/lp87565.c                             |  92 ++++++++
>>>>>  include/linux/mfd/lp87565.h                       | 270 ++++++++++++++++++++++
>>>>>  5 files changed, 420 insertions(+)
>>>>>  create mode 100644 Documentation/devicetree/bindings/mfd/lp87565.txt
>>>>>  create mode 100644 drivers/mfd/lp87565.c
>>>>>  create mode 100644 include/linux/mfd/lp87565.h
>