Received-SPF: pass (google.com: domain of linux-kernel+bounces-56358-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
From: "Miclaus, Antoniu" <Antoniu.Miclaus@analog.com>
To: Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Chris Packham
	<Chris.Packham@alliedtelesis.co.nz>
CC: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        "robh+dt@kernel.org"
	<robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org"
	<krzysztof.kozlowski+dt@linaro.org>,
        "conor+dt@kernel.org"
	<conor+dt@kernel.org>,
        "jdelvare@suse.com" <jdelvare@suse.com>,
        "linux@roeck-us.net" <linux@roeck-us.net>,
        "noname.nuno@gmail.com"
	<noname.nuno@gmail.com>,
        "linux-rtc@vger.kernel.org"
	<linux-rtc@vger.kernel.org>,
        "devicetree@vger.kernel.org"
	<devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>,
        "linux-hwmon@vger.kernel.org"
	<linux-hwmon@vger.kernel.org>,
        Ibrahim Tilki <Ibrahim.Tilki@analog.com>,
        "Arslanbenzer, Zeynep" <Zeynep.Arslanbenzer@analog.com>
Subject: RE: [PATCH v6 2/2] dt-bindings: rtc: add max313xx RTCs
Thread-Topic: [PATCH v6 2/2] dt-bindings: rtc: add max313xx RTCs
Thread-Index: 
 AQHaVYLwKZaLnt2to0CJhEutu94e97D2uKUAgAcP3QCAAA7jgIAAB/oAgAAJ8ACAAMAXAA==
Date: Wed, 7 Feb 2024 09:54:45 +0000
Message-ID: 
 <CY4PR03MB3399844E78F84B6CFA18280A9B452@CY4PR03MB3399.namprd03.prod.outlook.com>
References: <20240202025241.834283-1-chris.packham@alliedtelesis.co.nz>
 <20240202025241.834283-3-chris.packham@alliedtelesis.co.nz>
 <aecd80a3-a017-405f-b77d-6deda67ef704@linaro.org>
 <5d4b7fa1-5cc2-4a4a-8fa4-d2c7a8d070b7@alliedtelesis.co.nz>
 <20240206211237d9192660@mail.local>
 <e7a21789-9253-4185-98ed-e335d0167df4@alliedtelesis.co.nz>
 <20240206221644f524816e@mail.local>
In-Reply-To: <20240206221644f524816e@mail.local>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 
 =?us-ascii?Q?5Oh4FUfku7gnkUvK05tkREM2M1yCptbra7k3XKXb9fG7xT7uX5djmMPFhEwy?=
 =?us-ascii?Q?lLwbjGGe8jKTQUjRobsFuw4zt3qDjhjEy9WNhivai0kYi0fTRpuPld3w+/Fm?=
 =?us-ascii?Q?4RtzTTZrXVxSCFVcC/yLbp45K3Nri6utT3Y0rDN3Otfg1OaDkJ4K1bk7zxh+?=
 =?us-ascii?Q?dEvvex/omuR3RVuZSvuEh4Ylk2PYOL3i4zmQ2YmKWiMfq2XsdvaqB0gA1+JW?=
 =?us-ascii?Q?XvUgwT+ULEsB5NZ4zjaf4nOnTCe8tgquLTR7i7V2tH7II59hjetjbJYpHrOy?=
 =?us-ascii?Q?TpRRNZ63fo+3edczjZVnQYupJ+hTxyvwhM1YsRpk1TcR5HjWzfv7BJHOUdMd?=
 =?us-ascii?Q?MLPrItdyNHBEcrxmZI1lDMT8jiyel/UdH+sam4TwpZWsL9GQ10fwyZhMG1Qz?=
 =?us-ascii?Q?qTv9jqoqlZQbthbaoK2u5a0KZv0BMbnCuiZcyo9seNnrD5kEmbMkxZi3aRv9?=
 =?us-ascii?Q?A+j2ARds/E6I/bVS95+0NPF4dC8G4ifJ3iLayhPwzLTfjg/62u9LplHf1XPn?=
 =?us-ascii?Q?2kkAxeORkjHhGO6v1/DaHVLnQBLAd2iidBlb52ubnSd0vjLzVAyv8P3WDSKP?=
 =?us-ascii?Q?MDXfZi+c9UFAtUT/v1RBT5ZprjJAyPj952b67LHdz02R3HD91VTSaCNuIiyJ?=
 =?us-ascii?Q?Hb7f/lHE9d6S4Lj+5KbJExmzLDlpDga1Tnzh8eEFME/3sNCiiYN6SP1C6iRU?=
 =?us-ascii?Q?UelLRrw2D33mv1G0otBQ6WMRv6ZON5W3nl59EGKyhJLLjUAPn+Lh7y2GXGC9?=
 =?us-ascii?Q?1RJp+tE+oXw8ZtygBKYIYB4hu/eMS194ZZb5FYIr/a9GAyvkfgj5I4Furv4N?=
 =?us-ascii?Q?JO96WHc95bvEtkp1Jn3MvIK2Q4eIA/BUCmgCAWV3mtZKhLEixhUyK7VK9BLl?=
 =?us-ascii?Q?vPKLeeEKJeElwQtosapXDPlbD355CRdyRmHTTaDxBXO2Mz8fn+ZFchZU+psF?=
 =?us-ascii?Q?gn0elPgeK+EkF4Pz202p02T9y9QpV5mLLBE1GRJ5nBJuEEP3HwuZwmALmGEV?=
 =?us-ascii?Q?UrwgwnX6wVzNYyOIavW7kOr3wSHb+yXwkA3Cjj1Re2KU4QfolKuBIilC/4+A?=
 =?us-ascii?Q?/JIkfFV+NiJhxq6gYc9usqKswr/jJze+rg1FMlHIywD6h2/mAK7fEVYoikz/?=
 =?us-ascii?Q?5SLuq27Arzn5Et6/a6Hm36+nmORlfOWXMAy8SteFx9lbW7rY7KjIRbJoLVia?=
 =?us-ascii?Q?VVAVN/JrDboYS9nVlY36mKzyaVNOa0n4Z95tmMi0YpC2BFjrAsr2jvkhRHzR?=
 =?us-ascii?Q?c49Cbhpm54TdC2EgEG1rJyCI4hEC9lZ05PCAlf+JWpp4t7fUlCBA97TEVoru?=
 =?us-ascii?Q?dl4CcrP+M6Z0+pb8YSbU+sD+1wKF2ZksVwfbyoP5von+t42AtYw+eOA+EcWg?=
 =?us-ascii?Q?50kyD7nkZN5+WOr7EPc4kii/8++K53tMvU1zYSvDJi90zuPCYNBVBC0wojI5?=
 =?us-ascii?Q?/ag/2sbWYT9HqV+eF3C7EfD6bq1zrUfdSdk3ADopAlTWMdhTZG6dKgzbxazg?=
 =?us-ascii?Q?8kaAIoxFOaOj9ZIIkvc48wC0+m3HWbcbrsMC2Tmizan4XvMj0jjcYrcb4HaI?=
 =?us-ascii?Q?ktnQP6bxEfAsAcTpcjcitbd8fHZxj8c5KRdlOOT4EnUgEqBEZbxny457WH2J?=
 =?us-ascii?Q?pX6RabZRbtG/7xNvEWttshubewQK0wbYi9R5ZxRGJDHm?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
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: b47a49bd-36d9-49fd-314e-08dc27c2d0bb
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Feb 2024 09:54:45.8500
 (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: 0B5QyzWv/evhNC5czZSn+NMd2EQhQJR0Lh3YIyoeWivof/OpiKJZu4pUgbjnT+BCG1A43pmsB3+D28BdKb4wN+WfXMWMAidLdJ1UoYUN6dA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR03MB4975
X-Proofpoint-GUID: rVBKUgEQQ221E6YrwThhS1c2AwcsJkbI
X-Proofpoint-ORIG-GUID: rVBKUgEQQ221E6YrwThhS1c2AwcsJkbI
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-02-07_04,2024-01-31_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 bulkscore=0 phishscore=0
 suspectscore=0 adultscore=0 clxscore=1011 lowpriorityscore=0
 malwarescore=0 mlxlogscore=742 priorityscore=1501 spamscore=0
 impostorscore=0 mlxscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.19.0-2401310000 definitions=main-2402070073

> On 06/02/2024 21:41:10+0000, Chris Packham wrote:
> >
> > On 7/02/24 10:12, Alexandre Belloni wrote:
> > > On 06/02/2024 20:19:20+0000, Chris Packham wrote:
> > >> That is an incredibly good point. The max31335 binding covers one
> specific chip. This binding covers more and with that there are a few mor=
e
> properties that the max31335 on it's own doesn't have (e.g. the clock
> consumer, the ability to have different i2c addresses). Binding wise I co=
uld
> probably roll all of the max31335 into this max313xx binding.
> > >>
> > >> Driver wise things are a bit trickier. I've only got access to one o=
f
> > >> the variants so I am hoping to leverage the work Ibrahim had already
> > >> done. I could attempt to incorporate max31335 support into the
> > >> max313xx driver but I wouldn't really be able to test it properly an=
d
> > >> there is a reasonably high chance of regressing something.
> > > But I won't take a separate driver. Everything would be better if Ana=
log
> > > was sharing the datasheets...
> >
> > The datasheets are pretty accessible so I'd give Analog a pass on that
> > (they're certainly better than some vendors). I'll include some links o=
n
> > the next update.
> >
>=20
> The max31335 is not available
>
=20
Indeed, the max31355 datasheet is not available yet. This is also stated in=
 the
cover letter of the patch series for max31335.

It was an urgent request from a customer to have it mainline as soon as pos=
sible.

If there are questions about the part functionality, I can definitely help.

> > I'll attempt to roll the max31335 into the max313xx driver.
>=20
> I guess this would be the opposite. Renaming a driver breaks existing
> kernel configurations.
>=20
> --
> Alexandre Belloni, co-owner and COO, Bootlin
> Embedded Linux and Kernel engineering
> https://urldefense.com/v3/__https://bootlin.com__;!!A3Ni8CS0y2Y!7A0VM
> Nj1SLsNWZ8SnPrFwG6KX2TqjeLQi38EYIV164_5MWlQ7yCFvp8yOjaswIRNLC
> 0EK-_bhHEfy6xAXEJimVEW8BZXdn_r$