Re: [PATCH] ARM: dts: socfpga: remove syscon compatible string for sysmgr node

[Dinh Nguyen <dinguyen@kernel.org>]

On 1/17/25 09:42, niravkumar.l.rabara@intel.com wrote:
> From: Niravkumar L Rabara <niravkumar.l.rabara@intel.com>
> 
> The SoCFPGA System Manager(sysmgr) dt bindings do not use the syscon
> compitible, nor does the Linux system manager driver rely on it.
> Remove "syscon" for Arria5, Arria10 and Cyclon5 sysmgr node and fixed
> dtbs_check warnings like:
> 
> socfpga_arria5_socdk.dtb: sysmgr@ffd08000: compatible: 'oneOf' conditional failed, one must be fixed:
>     ['altr,sys-mgr', 'syscon'] is too long
>     'altr,sys-mgr-s10' was expected
>     'altr,sys-mgr' was expected
>     from schema $id: http://devicetree.org/schemas/soc/altera/altr,sys-mgr.yaml#
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202501102323.Xnte2yhi-lkp@intel.com/
> Signed-off-by: Niravkumar L Rabara <niravkumar.l.rabara@intel.com>
> ---
>   arch/arm/boot/dts/intel/socfpga/socfpga.dtsi         | 2 +-
>   arch/arm/boot/dts/intel/socfpga/socfpga_arria10.dtsi | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/intel/socfpga/socfpga.dtsi b/arch/arm/boot/dts/intel/socfpga/socfpga.dtsi
> index 35be14150f41..f124fb72e260 100644
> --- a/arch/arm/boot/dts/intel/socfpga/socfpga.dtsi
> +++ b/arch/arm/boot/dts/intel/socfpga/socfpga.dtsi
> @@ -853,7 +853,7 @@ spi1: spi@fff01000 {
>   		};
>   
>   		sysmgr: sysmgr@ffd08000 {
> -			compatible = "altr,sys-mgr", "syscon";
> +			compatible = "altr,sys-mgr";
>   			reg = <0xffd08000 0x4000>;
>   		};
>   
> diff --git a/arch/arm/boot/dts/intel/socfpga/socfpga_arria10.dtsi b/arch/arm/boot/dts/intel/socfpga/socfpga_arria10.dtsi
> index 6b6e77596ffa..015120fb4b02 100644
> --- a/arch/arm/boot/dts/intel/socfpga/socfpga_arria10.dtsi
> +++ b/arch/arm/boot/dts/intel/socfpga/socfpga_arria10.dtsi
> @@ -792,7 +792,7 @@ scu: snoop-control-unit@ffffc000 {
>   		};
>   
>   		sysmgr: sysmgr@ffd06000 {
> -			compatible = "altr,sys-mgr", "syscon";
> +			compatible = "altr,sys-mgr";
>   			reg = <0xffd06000 0x300>;
>   			cpu1-start-addr = <0xffd06230>;
>   		};

Did you test this patch on actual hardware? Unless something has changed 
in the system manager driver, this will probably cause the system hang.

Dinh