From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr2-f43.google.com (mail-wr2-f43.google.com [74.125.225.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0940A545D9A for ; Tue, 22 Sep 2026 12:53:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.225.107 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790081609; cv=none; b=lzxBjffInCu1fwwio9QopRyW2KPIPLTmI9SwLJylv0lEYbfgEtlCxfRCfUUO0hM/XLjJimAwTkY8aCXL2Weh2VhXtQ3tAmQsxnGFUw5SQclh+TjozsjiVAPy+rmc0g6GII6Aazivz2DAgw/q3cNRQW9/Jobl4nrGTlANKlW/t40= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790081609; c=relaxed/simple; bh=v4Ul7xXqzvPTm1KnAhXO58P90eD2KlGTNsegXWmzADU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=XA3VhjnuJ4foOUVrBK/gFiRDiD0dmrdLeTnyWxdod+eNARtrvuTXGX+Z4YJVk2bIq79EjY6gd9eZ1jPFqMnAHQygJ1/HaYZV6Ps47sLDjJCYQG2vR4IjW2+uOPyVjwLVswyiUtozv+3B9TPYNOyELQQ2YudUKn6kZ2FWw8VSjgw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b=YhybTcQE; arc=none smtp.client-ip=74.125.225.107 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b="YhybTcQE" Received: by mail-wr2-f43.google.com with SMTP id ffacd0b85a97d-482f62ccdb1so1434413f8f.1 for ; Tue, 22 Sep 2026 05:53:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre.com; s=google; t=1790081599; x=1790686399; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=UsAsHIO1y8RCUKiEPTTkY3IWmv6reNZonvZkfCzW3Ag=; b=YhybTcQEyKITrmRourFyhi61KkzUyB1QwEiSQUkUdZrWhni2DxJp7ta4BVBp9zNwRF y2I2IRbuSAEwaD27Ia+kB2jfwDci4L+rdlUS8iDjq4WZx5WmunC0XGR9eLRYI0L+G4rC rlXWSwCHXLPFWJpIjRKlLVxRYqkvp51ft5gaVj1mBca1fv5h5UcMeOuwdEpcGx26pdtS IvlINlotnj4BPFUg8Qc+oal3YsHg4Co+Nb0tHlX7GvLle9U+SBySrI+6eB9IRQ/L9RH5 Pd/hU95ZC0LGje58BGN4gks3b3x5Q96FoAqLfWJNHXETdcg5n115ejuEhmxM57Nrnkft A7wA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790081599; x=1790686399; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=UsAsHIO1y8RCUKiEPTTkY3IWmv6reNZonvZkfCzW3Ag=; b=AGR8IIBBBdDXMqvHkwkR9CLpf/ZUbhzaUJNld6UN3OGYNWq/zAVJmuXfW4Y04CobQM luVzHNKNi4ri4OW4SoqBFmDBUE1MNll/cYQXcUQaYCyQxQ/Kc64Wtu8kfISXASnX6fqR uWCTHQnu9rLNnVO4zW/4vecuIwBaYIvcEhvZHmoOfYsFg++5cFDVrQqNYdGvdkalW4uj LIyiVNKnqsoZJtnaaZzoBgJExFlQmQZ52Uvu3juG+FtVnBrTGzQ/ZkCs7uQ4UZ9KybUx CsfTM4uEzq7A6hzYrZbjXmE8SUn1SjNVMv1WUiv+9G+RGm0E4GwJVBBSgbjHm/GgJiiC xP7Q== X-Forwarded-Encrypted: i=1; AKwUvBzUDNxR1pNbp6kQ4Lr2BWKGtqizvWX8e1nzkWqUORnyA6Tv3qikuS0YQBMVtwIHsvyAKYb6/277X/Nv@vger.kernel.org X-Gm-Message-State: AFuF++n9WhVTHcCnSnAt0KxnuhT6X6mfvoTrcJr59beCoTdM5FAJvR+2 wom9ZBIUTFLzm1h1h4ITwX14W+ODi9MQzbinSg4AgUM1ymgyinSXdFqcP8/4TSuy8Y8= X-Gm-Gg: AYBFou17Tq1Bv/tjg23YFRtn2OHu1OTC/KaOMNVd3ntyV5MN4/JCD3+UgN6uKW6ds8O oj4Tqz5IgeZzyyTtnqrBQ4T8DcPHQeg7MJ2NMIX8Qskb3U0TIA0SRBhFq7klJWiJsIz7JxesW3W KF95SQ3nA6gF17qq6FF7UY7Lwtja17/JweoZb2TToqLKfvLPl0f+prLd8owaAFtbAIfManiF8K9 8aRYEnU17Pn/nJ8o2Uq0UuSCjCkmX6f3cqKlZMoipkoqTDoViQ7lk0MQoG3OfH73KpKZ0ZK70Jd DKVkuyIaJO3UmXVqZIvSqxw8B4lNRo+a+WMKCir6AwPMzlPuCjbWuAWgYWVndOce0FDhJlOTupz JlNgb9LgcMCgChr+TLpO+IxjxgBO5XY3R3IBR8AFNNckWZagIyVZrVpO7Mu+jGXvo570cHVjhwt /g4LPC1K71WEzIJm6zqbrVvvcZRy9SVxeTlmRu9kIjCwK6CoLB5bCdaQ48WO1gD2y0AlAbnuRzG MGKxUKFEydW3gImMvXmrTmgkAUMVcpe13jargU5zQu8BpXyJ8rnkgXmeYqfOw== X-Received: by 2002:a05:6000:4608:b0:487:f31:857f with SMTP id ffacd0b85a97d-488610118bamr4476584f8f.15.1790081599025; Tue, 22 Sep 2026 05:53:19 -0700 (PDT) Received: from localhost (p200300f65f19a9041d0e57515b2ea4c8.dip0.t-ipconnect.de. [2003:f6:5f19:a904:1d0e:5751:5b2e:a4c8]) by smtp.gmail.com with UTF8SMTPSA id ffacd0b85a97d-48862731a53sm4931099f8f.4.2026.09.22.05.53.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Sep 2026 05:53:18 -0700 (PDT) Date: Tue, 22 Sep 2026 14:53:17 +0200 From: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= To: Alex Elder Cc: sboyd@kernel.org, bmasney+clk@redhat.com, jbrunet+clk@baylibre.com, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, lee@kernel.org, andersson@kernel.org, konradybcio@kernel.org, abelvesa@kernel.org, kees@kernel.org, gustavoars@kernel.org, p.zabel@pengutronix.de, daniel@riscstar.com, mohd.anwar@oss.qualcomm.com, lorenzo.bianconi@oss.qualcomm.com, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, mfd@lists.linux.dev, linux-arm-msm@vger.kernel.org, linux-hardening@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/4] clk: toshiba: introduce a TC9564 SoC clock and reset driver Message-ID: References: <20260918165234.687224-1-elder@riscstar.com> <20260918165234.687224-4-elder@riscstar.com> <28ebbf27-24db-4b5f-87f0-831d13f93b8f@riscstar.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="ztliwx3eij5ch7ku" Content-Disposition: inline In-Reply-To: <28ebbf27-24db-4b5f-87f0-831d13f93b8f@riscstar.com> --ztliwx3eij5ch7ku Content-Type: text/plain; protected-headers=v1; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH 3/4] clk: toshiba: introduce a TC9564 SoC clock and reset driver MIME-Version: 1.0 Hello Alex, On Tue, Sep 22, 2026 at 07:37:53AM -0500, Alex Elder wrote: > On 9/20/26 2:39 PM, Uwe Kleine-K=F6nig wrote: > > On Fri, Sep 18, 2026 at 11:52:32AM -0500, Alex Elder wrote: > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > > > +#include > >=20 > > Please rely on linux/platform_device.h to provide of_device_id and drop > > including . (If you want to go full iwyu, > > include instead.) >=20 > In general I would like to "go full iwyu" but have never heard > it expressed that way. Can you recommend a tool available that > will help me do that? There is `apt install iwyu`, but I only tried that shortly and didn't manage to make it work nicely with-in the kernel. > It looks like "mod_devicetable.h" doesn't help much unless I > need struct_cpu_feature (which I do not), so I'll include "of.h". Yeah, and for struct cpu_feature there is https://lore.kernel.org/all/ef103a4b9e1cd7477fa52ee347c4ec301354fba9.178998= 8540.git.u.kleine-koenig@baylibre.com/ =2E =20 > > > +static const struct of_device_id tc9564_clk_ids[] =3D { > > > + { .compatible =3D "toshiba,tc9564-clock" }, > > > + { }, > > > +}; > >=20 > > Drop the trailing comma after the list terminator please. >=20 > OK. Only for the list terminator, right, because nothing > will ever follow it? Non-empty final initializer is OK > to have a comma as far as you're concerned? correct. A non-empty last item even should have a comma. Best regards Uwe --ztliwx3eij5ch7ku Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmqyejsACgkQj4D7WH0S /k47Vgf+MRlVLPMGuXHor1vwVO9Aio3Q8yy+NUEK/fWTE+YEMTA1w+6YqsgeiJ18 k46MRbkEremJC2B/gZElmlpu4paEu99Qg+kif1t+k21ZaiC9Kl71V2ue+dNtbP93 U6/uxACp6gAw5u3yAf2GFlAOJhCXVBaeVd25LntVeVdhRYQb48cp6V5Uo2C2ukK5 WfEPetJwrAilKQUBpNladzLOIXT4/xVdzLTl1kQCTeJ+k+4f9ubiLI5C6cJOsf2j MC4jBXoimClS2rp5jMK42D8blKcvVDH4Z4X87FPzl1sqRy8EI+nFftpZt0hSW/ai gYSqxdYQ0cXLyu2XzCARC0BSUETt/g== =zJTm -----END PGP SIGNATURE----- --ztliwx3eij5ch7ku--