Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.36 as permitted sender) client-ip=23.128.96.36;
From:   "Miclaus, Antoniu" <Antoniu.Miclaus@analog.com>
To:     Rob Herring <robh@kernel.org>
CC:     "Matyas, Daniel" <Daniel.Matyas@analog.com>,
        Jean Delvare <jdelvare@suse.com>,
        Guenter Roeck <linux@roeck-us.net>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        "linux-hwmon@vger.kernel.org" <linux-hwmon@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 1/2] dt-bindings: hwmon: max31827: use supply pin name
Thread-Topic: [PATCH 1/2] dt-bindings: hwmon: max31827: use supply pin name
Thread-Index: AQHZ7JbW8bVvOja/3EqvTv/jlHNtWLAnWa2AgAEcYAA=
Date:   Sat, 23 Sep 2023 14:19:45 +0000
Message-ID: <CY4PR03MB3399B818487F87D7297EC33F9BFEA@CY4PR03MB3399.namprd03.prod.outlook.com>
References: <20230921142005.102263-1-antoniu.miclaus@analog.com>
 <20230922211201.GA3600661-robh@kernel.org>
In-Reply-To: <20230922211201.GA3600661-robh@kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?wMkRZ+TAbDIIfTZK+bbMgLyDNOGnMkDRbM4ku190aK65POPH/M9rm50KVWlp?=
 =?us-ascii?Q?Wi9IhL0ihCtjU9UW97tFPQQABp5a3j4pqjU68AukhIeB7bshbiOXacCgfxGn?=
 =?us-ascii?Q?/n9/tACVXhmjJuc9ajukJ9cTCzeF+HpcLPCVtXbaWigwGg5HOhqezXDNFoC/?=
 =?us-ascii?Q?RpE3iSECZ/uRAxqwCVnhxkBJ3MrCxrhKcPG1gR3UFAFmRKto/3GzbkbIcJsX?=
 =?us-ascii?Q?+LpX1sLTjW2cuyfluBvjpHsNlJNofWwy5bAChaTXr2AvpPHU7Xyj7kZoM6h9?=
 =?us-ascii?Q?5Ud8oXZ8tNTgHc88Pm79U/Wxy2OQJtORIu5ZwkiugEZhnQsT+Gc9sCTsDhoE?=
 =?us-ascii?Q?L2gaYp0ZmWChhvQIW5zYpqTVGrYWOie6bHpQ5B4nnEaALHbJ1IP+Th1DXtHx?=
 =?us-ascii?Q?rfX9JDNXHihUjcHUhlGP89ljQ2n/1f7mYvohwXlhrSp9sdM4+IodBk37dCdn?=
 =?us-ascii?Q?cDZr+Y5hR9MtOk4QlRwe0S2hhFFmPPkjlq5IAb6crlV21YcZWTZx8vW2Tise?=
 =?us-ascii?Q?eLC4fjUZEWxswqz5MQyQE4qCENZW0KX6LJrLARDK0ny7mj9Okt/ANZ6tWrJ7?=
 =?us-ascii?Q?7c/mYljG0ersDWGllfjxiqwMFJwYihou+39HaSE910Qt+p21LAfUTlislpIm?=
 =?us-ascii?Q?6xK3QwRi3AmemQ6yFq2Uhcb7PbBwUVU140bewnLbrTyK2zUCb6JJ77Urka6u?=
 =?us-ascii?Q?j1gi2BERM1CiyDD8et5RLfnqhVjy0Dd2aW37JUyUGGyM8+ezzWBKWcXOfVvH?=
 =?us-ascii?Q?SJIESNpOKjntLn9hkExMsfS9lldp/T7FvDHQ2DmAUutlN5RqRs5eTEHutf4H?=
 =?us-ascii?Q?slZtMY2zAlSpFog4k2KjIu1prA5OtRXbFtUGxFDbJ+3MzR2oevGtomFR5bRo?=
 =?us-ascii?Q?JqFhkG5DKvSIGnqxV37KhFqui8Dx+qeBDq5YMWa3ZdsDDDbv2EcEzahMI/76?=
 =?us-ascii?Q?klE6nztxgtJ8ni/sPAk119pQaA1OmZdy2pgtBc//YZmQXO738b3SETy0SLOL?=
 =?us-ascii?Q?edVWk+lHNzVZGtsFHf0YMee7uNWfc0S7bSu1hiMALJZVYYol29dtabiy5YqS?=
 =?us-ascii?Q?koYNEywH4u7Q+YltKqscJJvAZoTb28WfJ/5s02k0/m6gdG9m3E/Xv8hqPBs8?=
 =?us-ascii?Q?3YzbCgdlzNFcj/rsZntPVzMiHqKR8KKSvkM4FHQEz57Lt/R/z2ojc3i5Cp0d?=
 =?us-ascii?Q?+rHEjXiIMIL00fUZg6eM5sXoOAHy8UFI6MWEs+jSsXqUuvMOrdggtc/SF870?=
 =?us-ascii?Q?klbBoG+vM5g0cR+jMOsvZQhT3szHK/kGGoApSNvrUju10uhBcrqNV5mTAAUG?=
 =?us-ascii?Q?5KkinX+pZfePhQ9HKQNb3Q/9qwFVdUF6NRMjXz0LmHpAhK68XxN12PDche/b?=
 =?us-ascii?Q?okzTPytvYS0bLZJ2gq37mybtODXmY+z/RbNmzDgqEiFsWWvuOoU/02OQ/Nxk?=
 =?us-ascii?Q?TY6SWm1mab+o2KMlX/P9BOMgtXabhARWesUGKX6LwGXkaxhxpK/K32r95BT5?=
 =?us-ascii?Q?hUf6AeYz1mnBZXUmKIhYXN+eLiPcKKbiJdR46IpZxbkIpBgHll+K2R5JVjuc?=
 =?us-ascii?Q?BDYrpxlUrtEYw2zYnCXpQbcU5mN2+Tmgs2oiW7CbsJx8aU2hvkSOsiQ+sWmb?=
 =?us-ascii?Q?lEf9iF9ClaCAk/ahnqueZ7woeRtTDsENR3zTtXon+Gc3?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: analog.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CY4PR03MB3399.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c83a3c4f-313a-4db4-8a85-08dbbc402357
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Sep 2023 14:19:45.9871
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: eaa689b4-8f87-40e0-9c6f-7228de4d754a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: ZZoxgGKT5OaE2mibNyzlO8rlNxGS91b8kJ7iZobJklOMM5Puf0hd/zq6ywpzjZdQ6HLwSftLhBrXc3rS7Delw22Qk+9DTGt3HOPMwkLx4+c=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR03MB5953
X-Proofpoint-ORIG-GUID: TGC7QV6jlwMHs6QelDgeysT-UZWCg4Cn
X-Proofpoint-GUID: TGC7QV6jlwMHs6QelDgeysT-UZWCg4Cn
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.267,Aquarius:18.0.980,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2023-09-23_12,2023-09-21_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 lowpriorityscore=0
 bulkscore=0 spamscore=0 malwarescore=0 suspectscore=0 adultscore=0
 phishscore=0 mlxlogscore=999 mlxscore=0 impostorscore=0 priorityscore=1501
 clxscore=1011 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.19.0-2309180000 definitions=main-2309230119
