From: Conor Dooley conor.dooley@microchip.com
Aurelien reported probe failures due to the csi node being enabled without having a camera attached to it. A camera was in the initial submissions, but was removed from the dts, as it had not actually been present on the board, but was from an addon board used by the developer of the relevant drivers. The non-camera pipeline nodes were not disabled when this happened and the probe failures are problematic for Debian. Disable them.
CC: stable@vger.kernel.org Fixes: 28ecaaa5af192 ("riscv: dts: starfive: jh7110: Add camera subsystem nodes") Closes: https://lore.kernel.org/all/Zw1-vcN4CoVkfLjU@aurel32.net/ Reported-by: Aurelien Jarno aurelien@aurel32.net Signed-off-by: Conor Dooley conor.dooley@microchip.com --- CC: Emil Renner Berthing kernel@esmil.dk CC: Rob Herring robh@kernel.org CC: Krzysztof Kozlowski krzk+dt@kernel.org CC: Conor Dooley conor+dt@kernel.org CC: Changhuang Liang changhuang.liang@starfivetech.com CC: devicetree@vger.kernel.org CC: linux-riscv@lists.infradead.org CC: linux-kernel@vger.kernel.org --- arch/riscv/boot/dts/starfive/jh7110-common.dtsi | 2 -- 1 file changed, 2 deletions(-)
diff --git a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi index c7771b3b64758..d6c55f1cc96a9 100644 --- a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi +++ b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi @@ -128,7 +128,6 @@ &camss { assigned-clocks = <&ispcrg JH7110_ISPCLK_DOM4_APB_FUNC>, <&ispcrg JH7110_ISPCLK_MIPI_RX0_PXL>; assigned-clock-rates = <49500000>, <198000000>; - status = "okay";
ports { #address-cells = <1>; @@ -151,7 +150,6 @@ camss_from_csi2rx: endpoint { &csi2rx { assigned-clocks = <&ispcrg JH7110_ISPCLK_VIN_SYS>; assigned-clock-rates = <297000000>; - status = "okay";
ports { #address-cells = <1>;
Hi,
On 2024-10-16 21:11, Conor Dooley wrote:
From: Conor Dooley conor.dooley@microchip.com
Aurelien reported probe failures due to the csi node being enabled without having a camera attached to it. A camera was in the initial submissions, but was removed from the dts, as it had not actually been present on the board, but was from an addon board used by the developer of the relevant drivers. The non-camera pipeline nodes were not disabled when this happened and the probe failures are problematic for Debian. Disable them.
CC: stable@vger.kernel.org Fixes: 28ecaaa5af192 ("riscv: dts: starfive: jh7110: Add camera subsystem nodes") Closes: https://lore.kernel.org/all/Zw1-vcN4CoVkfLjU@aurel32.net/ Reported-by: Aurelien Jarno aurelien@aurel32.net Signed-off-by: Conor Dooley conor.dooley@microchip.com
CC: Emil Renner Berthing kernel@esmil.dk CC: Rob Herring robh@kernel.org CC: Krzysztof Kozlowski krzk+dt@kernel.org CC: Conor Dooley conor+dt@kernel.org CC: Changhuang Liang changhuang.liang@starfivetech.com CC: devicetree@vger.kernel.org CC: linux-riscv@lists.infradead.org CC: linux-kernel@vger.kernel.org
arch/riscv/boot/dts/starfive/jh7110-common.dtsi | 2 -- 1 file changed, 2 deletions(-)
Thanks for picking up that issue. I confirm this fix the "problem".
To give some more details, the problem for Debian is that it appears as an error, with the line in red in dmesg or journalctl. However we would like to be able to provide a kernel with such drivers enabled so that users with a camera can just use an overlay or patch their device tree.
Tested-by: Aurelien Jarno aurelien@aurel32.net Reviewed-by: Aurelien Jarno aurelien@aurel32.net
Regards Aurelien
Conor Dooley wrote:
From: Conor Dooley conor.dooley@microchip.com
Aurelien reported probe failures due to the csi node being enabled without having a camera attached to it. A camera was in the initial submissions, but was removed from the dts, as it had not actually been present on the board, but was from an addon board used by the developer of the relevant drivers. The non-camera pipeline nodes were not disabled when this happened and the probe failures are problematic for Debian. Disable them.
CC: stable@vger.kernel.org Fixes: 28ecaaa5af192 ("riscv: dts: starfive: jh7110: Add camera subsystem nodes") Closes: https://lore.kernel.org/all/Zw1-vcN4CoVkfLjU@aurel32.net/ Reported-by: Aurelien Jarno aurelien@aurel32.net Signed-off-by: Conor Dooley conor.dooley@microchip.com
Thanks!
Reviewed-by: Emil Renner Berthing emil.renner.berthing@canonical.com
CC: Emil Renner Berthing kernel@esmil.dk CC: Rob Herring robh@kernel.org CC: Krzysztof Kozlowski krzk+dt@kernel.org CC: Conor Dooley conor+dt@kernel.org CC: Changhuang Liang changhuang.liang@starfivetech.com CC: devicetree@vger.kernel.org CC: linux-riscv@lists.infradead.org CC: linux-kernel@vger.kernel.org
arch/riscv/boot/dts/starfive/jh7110-common.dtsi | 2 -- 1 file changed, 2 deletions(-)
diff --git a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi index c7771b3b64758..d6c55f1cc96a9 100644 --- a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi +++ b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi @@ -128,7 +128,6 @@ &camss { assigned-clocks = <&ispcrg JH7110_ISPCLK_DOM4_APB_FUNC>, <&ispcrg JH7110_ISPCLK_MIPI_RX0_PXL>; assigned-clock-rates = <49500000>, <198000000>;
status = "okay";
ports { #address-cells = <1>;
@@ -151,7 +150,6 @@ camss_from_csi2rx: endpoint { &csi2rx { assigned-clocks = <&ispcrg JH7110_ISPCLK_VIN_SYS>; assigned-clock-rates = <297000000>;
status = "okay";
ports { #address-cells = <1>;
-- 2.45.2
From: Conor Dooley conor.dooley@microchip.com
On Wed, 16 Oct 2024 21:11:15 +0100, Conor Dooley wrote:
Aurelien reported probe failures due to the csi node being enabled without having a camera attached to it. A camera was in the initial submissions, but was removed from the dts, as it had not actually been present on the board, but was from an addon board used by the developer of the relevant drivers. The non-camera pipeline nodes were not disabled when this happened and the probe failures are problematic for Debian. Disable them.
[...]
Applied to riscv-soc-fixes, thanks!
[1/1] riscv: dts: starfive: disable unused csi/camss nodes https://git.kernel.org/conor/c/2e11e78667db
Thanks, Conor.
linux-stable-mirror@lists.linaro.org
-
Aurelien Jarno -
Conor Dooley -
Emil Renner Berthing