Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Tilki, Ibrahim" <Ibrahim.Tilki@analog.com>
To:     Guenter Roeck <linux@roeck-us.net>,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
CC:     "a.zummo@towertech.it" <a.zummo@towertech.it>,
        "alexandre.belloni@bootlin.com" <alexandre.belloni@bootlin.com>,
        "jdelvare@suse.com" <jdelvare@suse.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "linux-rtc@vger.kernel.org" <linux-rtc@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-hwmon@vger.kernel.org" <linux-hwmon@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "Arslanbenzer, Zeynep" <Zeynep.Arslanbenzer@analog.com>
Subject: RE: [PATCH 2/2] dt-bindings: rtc: add bindings for max313xx RTCs
Thread-Topic: [PATCH 2/2] dt-bindings: rtc: add bindings for max313xx RTCs
Thread-Index: AQHY48BZbMpT/SjNa0SzYqltq8qQ864VyDiAgAAQ9QCAAX5sEA==
Date:   Thu, 20 Oct 2022 14:25:30 +0000
Message-ID: <DM8PR03MB6247FE3C128DE9FE5BF6F591962A9@DM8PR03MB6247.namprd03.prod.outlook.com>
References: <20221019133910.282-1-Ibrahim.Tilki@analog.com>
 <20221019133910.282-2-Ibrahim.Tilki@analog.com>
 <3d07998b-f02a-036c-af66-883671ac1730@linaro.org>
 <20221019153456.GD4602@roeck-us.net>
In-Reply-To: <20221019153456.GD4602@roeck-us.net>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?VzeExUzNq3sfHmoGUhMoBfNdkVc97dxb9tWoyh6KDz0fFIORGDDfXkFDxoyo?=
 =?us-ascii?Q?8z3goQ3ERGVXjuODnTvNHMIMzmoY3/JK85u+sliV+evUw23OvYb+v3s6bOzb?=
 =?us-ascii?Q?jEsF05Oplw/2rLSL8LRlOGIyV/oVfpcEIcgjxFsnVItf+JCmZvVMSQmQ8s/z?=
 =?us-ascii?Q?xpo+eKMgtUi26b0oYO10WHNDdfx2USXqm9mSEJkp8UYPfkCNaCIMHruHsyiQ?=
 =?us-ascii?Q?OGAsAYm2ink3ywZDaMtR1uzEU8zQAVqrSEjwTb1SnyIN9TG51O1grXde4cWi?=
 =?us-ascii?Q?9TOICleGWrVYVoi3OZ7UfrJVVY6TFiW+2OFOJDGUm6b/hMIKNE84mT8DzHsa?=
 =?us-ascii?Q?KOLdA8/uAElzpr2yXOoM0gFpcQpNR0g4uO1WnePdhwuwT+ckF/QqE1i/L/DH?=
 =?us-ascii?Q?pq+5dMzgT/NSLvZA7XLHPldkTA7WRp2QW6aaIrzWuJSntzqca30nsv9Q6wSl?=
 =?us-ascii?Q?JEx6U+7l1Ge0TfSaPP1INb/S5nL8KhCOtr/aaHckM0lmQsHeos0ZF3U1625o?=
 =?us-ascii?Q?5Q2Wl5qBL9Qxau9MA0cLBU9lXm0Y8JUjHn3wY94EzWB13c7WbojP+wp7bcxx?=
 =?us-ascii?Q?zJ+lDfZr/GyLa+6/I8KVzPsoPGX2YVUJzRZ/BCHCp4Vz8Q/8q7zX90y9PWwE?=
 =?us-ascii?Q?w2PvCB1uh4sITlnJiwBdmGOle+r6yo0oaFk84/4cUt1mmbmEzqATmXiOcokY?=
 =?us-ascii?Q?J765i/TyuQ4/H83HqpCXm/58kcV1MCa/JgX1Bcn9MLVuZml8UN9FUkpvuPdt?=
 =?us-ascii?Q?J9Ltjm/QcAVmYU7qAx49ToDGZI2j0enpSaF4/yCuqxf/MuIzL//zGsMFexJS?=
 =?us-ascii?Q?8I1fjJ/XsphgSmvkkEiWg8uWxQTwFRrbfkeYnReJznLCce3HUBSm8A63nDiA?=
 =?us-ascii?Q?L7agLp/JrmWC1F0Sm7b6JgE2POUkenmbSUEvhHIpUzbC/JyL/n+olS4TbDp6?=
 =?us-ascii?Q?ilko6KJj22BYG8eYaTY9eei3FpOnxu9bcR5iaERVWQO9mf1zcb3QJ2XcFryY?=
 =?us-ascii?Q?cl132jwA4/i1xFsOITQrUosBMaF1bD2KRErhz+gSCPbgGk/DBGqaB5ws8KaE?=
 =?us-ascii?Q?h7lyjXxBi66zbidT8T8eHm4/M589d434FewWXVSMe82wbgdKwn30ZkljZYPS?=
 =?us-ascii?Q?vkfRIbhLCm7Awjkm3HNay/N+iHiKIniIxq2sPCAvrkpsATqfJ3E0eCJlFoS+?=
 =?us-ascii?Q?akl+fg9JuOXh0eyBiYMO6kHSB2Vq4IH9XaswUdVvDwAXtTLHHze4YiE71Tb9?=
 =?us-ascii?Q?JKrg0t+E6wDR3rAUiZ1z5YZgL6MltrDNyTNhBLgPKdZ7XmmQeam+37WxevTd?=
 =?us-ascii?Q?GtjFALCExRGBBLISPNTsj1XmdhtzwZNHp8s2E5egDltY9Cd70XjZSbe6YaMR?=
 =?us-ascii?Q?9ooLXLAMr/jQ2cW8BAm+f3yJknh8elojEltJwF28X0GwHQnx0IV+Obi/t/6/?=
 =?us-ascii?Q?BlwizNVKbHcEFPS146fy8rS0nYle0e46sg/ijb9bDCnyDTh52HuCZIEHZD+s?=
 =?us-ascii?Q?xXUSvXcCsaW8vEeNiSGBtLvu1NrlLsUvWUm59YmrFJFY0RIHVZLSEmtjsqFB?=
 =?us-ascii?Q?9JKIDcWWcxSs51f21/ORLb54UVP1uX9d++eKzIf7?=
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: DM8PR03MB6247.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 330ac615-832c-41db-f72d-08dab2a6f184
X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Oct 2022 14:25:31.2528
 (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: 1GANHdjyQ4tXNWY+/ETMJH8LLbTHWrz9hygIhTn4GxMHpRniS/q4oGbsvKhIxGjYZxadxgiiveI/E1199xorP6Ypv3aEovr19lG0tqQnL9I=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR03MB6420
X-Proofpoint-GUID: ElQE9MWClaiATxOlelCT0VA9s7gr8IXN
X-Proofpoint-ORIG-GUID: ElQE9MWClaiATxOlelCT0VA9s7gr8IXN
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.895,Hydra:6.0.545,FMLib:17.11.122.1
 definitions=2022-10-20_05,2022-10-20_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 lowpriorityscore=0
 mlxlogscore=999 spamscore=0 malwarescore=0 impostorscore=0 bulkscore=0
 phishscore=0 priorityscore=1501 suspectscore=0 clxscore=1011 mlxscore=0
 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2209130000 definitions=main-2210200085
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> On Wed, Oct 19, 2022 at 10:34:15AM -0400, Krzysztof Kozlowski wrote:
> > On 19/10/2022 09:39, Ibrahim Tilki wrote:
> > > Devicetree binding documentation for Analog Devices MAX313XX RTCs
> > >=20
> > > Signed-off-by: Ibrahim Tilki <Ibrahim.Tilki@analog.com>
> > > Signed-off-by: Zeynep Arslanbenzer <Zeynep.Arslanbenzer@analog.com>
> > > ---
> > > .../devicetree/bindings/rtc/adi,max313xx.yaml | 163 +++++++++++++++++=
+
> > > 1 file changed, 163 insertions(+)
> > > create mode 100644 Documentation/devicetree/bindings/rtc/adi,max313xx=
.yaml
> > >=20
> > > diff --git a/Documentation/devicetree/bindings/rtc/adi,max313xx.yaml =
\
> > > b/Documentation/devicetree/bindings/rtc/adi,max313xx.yaml new file mo=
de 100644
> > > index 000000000..1aa491799
> > > --- /dev/null
> > > +++ b/Documentation/devicetree/bindings/rtc/adi,max313xx.yaml
> > > @@ -0,0 +1,163 @@
> > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> > > +# Copyright 2022 Analog Devices Inc.
> > > +%YAML 1.2
> > > +---
> > > +$id: http://devicetree.org/schemas/rtc/adi,max313xx.yaml#
> > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > +
> > > +title: Analog Devices MAX313XX series I2C RTC driver
> >=20
> > Drop "driver" unless it is some hardware-related term.
> >=20
> > > +
> > > +maintainers:
> > > +  - Ibrahim Tilki <Ibrahim.Tilki@analog.com>
> > > +  - Zeynep Arslanbenzer <Zeynep.Arslanbenzer@analog.com>
> > > +
> > > +description: Bindings for the Analog Devices MAX313XX series RTCs.
> >=20
> > Drop "Bindings for"
> >=20
> > > +
> > > +properties:
> > > +  compatible:
> > > +    enum:
> > > +      - adi,max31328
> > > +      - adi,max31329
> > > +      - adi,max31331
> > > +      - adi,max31334
> > > +      - adi,max31341
> > > +      - adi,max31342
> > > +      - adi,max31343
> >=20
> > This looked familiar... and indeed it is.
> >=20
> > https://lore.kernel.org/all/a382fdee-3672-50b8-cd58-85563b9d9079@linaro=
.org/
> >=20
> > Where is the changelog? What are the differences? How can we understand
> > what is happening here?
> >=20
>=20
> I hope that other submission was an error; otherwise it tried to sneak in
> the bindings for max313xx under an unrelated subject.
>=20
> Guenter

This binding was included in the v6 of max11410 patch by mistake and I
mentioned it in v8 cover letter:

https://lore.kernel.org/all/20221003105903.229-1-Ibrahim.Tilki@analog.com/

I still addressed your comments but did not include a changelog since
this is the first patch of a new series.

I'm sorry for the confusion.

Ibrahim