Received-SPF: pass (google.com: domain of linux-kernel+bounces-49451-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Biju Das <biju.das.jz@bp.renesas.com>
To: Claudiu.Beznea <claudiu.beznea@tuxon.dev>, "wim@linux-watchdog.org"
	<wim@linux-watchdog.org>, "linux@roeck-us.net" <linux@roeck-us.net>,
	"robh@kernel.org" <robh@kernel.org>, "krzysztof.kozlowski+dt@linaro.org"
	<krzysztof.kozlowski+dt@linaro.org>, "conor+dt@kernel.org"
	<conor+dt@kernel.org>, "p.zabel@pengutronix.de" <p.zabel@pengutronix.de>,
	"geert+renesas@glider.be" <geert+renesas@glider.be>, "magnus.damm@gmail.com"
	<magnus.damm@gmail.com>
CC: "linux-watchdog@vger.kernel.org" <linux-watchdog@vger.kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-renesas-soc@vger.kernel.org" <linux-renesas-soc@vger.kernel.org>,
	Claudiu.Beznea <claudiu.beznea@tuxon.dev>, Claudiu Beznea
	<claudiu.beznea.uj@bp.renesas.com>
Subject: RE: [PATCH v3 1/8] watchdog: rzg2l_wdt: Select PM
Thread-Topic: [PATCH v3 1/8] watchdog: rzg2l_wdt: Select PM
Thread-Index: AQHaVa2xeucsotW+XUyR5O7FIAVIxLD2sMaAgAADP+A=
Date: Fri, 2 Feb 2024 08:14:43 +0000
Message-ID:
 <TYCPR01MB11269B343C70AA99CBDB3BD8586422@TYCPR01MB11269.jpnprd01.prod.outlook.com>
References: <20240202075849.3422380-1-claudiu.beznea.uj@bp.renesas.com>
 <20240202075849.3422380-2-claudiu.beznea.uj@bp.renesas.com>
 <TYCPR01MB112694BA3A47579AE8BC1752B86422@TYCPR01MB11269.jpnprd01.prod.outlook.com>
In-Reply-To:
 <TYCPR01MB112694BA3A47579AE8BC1752B86422@TYCPR01MB11269.jpnprd01.prod.outlook.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?UH0MyvoNW3cXixdvo9dqJmN3cfyPdsSoReSTFLHOZvzjH7o79tdcRm5oSSMA?=
 =?us-ascii?Q?SG1HQ8Pdt755xjzvt6gaizrVP3tWyTa8xoUufVVhwRNHP/iRarxAM9aYuv98?=
 =?us-ascii?Q?+ivQ0gL2dmQDpDDtbXrDUZgy1+jMGCYLXjmQQ2tBoAx7eppUF9t3KwODr8H8?=
 =?us-ascii?Q?T+f/EZiMjKhMtMx3rLmVoiGEZ0sJbT6RpcHuk51mK9TqBd0KGBoxPGCoF9C+?=
 =?us-ascii?Q?XsJjqbVpvk0OtY1x1Jae92lJxLtuQJ2IoYpVT5HAOzvM2vpv9XYBhzVEbKM6?=
 =?us-ascii?Q?MOWQHWsF6VHCJmycKYB1Lz4lgJ01Gg0lhyH4djMo3enrIIs6eQY6ouZpNxMT?=
 =?us-ascii?Q?MF4wBhOVPvkmW1CI5ylif8wUTyTP4hvcwIZ7VD3XBMLw6zT3wKQJW2/LImec?=
 =?us-ascii?Q?nvyOai9fnyHZpfg49aLUr1IjWE1xjLQEBfA560zdjAFGIDBrGZy0gbzOTU63?=
 =?us-ascii?Q?dQQiMeCDlcGt92nsB/Le4jE5HH2d8DRM9jZG7x5fV++INK5uKDhFo02MErCn?=
 =?us-ascii?Q?xc2qfQDUndavSD+auu2fljprIdk+vRP1wPj1g+gdqzBoXrcF8RXn+z4oT4Rl?=
 =?us-ascii?Q?jHaseS8JtJMZYgJ5DMsMVpfjZ5JLXi+SyGffMqjf3yDVwhBfnw93AIWt/X+l?=
 =?us-ascii?Q?Hmo5VvjcCi0XxFzOxsf2RoIf0cRLUcsyVACYanfXtduAeQNhUOTlFPuy0XeW?=
 =?us-ascii?Q?nWXhZkiwBUCz90Qcl0AN4j928pYBQYAc2JA652blTmX1au4vbHYY0HuF++p1?=
 =?us-ascii?Q?/nOrqmXtTD05im0/S7LanJKw7G7OMMz2JU7v0sCbxeD0I2jLRg8HnAG5iqUO?=
 =?us-ascii?Q?RcBeHW3Pp5TUe9Ll/GqOqxQgWcoZUOKKanKT8gumf35NCDgyttu8pDdz+l05?=
 =?us-ascii?Q?ualeQTH+ZZEdO5lxbn1zUO0Wa7PzuDv9j8F3dlyZ2BKYAe74qldIW0P9ROxS?=
 =?us-ascii?Q?xj/tohp4Dx1Y93sXG2UeqVZiYw8AZ3qwQnuLKIrpdmGg5u8KsPSU7sgl5Iga?=
 =?us-ascii?Q?EIu8RaPv4/puvzHrQ078FIZGMNItTPniw9VUgLN6fjtjGUoqRxOtiwRqxM0K?=
 =?us-ascii?Q?E0qKK1GBhF6s2eagyz9Y6j+KOKGcjRKZf9G4/12kDwWGi0QQXXG9hWwcxdIi?=
 =?us-ascii?Q?Sk4peKwTSDoJIUsXiyqJR3PMVdB6CZ3K3z5wtCyQqsdM2Uo2ROFo+Lnfnxey?=
 =?us-ascii?Q?eKQAXvf/2XfvveBGVZBY/JDFdM8yp0sSOt/CF4LZSVFL3YJ3nXigv/FZfpZl?=
 =?us-ascii?Q?aEfRcl73rP54YSCTihtemhp9GinVAdJjZQ5nPCiMl0IGTmtwIRQGEjl+ASNo?=
 =?us-ascii?Q?fcP2L5425gDsz1LjhEXXKLxckzlYyGS/HGAFD82y371lhmTeq9OYjlEUYLKs?=
 =?us-ascii?Q?/zXJYKsek50QuMplgZle+yr/dObg4PfIhPN9WHuiS9GTsceGsQESZ1Bv/ajM?=
 =?us-ascii?Q?VlMY+xnsIQh8MDD6151RBsK/XSEEGyUj07pp8fcolNBfZMlPx9qumVHKiweP?=
 =?us-ascii?Q?EAfvmiTQlz2WwoZgGQ539ZHlNvSnUUWbL6N5dEmTVGMWqNMSIYCWNmqdkWWg?=
 =?us-ascii?Q?0h14Bd2nLjZyTA9P6CvZySSAw193RYSyk8JC59dn+B8M+LabZLXv9ctJY9Zt?=
 =?us-ascii?Q?fQ=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: TYCPR01MB11269.jpnprd01.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 03e52585-48ba-47a2-c174-08dc23c702ed
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Feb 2024 08:14:43.4287
 (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: Xrx2l3A9J019BE+aSGl077HIKocVoLj5CSqPLtaZxA3IpjeGvyZCZesH9qYn44W1BzTYIDWWq7UTfea6Dzxd8jHjACtbwLoo76HQGXA1UO4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYAPR01MB5626


> -----Original Message-----
> From: Biju Das
> Sent: Friday, February 2, 2024 8:03 AM
> Subject: RE: [PATCH v3 1/8] watchdog: rzg2l_wdt: Select PM
>=20
> Hi Claudiu Beznea,
>=20
> > -----Original Message-----
> > From: Claudiu <claudiu.beznea@tuxon.dev>
> > Sent: Friday, February 2, 2024 7:59 AM
> > Subject: [PATCH v3 1/8] watchdog: rzg2l_wdt: Select PM
> >
> > From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
> >
> > The rzg2l_wdt watchdog driver cannot work w/o CONFIG_PM=3Dy (e.g. the
> > clocks are enabled though pm_runtime_* specific APIs). To avoid
> > building a driver that doesn't work make it depend on CONFIG_PM.
> >
> > Suggested-by: Guenter Roeck <linux@roeck-us.net>
> > Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
> > ---
> >
> > Changes in v3:
> > - make driver depend on PM; with that the "unmet direct dependency"
> >   Reported-by: kernel test robot <lkp@intel.com>
> >   was also fixed
> > - adapt commit message
> >
> > Changes in v2:
> > - this patch is new
> >
> >  drivers/watchdog/Kconfig | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index
> > 7d22051b15a2..c9abe8f307bb 100644
> > --- a/drivers/watchdog/Kconfig
> > +++ b/drivers/watchdog/Kconfig
> > @@ -910,7 +910,7 @@ config RENESAS_RZN1WDT
> >
> >  config RENESAS_RZG2LWDT
> >  	tristate "Renesas RZ/G2L WDT Watchdog"
> > -	depends on ARCH_RENESAS || COMPILE_TEST
> > +	depends on (ARCH_RENESAS && PM) || COMPILE_TEST
>=20
> Since you are touching here, maybe ARCH_RZG2L?? like other RZ/G2L drivers=
.

Also RZ/V2M "ARCH_R9A09G011" as both these families using same driver.

Cheers,
Biju