linux-kernel - Re: [PATCH 0/6] mfd: 88pm800: Add 88pm860 device support

Open Source and information security mailing list archives

Message-ID: <CAJKOXPetR4KkfRVNz_QtVfFnjQ7p455QknErr1gnK9iBYgRY+A@mail.gmail.com>
Date:	Thu, 9 Jul 2015 21:04:27 +0900
From:	Krzysztof Kozlowski <k.kozlowski@...sung.com>
To:	Vaibhav Hiremath <vaibhav.hiremath@...aro.org>
Cc:	linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
	Kozik <k.kozlowski@...sung.com>, sameo@...ux.intel.com,
	linux-kernel@...r.kernel.org, robh+dt@...nel.org,
	lee.jones@...aro.org
Subject: Re: [PATCH 0/6] mfd: 88pm800: Add 88pm860 device support

2015-07-09 20:47 GMT+09:00 Vaibhav Hiremath <vaibhav.hiremath@...aro.org>:
> 88PM860 falls under 88pm800 family of devices, with some feature
> additions, for example, support for dual phase on BUCK1.

Hi,

There are already drivers for 88PM860: mfd, charger and battery. Isn't
you duplicating existing code?

Best regards,
Krzysztof

>
> This patch series, enabled chip ID support for 88pm860 in the driver
> and adds,
>   - Init time configuration support based on chip ID
>   - DT property for 32KHz low jitter clock enable
>   - DT property for dual phase enable
>
> TODO:
>  - Some of init time configurations are common to both 88pm800 and 88pm860
>    devices, but since I can not validate it, decided to move it to
>    88pm860 block only.
>    But if someone is willing to help me in validation, we can move it to
>    common code later. No issues there.
>  - Sleep/low-power-mode related configuration is also part of init time,
>    but since we are too far from sleep mode support and without testing
>    I do not want add anything, decided to take it later when we actually
>    start looking at sleep support.
>  - Init time configuration also includes pinmux setting for the device.
>    I am working on using pinctrl-single driver to have standard and generic
>    interface, hopefully it will get handled through pinctrl subsystem.
>    Link to RFC - https://patches.linaro.org/50604/
>
> Vaibhav Hiremath (6):
>   mfd: 88pm80x: Add 88pm860 chip type support
>   mfd: 88pm800: Add init time initial configuration support
>   mfd: devicetree: bindings: 88pm800: Add DT property for 32KHz output
>     enable
>   mfd: 88pm800: Enable 32KHZ XO low jitter clock out
>   mfd: devicetree: bindings: 88pm800: Add DT property for dual phase
>     enable
>   mfd: 88pm800: Add support for configuration of dual phase on BUCK1
>
>  Documentation/devicetree/bindings/mfd/88pm800.txt | 12 ++++
>  drivers/mfd/88pm800.c                             | 88 +++++++++++++++++++++++
>  drivers/mfd/88pm80x.c                             |  2 +
>  include/linux/mfd/88pm80x.h                       | 18 +++++
>  4 files changed, 120 insertions(+)
>
> --
> 1.9.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@...ts.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
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/

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