X-Spam-Status: No, score=2.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RCVD_IN_SBL_CSS,
	SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on pete.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (pete.vger.email [0.0.0.0]); Sat, 23 Sep 2023 07:20:31 -0700 (PDT)
X-Spam-Level: **


> On Thu, Sep 21, 2023 at 05:20:03PM +0300, Antoniu Miclaus wrote:
> > The actual hardware pin name for the supply of max31827 is vdd.
> > Update the dt-binding to reflect the hardware properties accordingly.
>=20
> Changing this breaks the ABI. I see the old one wasn't used by the
> driver, but that's just one driver potentially. You need some
> justification here why it's okay to break the ABI.
>=20
As I mentioned also in the commit description, the supply should match the
actual hardware pin name. Otherwise it might create confusion. Usually vref
refers to an external voltage reference pin used for ADC/DACs which is not
exactly the case for this part, taking into account that there is no "refer=
ence"
word mentioned in the datasheet at all. VREF and VDD are usually separate
hardware pins. There is a hint indeed in the dts example that the vref-supp=
ly=20
might be referenced to a vdd regulator node, but from my point of view
that is not enough. Moreover the current vref-supply is not handled at all =
in
the driver, it is only mentioned in the dt-binding (That's why I added a se=
cond
patch in the series handling the supply).

If the justification is not enough to apply this change, then I can keep on=
ly the
second patch, which handles the regulator in the driver and use the  old `v=
ref`
naming which currently appears only in the dt-binding.

Antoniu
> >
> > Signed-off-by: Antoniu Miclaus <antoniu.miclaus@analog.com>
> > ---
> >  Documentation/devicetree/bindings/hwmon/adi,max31827.yaml | 6 +++--
> -
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git
> a/Documentation/devicetree/bindings/hwmon/adi,max31827.yaml
> b/Documentation/devicetree/bindings/hwmon/adi,max31827.yaml
> > index 2dc8b07b4d3b..21f2d350373b 100644
> > --- a/Documentation/devicetree/bindings/hwmon/adi,max31827.yaml
> > +++ b/Documentation/devicetree/bindings/hwmon/adi,max31827.yaml
> > @@ -27,7 +27,7 @@ properties:
> >    reg:
> >      maxItems: 1
> >
> > -  vref-supply:
> > +  vdd-supply:
> >      description:
> >        Must have values in the interval (1.6V; 3.6V) in order for the d=
evice to
> >        function correctly.
> > @@ -35,7 +35,7 @@ properties:
> >  required:
> >    - compatible
> >    - reg
> > -  - vref-supply
> > +  - vdd-supply
> >
> >  additionalProperties: false
> >
> > @@ -48,7 +48,7 @@ examples:
> >          temperature-sensor@42 {
> >              compatible =3D "adi,max31827";
> >              reg =3D <0x42>;
> > -            vref-supply =3D <&reg_vdd>;
> > +            vdd-supply =3D <&reg_vdd>;
> >          };
> >      };
> >  ...
> > --
> > 2.42.0
> >