Received-SPF: pass (google.com: domain of linux-kernel+bounces-49471-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: Biju Das <biju.das.jz@bp.renesas.com>, 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+CAAANJEA==
Date: Fri, 2 Feb 2024 08:29:05 +0000
Message-ID:
 <TYCPR01MB11269FECF0067E0DC7D7CA0C986422@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>
 <TYCPR01MB11269B343C70AA99CBDB3BD8586422@TYCPR01MB11269.jpnprd01.prod.outlook.com>
In-Reply-To:
 <TYCPR01MB11269B343C70AA99CBDB3BD8586422@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?yx8eQfMlve+fKE6IcMKwCZUvgjnkIvV7QYaCYDDx29XvoyAUefQftheevfw6?=
 =?us-ascii?Q?WX5EKjjTuCAYuWIMLBJHuvaYJaEzWSJAG/YcgoUyK6GK5YoPiOdA5cicuSay?=
 =?us-ascii?Q?EPx613Uli50yWou1IC9rBNTyeP7up5g3t38K6QWfKyWuk/yvEKB/ty9AgqOL?=
 =?us-ascii?Q?8CyBVfr5njNofSLdhUIBmjDDNg7YT8R9visEhO7Gi0tiLb+W4fNARHeFELdB?=
 =?us-ascii?Q?TfalYcMISieddsnL66e26zanZsoFJUgAmTpbc8fN276AjBK945ReCjwgnvkj?=
 =?us-ascii?Q?MRKI8xWK2GaRllRfOzRQYCXBXSdlATyl5uNsRlCEFVot4zp458nhu9z8Y3nN?=
 =?us-ascii?Q?nKo7x73y5zXb1stYmiVKv+jwIbllszC9S0JW4MqJi5JAOHe2e93dyht8AFeG?=
 =?us-ascii?Q?pkrdgZ/UGAs4n2wIZmMzS+/UwHCqsAuQ5SWSD5Z58gfO8PDQx8GCHFn5mMDP?=
 =?us-ascii?Q?1sbNeWzvs8VBhq/hGmfoNsnoV11hIoOXJC5Jvyf13icXAi0E6ahXjH+lSO7y?=
 =?us-ascii?Q?XauYRv0XcdNhU0pdmC5hd72ZoZHy+7ap/UMIl8dDN/pD6C/+oseBP54g1KsJ?=
 =?us-ascii?Q?SdTHAjcZq4zscmtP8d4/g77fN5WMtTWRUjtNfu/6S59qhIEM9HIXYj/UQipZ?=
 =?us-ascii?Q?TN4BO9Fml4786I5+QxrPo14B+nQg3mimhrr3j36CjBbSQAC2ZEqeR0UGIZny?=
 =?us-ascii?Q?GbB53UYyymnCC94tnhoFd6IJw1xriJ5WMH9OV5fet0FHc69v+38qqfFG2UAJ?=
 =?us-ascii?Q?Hbv7OVYug8yKslQHX+ujKJeE4Nq3+Z5kY2FcFaHZ2TTB3wO9f9/EFOnPHyzm?=
 =?us-ascii?Q?2ouB/otxZMrL7niMNHaBjVFemM8uZbDA0P+V2i0q3Ymnb/WGxcVgWDC7/H8/?=
 =?us-ascii?Q?6C7gLNhNQlmfAQbCkNjBGbcjbbANQ/KuiKX4BIkfXYi1lUmJEZpNaWd1ZHR2?=
 =?us-ascii?Q?rmIlXhvOMVvOYdNKrhYCHnrGE08DZQSKK4nS5S790r2gS817SyM+go8WigIY?=
 =?us-ascii?Q?EdI4skL1r18LZRPQLyAPRuDAHWyv1Wf8pk1deY7x+6gJ9ZBIajP07wb61ltX?=
 =?us-ascii?Q?PiogTtt8oQbhjOfr8JS1ndgkl91CS9m78d1WBJCFTP9KFRltHinPBPQ/XmzS?=
 =?us-ascii?Q?xB8pSTMwp5uO6CZzXB5dQeSDhilE9yU1Hs6yiGPqbJLzKxJBXLnlVTFMqwsD?=
 =?us-ascii?Q?gniiWES6STatHm4KHzJaQZMAGPM0yTOBIRN8FMaAClXmqkrFaoq1F3sI4bxT?=
 =?us-ascii?Q?yHPRqyiE79QkWrgqvD2hQwoSERaUXYvq4hW3l71QevEPMl6pI4VNE39plBwZ?=
 =?us-ascii?Q?c30eGIEcPp4EjhEGqP5576LlxyN3LZkXFbNQ9ahNtEDHv/IGDe8X2fCQ+fQL?=
 =?us-ascii?Q?izroQOGl+Nz0unnDmXnz8IEAnwmTAkOAo5NUkSt8FWsSsH/OSAe5pr5JHXEp?=
 =?us-ascii?Q?nNP0OneynowmxESPu/mmxmsjpErohZQFJ1Lp7aMqz+VGD2ttRmRHjW0Ot/x/?=
 =?us-ascii?Q?WilwDFnNLtyJfMYoPOVhmh1M+lSKidkNeXDOSUw4/WP9BBaCMwGwvUpWci3U?=
 =?us-ascii?Q?qhgxxoMJS4QbU6II76F++wHuCv34R08YzIR7PMnMRLeHpEed0Zf52DqAmQp3?=
 =?us-ascii?Q?Rg=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: 099f7555-c050-4cb3-8846-08dc23c90493
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Feb 2024 08:29:05.1442
 (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: 5a3ekIq30UQVn1u0VB1s0AmY+3G6DKj8qAE6GhaGkJF14RFimthiF/hF0YPiJmuQG+1Xv4yPjOdgqntmo+xgvjOLvUmoHNvq8owRYPUKVnk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: OS3PR01MB8044

> > -----Original Message-----
> > From: Biju Das
> > Sent: Friday, February 2, 2024 8:03 AM
> > Subject: RE: [PATCH v3 1/8] watchdog: rzg2l_wdt: Select PM
> >
> > 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
> > >
> > > 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
> >
> > Since you are touching here, maybe ARCH_RZG2L?? like other RZ/G2L
> drivers.
>=20
> Also RZ/V2M "ARCH_R9A09G011" as both these families using same driver.

Also commit header it is "Depend on" not "Select"?

Cheers,
Biju