From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr2-f12.google.com (mail-wr2-f12.google.com [74.125.225.76]) (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 8EE335452B3 for ; Tue, 22 Sep 2026 12:53:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.225.76 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790081607; cv=none; b=Zlf23kjWT1iZyHTWysKEERs3pUjKw6ocTZGqoyZC3PRnLuGZFK53vI/zO5PlpZhXk9IDDXEHZn39JAfWadAs9wH9wJ409AUe/xYAnRg1HXp3+uK0gn6ds+0CbVHTeUBq5oLrzgYLKRuQ79EuxQjrJ6oE4qX+zvFYd+p0mVQs+9Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790081607; c=relaxed/simple; bh=v4Ul7xXqzvPTm1KnAhXO58P90eD2KlGTNsegXWmzADU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=E8NnFVF2GOgPGAizl0mzdO0CRZh3302VtmSTiZF6pdz8fI/kRorNVC3s3EVuf8als9kt1/SOV0mqVbXnbSWls4EkOefhTqsHCiDKbAr7JDcHpsU9x00v0xhG1qTGGNagPxokWXodoyr5KIFX09W7opNZOPWa16HUxukKyxmlR1w= 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.76 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-f12.google.com with SMTP id ffacd0b85a97d-482f62ccdb1so1434412f8f.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=lPRWj9jpdYdJVPKROqgGYJ272KwZkC9lvPuriOUO3MBV3MQDdRq95MB6gnZebKKp9t F00lMfu2zFEJ2eUA12kHF67KQEbRSscjz2PJYiKQz2zUpCjtJoMGSY2Fp7JWd99W7bwH wO5V8UgI7f0fLFLnoOpx4LIK2ESfPlA9Tp9lEvzHnKPbsJQBNLMddAMWV3e8QL1/Visu EKDLhJggOihyVgdRKKcL3MYgi4oXLcJQpS9nJk5bup4fP6l8g61htAQouVFQNGzG07vk b5bxvkDzLsLbdKfHen7eEMBuxLEre8D4faB2sDDLQOYZmFvjc/4po5P2qGpdggrNrUT3 xLyg== X-Forwarded-Encrypted: i=1; AKwUvBzou4MfjTeERbG7c36JlvsKVy1j2GHGJXU0wzZbkN8W5CCCXxiHlCXt2MRXTMaYyI7kFKFC+s4oMm0awdE=@vger.kernel.org X-Gm-Message-State: AFuF++khlSv0gIPgSxonUsksPXafFFqzOuVAM0cYshX3sYFs144SNJtW Jhn+pcRpAiYvgYD9u9SrUy78rlT0cEVWNlB9K6hWHKNXF48d7KCQ5tVj50yN1apZ1fU= X-Gm-Gg: AYBFou3cPL6eqGhh4VpyPYZcMlamNRMNf4XqnfMXa1ddTsZGcpEfxNhnBxfZoaLy2cJ WOHecv3AGCcvTTSyLhRsLm6uXxIb6S2fsXzPUx60MUUpTiMRoyqppkayCS/1xZ87jBO5vxzutXj JZP+L8z2Ur9jF/p1N/z1rdBKczeOpFZyvYCbM54XM+Ei7pzly5wGyYYV/I36rROOq0ceqSqUsHJ fY8ETzcVPEsUdM3gLwJWKKgY54zKDFp0mfkgnchghoc/WAE4vZSfCFrokzBDomJ7/Jrc/bhWwT5 pmV5KYlgfCtQ2HDBk9ONlUAF4DH6MggWnK6x1Rw4CQWVL1fkImG9+oEKYzQvNT07/msiff3lrSr ri4WcffFWwz3yAs+EcnCR468epzVGtHPqx6hCU/kpL+SicuD/ohz135wz5JCVxMRGie3wWowVg5 psg10MOtJsa3HlRsxW8IXfZDutlAsfRs3STV6t49Zs5iFjFwpKLNeuUMMNbk3p/Uub7JKuarUqe tWb3ZXyQTw612EBGCt3pdM1ahFBkm9jHXYlWvZp1lTZHea24wlyVqZMHqAwaQ== 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: linux-kernel@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--