Re: [PATCH] 85xx: Add several root nodes to probe

[Christophe Leroy <christophe.leroy@csgroup.eu>]

Le 22/04/2025 à 04:40, Ben Collins a écrit :
> T4240 fails to hafve ifc, rapidio, and localbus probed.
> 
> This matches other QorIQ platforms and ensures devices under these nodes
> get added as platform devices.

What do you mean by "this matches other platforms" ? I can't grep any 
other platform file with "fsl,ifc" or "rapidio"

Christophe

> 
> Signed-off-by: Ben Collins <bcollins@kernel.org>
> Cc: Scott Wood <oss@buserror.net>
> Cc: Madhavan Srinivasan <maddy@linux.ibm.com>
> Cc: Michael Ellerman <mpe@ellerman.id.au>
> Cc: linuxppc-dev@lists.ozlabs.org
> Cc: linux-kernel@vger.kernel.org
> ---
>   arch/powerpc/platforms/85xx/corenet_generic.c | 9 +++++++++
>   1 file changed, 9 insertions(+)
> 
> diff --git a/arch/powerpc/platforms/85xx/corenet_generic.c b/arch/powerpc/platforms/85xx/corenet_generic.c
> index c44400e95f551..c81b8a47f3b0f 100644
> --- a/arch/powerpc/platforms/85xx/corenet_generic.c
> +++ b/arch/powerpc/platforms/85xx/corenet_generic.c
> @@ -61,6 +61,9 @@ static const struct of_device_id of_device_ids[] = {
>   	{
>   		.compatible	= "simple-bus"
>   	},
> +	{
> +		.compatible	= "fsl,ifc"
> +	},
>   	{
>   		.compatible	= "mdio-mux-gpio"
>   	},
> @@ -91,6 +94,12 @@ static const struct of_device_id of_device_ids[] = {
>   	{
>   		.compatible	= "fsl,qe",
>   	},
> +	{
> +		.name           = "localbus",
> +	},
> +	{
> +		.name		= "rapidio",
> +	},
>   	/* The following two are for the Freescale hypervisor */
>   	{
>   		.name		= "hypervisor",