Re: [PATCH v2 01/10] dt-bindings: mfd: common ROHM PMIC properties

"Rob Herring (Arm)" <[email protected]>
Newsgroups org.kernel.vger.linux-rtc,dev.linux.lists.mfd,org.kernel.vger.linux-clk,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-gpio,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
On Tue, 04 Aug 2026 13:20:20 +0300, Matti Vaittinen wrote:
> From: Matti Vaittinen <[email protected]>
> 
> ROHM PMICs keep having similar functionalities from one generation to the
> next. Keeping the device-trees describing these functionalities similar
> helps users because:
>   1. They can re-use existing device-tree properties when switching from
>      one project, using a ROHM PMIC, to next project using newer
>      variant.
>   2. The users can find and re-use examples from old projects - even if
>      they never worked with ROHM PMICs before.
> Similarly, introducing new and different properties for same
> functionality previous variant had, will be adding confusion.
> 
> Sometimes the existing properties aren't 100% what reviewers would
> prefer. When issues are minor or just cosmetic, changing the existing
> properties is not feasible. Reviewers can't be expected to know which
> properties are new, and which are existing - and this can lead to
> unnecessary review discussion wasting time and energy from everyone.
> 
> Adding a common file for re-used ROHM PMIC properties should make it
> clearly visible that a property is re-used, when a new PMIC binding
> refers to this file instead of (re-)describing all the existing
> bindings. This will also help keeping the common properties identical
> across the variants.
> 
> Add new file listing commonly used ROHM PMIC properties.
> 
> Signed-off-by: Matti Vaittinen <[email protected]>
> 
> ---
> Revision history:
>  v2:
>   - New patch
> ---
>  .../bindings/mfd/rohm,pmic-pins.yaml          | 72 +++++++++++++++++++
>  1 file changed, 72 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mfd/rohm,pmic-pins.yaml
> 

Reviewed-by: Rob Herring (Arm) <[email protected]>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.