On 08/10/2018 01:57 PM, Simon Horman wrote:
> On Tue, Aug 07, 2018 at 10:16:16AM +0200, Geert Uytterhoeven wrote:
>> Hi Marek,
>>
>> On Sat, Aug 4, 2018 at 6:38 PM Marek Vasut <marek.va...@gmail.com> wrote:
>>> Add DA9063 PMIC node to the I2C bus.
>>>
>>> Signed-off-by: Marek Vasut <marek.vasut+rene...@gmail.com>
>>> Cc: Geert Uytterhoeven <geert+rene...@glider.be>
>>> Cc: Kuninori Morimoto <kuninori.morimoto...@renesas.com>
>>> Cc: Simon Horman <horms+rene...@verge.net.au>
>>> Cc: Wolfram Sang <wsa+rene...@sang-engineering.com>
>>> Cc: linux-renesas-soc@vger.kernel.org
>>> ---
>>> V2: - Replace shmobile with dts since it's a DT patch in subject
>>>     - Connect the 9063_IRQ# line to GP3_31
>>>     - Since the DA9063 is connected to both i2c1 and i2c7, connect
>>>       it to i2c7, which is the dedicated dvfs i2c.
>>
>> Thanks for the update!
>>
>>> --- a/arch/arm/boot/dts/r8a7794-silk.dts
>>> +++ b/arch/arm/boot/dts/r8a7794-silk.dts
>>> @@ -405,6 +405,23 @@
>>>         clock-frequency = <400000>;
>>>  };
>>>
>>> +&i2c7 {
>>> +       status = "okay";
>>> +       clock-frequency = <100000>;
>>> +
>>> +       pmic@58 {
>>> +               compatible = "dlg,da9063";
>>> +               reg = <0x58>;
>>> +               interrupt-parent = <&gpio3>;
>>> +               interrupts = <31 IRQ_TYPE_LEVEL_LOW>;
>>> +               interrupt-controller;
>>> +
>>> +               wdt {
>>> +                       compatible = "dlg,da9063-watchdog";
>>> +               };
>>
>> Given Silk has the full da9063 (unlike the "L" version on Porter), shouldn't
>> you add an rtc subnode?
>>
>> Oh, Silk also has the onkey pin wired, so perhaps you want to add an onkey
>> subnode, too? Does that feature work?
>>
>> BTW, Stout also has the onkey wired, but lacks the onkey subnode.
>>
>>> +       };
>>> +};
>>> +
>>
>> Nevertheless:
>> Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be>
> 
> Thanks, I have applied this patch for v4.20.
> 
> Marek,
> 
> please consider some follow-up patches to address the issues
> raised by Geert.

Thanks for the reminder, I missed the email. Patch is out.

-- 
Best regards,
Marek Vasut

Reply via email to