Re: [PATCH 1/3] regulator: max14577: Add proper dt-compatible strings

On Fri, Feb 21, 2020 at 11:44:03AM +0100, Marek Szyprowski wrote:

On 20.02.2020 17:56, Mark Brown wrote:

quoted

Why would we want to encode the particular way Linux happens to
represent regulators on a MFD into the DT binding?  It's not clear that
this is a generic thing (another OS might choose to have a separate
object for each regulator with no parent for example) and the compatible
isn't adding any information we didn't have already knowing about the
parent device.

Well, that's how the bindings for max14577/max77836 are defined:

Documentation/devicetree/bindings/mfd/max14577.txt

I've only fixed regulator, charger and extcon drivers to match the cells 
created by the current mfd driver.

We could just remove the compatible strings from the binding
documentation, they won't do any harm if we don't use them.