Received-SPF: pass (google.com: domain of linux-kernel+bounces-49405-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>, "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+XUyR5O7FIAVIxLD2sMaA
Date: Fri, 2 Feb 2024 08:03:06 +0000
Message-ID:
 <TYCPR01MB112694BA3A47579AE8BC1752B86422@TYCPR01MB11269.jpnprd01.prod.outlook.com>
References: <20240202075849.3422380-1-claudiu.beznea.uj@bp.renesas.com>
 <20240202075849.3422380-2-claudiu.beznea.uj@bp.renesas.com>
In-Reply-To: <20240202075849.3422380-2-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?GEMGXhwxYr5Co+PTUUg1A/WWGkxC617VqtB+wqJXE+PnCOLYdE0BfqYAxjeb?=
 =?us-ascii?Q?TyHjJ1LrVN33+Cy1/JGKh0ESohwPS+QTQPKQLnwgYvXjztztfLyAGhvoSS6x?=
 =?us-ascii?Q?7a+opEeA5G1nvDKjw0n/VAwoXc4Zg/9pVoaSfgaDRQNQecFoXvs2cuAtVWst?=
 =?us-ascii?Q?ZBMpEKxEyafSVVsEFm2wvgcCzf6ewO/nCoXe2dRwPHfUFuHGE3924HzO9FOz?=
 =?us-ascii?Q?Y4FI3udDKhPT+8l2VOhgfSBwr6pH1v7rDxjVRGjB1aHsL+vF0ZI8S1JlAm2j?=
 =?us-ascii?Q?ndXieNw9GKHgFeX8hHdbrL375Rp42JGYWm6RbAen3WXlVtLryw233qmxZ1DI?=
 =?us-ascii?Q?/H557b5XeeHlemwYXsc0a3Om6VSw64LGXnIeIifmHf5/y3jYt8vvDpHQJIOL?=
 =?us-ascii?Q?Isvoazuf5eo7Xib/BhWTIL32mf8LK8BbfxzlCrZ5eez4expE2O4LyNhGl+BF?=
 =?us-ascii?Q?OfbN7MOlWKm7xEW3oUDKrcJtRKUmCIFo3E4k7q0/GNo0OAG0UosxAGM7NY17?=
 =?us-ascii?Q?6ZFzdMFgOHPlMZRLoTd/S3I8ewaKF4iGvE2XPVREoKWT3eh3DyWLh6kMywWP?=
 =?us-ascii?Q?rBkhcphVkXYZsbn4ppsz/pF8dZdtA3Jjbsdj4GrCndc4yowFq4oRFBU6TKFn?=
 =?us-ascii?Q?PsVzPC9H9xReebG2N5ohkFthyR5p3p1x/8KXGAV7mBzoKYZ/DguTmKRcv4Y0?=
 =?us-ascii?Q?pX6Wx68W6sYnwFBxT9LrqQuJfbpLRsp+l0Gyj3qK69lQb+U/5R5tRRRhEZNv?=
 =?us-ascii?Q?2o4EF2K3gVnm2jDiGWhNC6g16t4ZxtFZX7nDYe0d9ywbTF6mDrT9Jh0DWN2M?=
 =?us-ascii?Q?xuobO9/U6lz0Rhw4sFFn1WZpjTge0ePGlX9Dp9kspw8Z9kpjvpZtEpsmTXM5?=
 =?us-ascii?Q?apyeMkYLJJ3kG2RIiuwvvZ4PnPZBpmWi7sCp8bgQHKwCv8sKa+zbPvZUhC/R?=
 =?us-ascii?Q?+iGdmUkJtsF9UGwZaNFeNrYWWi2PO24L+dFrvpFgXbLI5usoqee9ctjQ7eRm?=
 =?us-ascii?Q?9jv3StcWD3VxhUryC8+wWr4PTQrfWVFFHBcLSz56rwddIXRcxxfsyCrCLbD9?=
 =?us-ascii?Q?jBqaWx27zbUyPnocbR3ytjdT9tp5VjA3wS9/4plrw1v9MtIChlK/9d8E2gVv?=
 =?us-ascii?Q?z6vbRqpbyKBihUqVqdJTT5V0sD6GFIC42MP1Zxguggkn/Tu+CEjwd5Tel6YI?=
 =?us-ascii?Q?WKO42b+RJx4QSCfoAul5fHu/SNpjNCvVaBZBsTtORFljcd7xQ1RcH4v/ty+L?=
 =?us-ascii?Q?FYm2JZxU64raknFxnSKhNXiKUNlg6RlZsYQhSLwG4gQ7CemjFt5zNpQssxPi?=
 =?us-ascii?Q?Y2FdiKquQLgdT1pblimcirpu7WV/2Cmgrv7ppAw0si/HDCIb2Aa/r1CsUQyi?=
 =?us-ascii?Q?Mp/oEgAMGKDSRcLqcE18AIVhubFa5zJqY4OT6Q1nmrANv0y+3fokX4XZrC5o?=
 =?us-ascii?Q?ld4hGj7odFVW7dKuaeiIaTkiYcNenwsG6HoW9MV5gJyH688hzlgza6OA4lkJ?=
 =?us-ascii?Q?c7QY+EaELXg4xkVpBIKSaYycdkwMqb4uCg1ek+or7PDJXc9IK5g9WHP+biZy?=
 =?us-ascii?Q?noQ6NnUkPHO8bjkdARLUpAvGwgU+K/wWt9gFr50kKcF2FjPGn8RcrcCj9Usl?=
 =?us-ascii?Q?vw=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: 980c6869-fa32-413d-0f8f-08dc23c56350
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Feb 2024 08:03:06.1152
 (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: C0LrGxqCKBebluOmlrkcTF57QXcC2LBnBc5WrnWN6jfwaDFiRfG5/muwK9F8ihi4pf0AmhLq14Q6VPiayDjjonMFO8hyrADNgMa7Og5L2oo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYWPR01MB11274

Hi Claudiu Beznea,

> -----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
>=20
> From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
>=20
> The rzg2l_wdt watchdog driver cannot work w/o CONFIG_PM=3Dy (e.g. the clo=
cks
> are enabled though pm_runtime_* specific APIs). To avoid building a drive=
r
> that doesn't work make it depend on CONFIG_PM.
>=20
> Suggested-by: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
> ---
>=20
> 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
>=20
> Changes in v2:
> - this patch is new
>=20
>  drivers/watchdog/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>=20
> 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
>=20
>  config RENESAS_RZG2LWDT
>  	tristate "Renesas RZ/G2L WDT Watchdog"
> -	depends on ARCH_RENESAS || COMPILE_TEST
> +	depends on (ARCH_RENESAS && PM) || COMPILE_TEST

Since you are touching here, maybe ARCH_RZG2L?? like other RZ/G2L drivers.

Cheers,
BIju

>  	select WATCHDOG_CORE
>  	help
>  	  This driver adds watchdog support for the integrated watchdogs in
> the
> --
> 2.39.2