Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: K15dS+qzzbY1xjhLJL7gX16f92HHzS9PV9OYHL1A/6qcMySbagr3Bu+xptlD+wjBi7Fr7HgxLT
 hThTAT6UqAMC6uybEvvEjiCWDl53Dz7mvAqvRShRnsLcTBHVCyOudLy0Y+427EwHvKbW2LeiKE
 4g3pyGhVbimKszgkab2GxvCN05jmqJCM4sPqjCJxgqVxJrgSughqNdjaKUz0nluj30oSMlaa4c
 d6bz5ZizMRIf9LKJp7XbM/FQYxL4g8O3G828695PSSHXhqgIaBCrnO1eDOnj10t6ukKyoeH9Zu
 zqfwpqtOKm+6oe95suo1tK3n
Subject: Re: [PATCH] ARM: dts: at91: sama5d2_som1_ek: disable ISC node by
 default
To:     Eugen Hristev <eugen.hristev@microchip.com>
CC:     <robh+dt@kernel.org>, <ludovic.desroches@microchip.com>,
        <devicetree@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-kernel@vger.kernel.org>
References: <20210902121358.503589-1-eugen.hristev@microchip.com>
From:   Nicolas Ferre <nicolas.ferre@microchip.com>
Organization: microchip
Message-ID: <c3afbdb3-ad68-f277-7405-9e98af1c8f97@microchip.com>
Date:   Tue, 14 Sep 2021 17:03:16 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.11.0
MIME-Version: 1.0
In-Reply-To: <20210902121358.503589-1-eugen.hristev@microchip.com>
Content-Type: text/plain; charset="windows-1252"; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 02/09/2021 at 14:13, Eugen Hristev wrote:
> Without a sensor node, the ISC will simply fail to probe, as the
> corresponding port node is missing.
> It is then logical to disable the node in the devicetree.
> If we add a port with a connection to a sensor endpoint, ISC can be enabled.
> 
> Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>

Makes sense:
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>

And queued for at91-fixes in 5.15 as there is no point continuing to 
have a failing probe on this board.

Thanks, best regards,
   Nicolas

> ---
>   arch/arm/boot/dts/at91-sama5d27_som1_ek.dts | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts b/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
> index 614999dcb990..cd4672501add 100644
> --- a/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
> +++ b/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
> @@ -71,7 +71,6 @@ apb {
>   			isc: isc@f0008000 {
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_isc_base &pinctrl_isc_data_8bit &pinctrl_isc_data_9_10 &pinctrl_isc_data_11_12>;
> -				status = "okay";
>   			};
>   
>   			qspi1: spi@f0024000 {
> 


-- 
Nicolas Ferre