Re: [PATCH] samsung: common: ignore if CROS EC is not supported

2023-02-16 Thread Henrik Grimler
Hi Stefan,

On Wed, Feb 15, 2023 at 08:03:14PM +0100, Stefan Agner wrote:
> If the CROS device class is not compiled in, uclass returns not
> supported. Ignore this case as well.
> 
> This avoids boot failures on ODROID-XU4 without CONFIG_CROS_EC
> ending with:
> cros-ec communications failure -96
> Please reset with Power+Refresh
> Cannot init cros-ec device

Thanks, the same issue happens on odroid-u2 (and probably other
devices as well), issue was reported in
https://lists.denx.de/pipermail/u-boot/2023-January/504115.html, and I
sent an identical patch in
https://lists.denx.de/pipermail/u-boot/2023-February/508929.html.

Best regards,
Henrik Grimler

> Signed-off-by: Stefan Agner 
> ---
>  board/samsung/common/board.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/board/samsung/common/board.c b/board/samsung/common/board.c
> index 16ce5cb892..663d7ca991 100644
> --- a/board/samsung/common/board.c
> +++ b/board/samsung/common/board.c
> @@ -223,7 +223,7 @@ int board_late_init(void)
>   char mmcbootdev_str[16];
>  
>   ret = uclass_first_device_err(UCLASS_CROS_EC, );
> - if (ret && ret != -ENODEV) {
> + if (ret && ret != -ENODEV && ret != -EPFNOSUPPORT) {
>   /* Force console on */
>   gd->flags &= ~GD_FLG_SILENT;
>  
> -- 
> 2.39.1
> 


Re: [PATCH] samsung: common: ignore if CROS EC is not supported

2023-02-15 Thread Simon Glass
On Wed, 15 Feb 2023 at 12:03, Stefan Agner  wrote:
>
> If the CROS device class is not compiled in, uclass returns not
> supported. Ignore this case as well.
>
> This avoids boot failures on ODROID-XU4 without CONFIG_CROS_EC
> ending with:
> cros-ec communications failure -96
> Please reset with Power+Refresh
> Cannot init cros-ec device
>
> Signed-off-by: Stefan Agner 
> ---
>  board/samsung/common/board.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass