Received-SPF: pass (google.com: domain of linux-kernel+bounces-214483-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Biju Das <biju.das.jz@bp.renesas.com>
To: Claudiu.Beznea <claudiu.beznea@tuxon.dev>, "geert+renesas@glider.be"
	<geert+renesas@glider.be>, "mturquette@baylibre.com"
	<mturquette@baylibre.com>, "sboyd@kernel.org" <sboyd@kernel.org>,
	"robh@kernel.org" <robh@kernel.org>, "krzk+dt@kernel.org"
	<krzk+dt@kernel.org>, "conor+dt@kernel.org" <conor+dt@kernel.org>,
	"lee@kernel.org" <lee@kernel.org>, "alexandre.belloni@bootlin.com"
	<alexandre.belloni@bootlin.com>, "magnus.damm@gmail.com"
	<magnus.damm@gmail.com>
CC: "linux-renesas-soc@vger.kernel.org" <linux-renesas-soc@vger.kernel.org>,
	"linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-rtc@vger.kernel.org" <linux-rtc@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>, Claudiu.Beznea
	<claudiu.beznea@tuxon.dev>, Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Subject: RE: [PATCH 05/12] dt-bindings: rtc: renesas,rzg3s-rtc: Document the
 Renesas RZ/G3S RTC
Thread-Topic: [PATCH 05/12] dt-bindings: rtc: renesas,rzg3s-rtc: Document the
 Renesas RZ/G3S RTC
Thread-Index: AQHavitbIcpiZE+kDUKhRF+4WvcTrLHG4xNQ
Date: Fri, 14 Jun 2024 07:53:14 +0000
Message-ID:
 <TY3PR01MB113464811F43F19115FCF62A786C22@TY3PR01MB11346.jpnprd01.prod.outlook.com>
References: <20240614071932.1014067-1-claudiu.beznea.uj@bp.renesas.com>
 <20240614071932.1014067-6-claudiu.beznea.uj@bp.renesas.com>
In-Reply-To: <20240614071932.1014067-6-claudiu.beznea.uj@bp.renesas.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?m8F7BIAENstgr5BypBqq82Hgy3ZJAbVbvVO5F80cZq2yUPjefeQz35KTtVOL?=
 =?us-ascii?Q?ZPkWsMDRVfAV9lfbebndWBL6RSyyWpySUZbKTMis+roJgbIPhbOdtvkJA2Jh?=
 =?us-ascii?Q?1Dm/5Xp7m+uTCbbrlNOZtvFCrax5V6aQ8XmDa5GVi70V+u9eP/HI5I209tTp?=
 =?us-ascii?Q?4w0wCsdFLdSPioMNwTjuUNxlU/4dEJEqKmAEcQSxcoMGGeTL5hNE/D3k9NMg?=
 =?us-ascii?Q?h2UHhhBPn92NBWOPmQSXZjV7veG+BOxuPGGg33LYpVazxenTWZSzBXnISnY9?=
 =?us-ascii?Q?fLVv/fPdp/w8iwZpHvIRkwd9OzDlf62WPv1FW87Xza/4quDBGunV6Bd/Kmno?=
 =?us-ascii?Q?f/8/y5UfyV/6Dc2jbcNEZeO1JhE2nl7Y9DpovhKGHW9o5nQ90ns4cd/yeYUJ?=
 =?us-ascii?Q?xBVW8mT3fc+E0qkIg5LCtadLzBgF1rRKxKSTvcjIhYSjwCMyH8prXEgXc/Na?=
 =?us-ascii?Q?T8ySHeW+8w0PwDSXZ4mzLo/4z8BuwvwUmcMkdFqlSUFZw31YhSjLIicwu3hw?=
 =?us-ascii?Q?D2HALdV1NugavZMhzdXs07D8TpXHhLcqQAvEHqGWor4u4tzXHBAjpwZf0atX?=
 =?us-ascii?Q?/5Enw80jSrGHRQHQ9oWX0JwYa8W4qlKDti9qyLtUAi1A1QgfT7IRGoBEDDmR?=
 =?us-ascii?Q?DV5M7DlSaGmjpy6jnCGlHe7p1af1dkauCI1eoaqZnXL3oj5qyFS4CHIs0/h5?=
 =?us-ascii?Q?GVpp0fxE7bLcpR026gw387ELvyKHq8tw480yuBQmrXWG3MZ9ba9Mz/cXhzGN?=
 =?us-ascii?Q?0bxFLflPMQkiT+sn6jIy4f56ZGjUOWzNavmJzQeHWE2rQvsVNkl/3dazHZvY?=
 =?us-ascii?Q?myTcDYpSARc0jy4TFl95vlaejniEPBqIvNf3JdSnGtg6DcZaMR8pPppXfalK?=
 =?us-ascii?Q?zroq93x69t9ewTceOWbgtOQf9fScrBpW5RI6KmvK9TA+205HdLgEfFQOUEkx?=
 =?us-ascii?Q?etojhYGptbULaUffLFJMK8wICRZQW1HwMWQUPFkR7tmDLVxcFUdye9zm3XB4?=
 =?us-ascii?Q?xKJrXsYffr32QEgYSDn3HcvE1l+pGwwU1qA8dx3Wg/kwMprgIfeWZ9I2Ag8j?=
 =?us-ascii?Q?bEdkLE4fBAP3R1w5lSiPjpB6hawb6ccG+ebLQQZuiZKwyF9U3UEeTp/Nv0o6?=
 =?us-ascii?Q?/83TIMqQPMm1gHnHkNQYRuRNTbrmh5TwT/b4Lg2GZx97Mpnh0ya7Bj7Xgg4L?=
 =?us-ascii?Q?rcGv6DZ/KiMJh+3aBMTfwtPg1UvIFuHxQH7jukWuYWMNwdlP86ThuP1k5lcn?=
 =?us-ascii?Q?fe6gUw0sGw5QH/4jOuzhoCIJpZUFISJ9TgBaHhmPU5ypSl0hAlevwBER9cyM?=
 =?us-ascii?Q?6B1+Zvlr7lLmz6X/LMoghVbsJi5tRYSoWkLUuzIu78U+LjED37IOb5YBKEYC?=
 =?us-ascii?Q?XdMVad6lQeLou2rjg6DY/rRP2sBWQe4HVP7TP3eEKyFWO6LADpXsRuTpcQkw?=
 =?us-ascii?Q?2BsrzzDnriDmBSeKDhoZj/YG0ALsS75bdKspSezdcx2Qhk9PEsInJ/Xok8lq?=
 =?us-ascii?Q?lv97eSfwvy1wpB+24PagJITr/Jl46t/D7eok2afSpwQmDap8UVyQp1CpLWya?=
 =?us-ascii?Q?fayaTXzWuao5AJTyv+Juj9Vemd3EV+6+MWY8QZfffT6Ks6RjqpAmMVs4FicE?=
 =?us-ascii?Q?fw=3D=3D?=
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: bp.renesas.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: TY3PR01MB11346.jpnprd01.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5e6c4731-9a8f-40b7-d8fc-08dc8c470b81
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Jun 2024 07:53:14.3311
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 53d82571-da19-47e4-9cb4-625a166a4a2a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: htP/azTCDHM9nU37echMFbN4ETHBnVg9PzcCo8H82aiDsoV3GqWg8bZuSnFM+7lr5hVyhGCnRvL7ev8lwB8Rj5ekJ6XbyRlJt91A9SbtnM4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: OS3PR01MB10171

Hi Claudiu,

Thanks for the patch.

> -----Original Message-----
> From: linux-arm-kernel <linux-arm-kernel-bounces@lists.infradead.org> On =
Behalf Of Claudiu
> Sent: Friday, June 14, 2024 8:19 AM
> Subject: [PATCH 05/12] dt-bindings: rtc: renesas,rzg3s-rtc: Document the =
Renesas RZ/G3S RTC
>=20
> From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
>=20
> Document the RTC IP (RTCA-3) available on the Renesas RZ/G3S SoC.
>=20
> Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
> ---
>  .../bindings/rtc/renesas,rzg3s-rtc.yaml       | 60 +++++++++++++++++++
>  1 file changed, 60 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/rtc/renesas,rzg3s-r=
tc.yaml
>=20
> diff --git a/Documentation/devicetree/bindings/rtc/renesas,rzg3s-rtc.yaml
> b/Documentation/devicetree/bindings/rtc/renesas,rzg3s-rtc.yaml
> new file mode 100644
> index 000000000000..0e17f8a36155
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/rtc/renesas,rzg3s-rtc.yaml
> @@ -0,0 +1,60 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/rtc/renesas,rzg3s-rtc.yaml#

Please make it generic renesas,rtca3-rtc.yaml. Future SoCs may use this IP.
So use IP name instead.

Cheers,
Biju

> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Real Time Clock for Renesas RZ/G3S SoC
> +
> +maintainers:
> +  - Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
> +
> +properties:
> +  compatible:
> +    const: renesas,rzg3s-rtc
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    maxItems: 3
> +
> +  interrupt-names:
> +    items:
> +      - const: alarm
> +      - const: period
> +      - const: carry
> +
> +  clocks:
> +    maxItems: 1
> +    description: RTC counter clock
> +
> +  clock-names:
> +    maxItems: 1
> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts
> +  - interrupt-names
> +  - clocks
> +  - clock-names
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    #include <dt-bindings/interrupt-controller/arm-gic.h>
> +    #include <dt-bindings/interrupt-controller/irq.h>
> +
> +    rtc: rtc@1004ec00 {
> +        compatible =3D "renesas,rzg3s-rtc";
> +        reg =3D <0x1004ec00 0x400>;
> +        interrupts =3D <GIC_SPI 315 IRQ_TYPE_LEVEL_HIGH>,
> +                     <GIC_SPI 316 IRQ_TYPE_LEVEL_HIGH>,
> +                     <GIC_SPI 317 IRQ_TYPE_LEVEL_HIGH>;
> +        interrupt-names =3D "alarm", "period", "carry";
> +        clocks =3D <&vbattclk>;
> +        clock-names =3D "counter";
> +        status =3D "disabled";
> +    };
> --
> 2.39.2
>=20