Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 27 May 2021 14:03:32 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Peter Geis <pgwipeout@gmail.com>
Cc:     Rudi Heitbaum <rudi@heitbaum.com>, devicetree@vger.kernel.org,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        "open list:ARM/Rockchip SoC..." <linux-rockchip@lists.infradead.org>,
        Ezequiel Garcia <ezequiel@collabora.com>, chenjh@rock-chips.com
Subject: Re: [PATCH] regulator: fan53555: add back tcs4526
Message-ID: <YK+YpBm9M2GbFGec@sirena.org.uk>
References: <20210526162342.GA20@8bbba9ba63a4>
 <CAMdYzYpZoKs3P62j02RW-+5BEpqC9JL3apjucTWLWmvNFrOrCg@mail.gmail.com>
 <20210527105943.GA441@7698f5da3a10>
 <CAMdYzYq8c4=D6_LxG2jPfbtPkOp5tN8oRA0zKt_OM6HLbnfFTw@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="nVOdrmUDn/PExWf2"
Content-Disposition: inline
In-Reply-To: <CAMdYzYq8c4=D6_LxG2jPfbtPkOp5tN8oRA0zKt_OM6HLbnfFTw@mail.gmail.com>
Precedence: bulk


--nVOdrmUDn/PExWf2
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Thu, May 27, 2021 at 07:26:01AM -0400, Peter Geis wrote:
> On Thu, May 27, 2021 at 6:59 AM Rudi Heitbaum <rudi@heitbaum.com> wrote:

> > > Since you aren't adding any functional code, is there a particular
> > > reason you can't just add the chip id and simply use the tcs4525
> > > compatible?
> > > This will prevent you from needing to modify the dt-bindings as well.

..

> > I chose to follow the example of silergy,syr827 and silergy,syr828 for
> > tcs4526 (given I made the mistake in assuming that support for tcs4525
> > meant support for tcs4525.) This would maintain consistency of naming of
> > tcs4526 throughout the source. Is that ok?

> It's fine to have both compatibles (and avoids confusion in
> device-trees), just remember to update the dt-bindings as well.
> It's funny to see drivers with both schemes, so we really have to
> decide which path we want to go down.
> Considering the syr827/syr828 as convention, we should probably just
> go down that route for consistency within the driver.

It is generally safer for the DT to be explicit about exactly
what the hardware is and then double check that this matches the
actual hardware, this gives more room for handling things with
quirks if needed in future and makes the life of people writing
DTs for boards easier since they don't need to remap part
numbers from the schematic to the DT.

> > +&gpu {
> > +       mali-supply = <&vdd_gpu>;
> > +       assigned-clocks = <&cru ACLK_GPU>;
> > +       assigned-clock-rates = <200000000>;
> > +       status = "okay";
> > +       /delete-property/ operating-points-v2;

> Removal of the operating points kind of makes the gpu regulator moot,
> don't you think?

It's still better to say what the supply is even if it can't be
changed - that stops you getting warnings about substituting in a
dummy regulator and allows the consumer to read the current state
of the regulator in case that's useful.

--nVOdrmUDn/PExWf2
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmCvmKAACgkQJNaLcl1U
h9AS5gf+MyXoR1SRd4piOK0UplV96s3jVB5W+Nu4kZPj7iNbaUP7Mpj/yHR6Bm53
rjwUbCpnDJpt+Kjf3uJrSTHim8VYXidXiTykeEYyZTX9SFPRELLatDCd+lxyJSBT
aAGGOMyOX0tQR8+0IqHjS3iFX4/C7KoUcDVeMRILQGJj3x+QGZonSFssqrjjOnbv
L1LE/qJ/AhA8MNYdh027mDk2GcT6Pkv7nVTrf0qubDJNffLVA6NwjR3/xe4IKBph
DqWIj4LZeJGGH8XLp5yOab05BOvBQRFE0UX117GRvfvogOuOSmSzS/EeWS5sC9tS
YLW8k4v4p1yHrpDNwQ4YsE4X6XcorQ==
=smBU
-----END PGP SIGNATURE-----

--nVOdrmUDn/PExWf2--