[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 02/10] hw/arm/boot: Fix devicetree warning about the PSCI nod
From: |
Peter Maydell |
Subject: |
Re: [PATCH 02/10] hw/arm/boot: Fix devicetree warning about the PSCI node |
Date: |
Wed, 24 Aug 2022 20:33:54 +0100 |
On Wed, 24 Aug 2022 at 16:51, Jean-Philippe Brucker
<jean-philippe@linaro.org> wrote:
>
> dt-validate warns that an implementation compatible with arm,psci-1.0
> shouldn't have arm,psci in their compatible string.
>
> psci: compatible: 'oneOf' conditional failed, one must be fixed:
> ['arm,psci-1.0', 'arm,psci-0.2', 'arm,psci'] is too long
> From schema: linux/Documentation/devicetree/bindings/arm/psci.yaml
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
> hw/arm/boot.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/arm/boot.c b/hw/arm/boot.c
> index ada2717f76..527918227e 100644
> --- a/hw/arm/boot.c
> +++ b/hw/arm/boot.c
> @@ -493,7 +493,7 @@ static void fdt_add_psci_node(void *fdt)
> const char comp[] = "arm,psci-0.2\0arm,psci";
> qemu_fdt_setprop(fdt, "/psci", "compatible", comp, sizeof(comp));
> } else {
> - const char comp[] = "arm,psci-1.0\0arm,psci-0.2\0arm,psci";
> + const char comp[] = "arm,psci-1.0\0arm,psci-0.2";
> qemu_fdt_setprop(fdt, "/psci", "compatible", comp, sizeof(comp));
> }
This doesn't look right.
Documentation/devicetree/bindings/arm/psci.yaml says
"arm,psci-1.0" means "complies to PSCI 1.0",
"arm,psci-0.2" means "complies to PSCI 0.2",
and "arm,psci" means "complies to pre-0.2 PSCI"
If you want to drop "arm,psci" then you should be arguing why
we're not compliant with pre-0.2 PSCI. Maybe we aren't and we
shouldn't be advertising it, but you need more rationale than
"dt-validate complained".
thanks
-- PMM
- [PATCH 00/10] hw/arm/virt: Fix dt-schema warnings, Jean-Philippe Brucker, 2022/08/24
- [PATCH 01/10] hw/arm/virt: Fix devicetree warning about the root node, Jean-Philippe Brucker, 2022/08/24
- [PATCH 06/10] hw/arm/virt: Fix devicetree warning about the gpio-key node, Jean-Philippe Brucker, 2022/08/24
- [PATCH 02/10] hw/arm/boot: Fix devicetree warning about the PSCI node, Jean-Philippe Brucker, 2022/08/24
- Re: [PATCH 02/10] hw/arm/boot: Fix devicetree warning about the PSCI node,
Peter Maydell <=
- [PATCH 07/10] hw/arm/virt: Fix devicetree warnings about node names, Jean-Philippe Brucker, 2022/08/24
- [PATCH 03/10] hw/arm/virt: Fix devicetree warnings about the GIC node, Jean-Philippe Brucker, 2022/08/24
- [PATCH 04/10] hw/arm/virt: Use "msi-map" devicetree property for PCI, Jean-Philippe Brucker, 2022/08/24
- [PATCH 05/10] hw/arm/virt: Fix devicetree warning about the timer node, Jean-Philippe Brucker, 2022/08/24
- [PATCH 10/10] hw/arm/virt: Fix devicetree warnings about the virtio-iommu node, Jean-Philippe Brucker, 2022/08/24
- [PATCH 09/10] hw/arm/virt: Fix devicetree warnings about the SMMU node, Jean-Philippe Brucker, 2022/08/24