Re: [PATCH v4 4/4] arm64: dts: broadcom: Add support for BCM2712

From: Andrea della Porta <andrea.porta@suse.com>
To: Stefan Wahren <wahrenst@gmx.net>
Cc: Andrea della Porta <andrea.porta@suse.com>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Florian Fainelli <florian.fainelli@broadcom.com>,
	Ray Jui <rjui@broadcom.com>,
	Scott Branden <sbranden@broadcom.com>,
	Broadcom internal kernel review list
	<bcm-kernel-feedback-list@broadcom.com>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	Adrian Hunter <adrian.hunter@intel.com>,
	Kamal Dasu <kamal.dasu@broadcom.com>,
	Al Cooper <alcooperx@gmail.com>,
	devicetree@vger.kernel.org, linux-rpi-kernel@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-mmc@vger.kernel.org
Subject: Re: [PATCH v4 4/4] arm64: dts: broadcom: Add support for BCM2712
Date: Thu, 30 May 2024 11:24:00 +0200	[thread overview]
Message-ID: <ZlhFsGw7PaseT_08@apocalypse> (raw)
In-Reply-To: <f94cf0fb-a9a2-4447-9b32-7f09c2a37cf6@gmx.net>

Hi Stefan,

On 08:11 Thu 30 May     , Stefan Wahren wrote:
[cut...] 
> > +	clocks {
> > +		/* The oscillator is the root of the clock tree. */
> > +		clk_osc: clk-osc {
> > +			compatible = "fixed-clock";
> > +			#clock-cells = <0>;
> > +			clock-output-names = "osc";
> > +			clock-frequency = <54000000>;
> > +		};
> > +
> > +		clk_vpu: clk-vpu {
> > +			#clock-cells = <0>;
> > +			compatible = "fixed-clock";
> > +			clock-frequency = <750000000>;
> > +			clock-output-names = "vpu-clock";
> > +		};
> Is the VPU clock really fixed or is it just a workaround for minimal
> boot support?

Unfortunately I don't have enough insight on this due to lack of documentation.
I had to take it by faith from downstream dts.

Many thanks,
Andrea