[U-Boot] [PATCH 1/8] imx-common: add board_video_skip

[Stefano Babic]

Hi Eric,

On 02/04/2014 21:57, Eric Bénard wrote:
> this function is shared by several board and thus can be factorized
> 
> Signed-off-by: Eric Bénard <eric at eukrea.com>
> ---
>  arch/arm/imx-common/Makefile            |  1 +
>  arch/arm/imx-common/video.c             | 55 +++++++++++++++++++++++++++++++++
>  arch/arm/include/asm/imx-common/video.h | 20 ++++++++++++
>  3 files changed, 76 insertions(+)
>  create mode 100644 arch/arm/imx-common/video.c
>  create mode 100644 arch/arm/include/asm/imx-common/video.h
> 
> diff --git a/arch/arm/imx-common/Makefile b/arch/arm/imx-common/Makefile
> index 16809fe..2a7fc42 100644
> --- a/arch/arm/imx-common/Makefile
> +++ b/arch/arm/imx-common/Makefile
> @@ -19,6 +19,7 @@ obj-y	+= misc.o
>  endif
>  ifeq ($(SOC),$(filter $(SOC),mx6))
>  obj-$(CONFIG_CMD_SATA) += sata.o
> +obj-$(CONFIG_IMX_VIDEO_SKIP) += video.o
>  endif
>  obj-$(CONFIG_CMD_BMODE) += cmd_bmode.o
>  obj-$(CONFIG_CMD_HDMIDETECT) += cmd_hdmidet.o
> diff --git a/arch/arm/imx-common/video.c b/arch/arm/imx-common/video.c
> new file mode 100644
> index 0000000..8db5a84
> --- /dev/null
> +++ b/arch/arm/imx-common/video.c
> @@ -0,0 +1,55 @@
> +/*
> + * SPDX-License-Identifier:	GPL-2.0+
> + */
> +
> +#include <common.h>
> +#include <asm/errno.h>
> +#include <asm/imx-common/video.h>
> +
> +extern struct display_info_t const displays[];
> +extern size_t display_number;
> +
> +int board_video_skip(void)
> +{
> +	int i;
> +	int ret;
> +	char const *panel = getenv("panel");
> +	if (!panel) {
> +		for (i = 0; i < display_number; i++) {
> +			struct display_info_t const *dev = displays+i;
> +			if (dev->detect && dev->detect(dev)) {
> +				panel = dev->mode.name;
> +				printf("auto-detected panel %s\n", panel);
> +				break;
> +			}
> +		}
> +		if (!panel) {
> +			panel = displays[0].mode.name;
> +			printf("No panel detected: default to %s\n", panel);
> +			i = 0;
> +		}
> +	} else {
> +		for (i = 0; i < display_number; i++) {
> +			if (!strcmp(panel, displays[i].mode.name))
> +				break;
> +		}
> +	}
> +	if (i < display_number) {
> +		ret = ipuv3_fb_init(&displays[i].mode, 0,
> +				    displays[i].pixfmt);
> +		if (!ret) {
> +			displays[i].enable(displays+i);
> +			printf("Display: %s (%ux%u)\n",
> +			       displays[i].mode.name,
> +			       displays[i].mode.xres,
> +			       displays[i].mode.yres);
> +		} else
> +			printf("LCD %s cannot be configured: %d\n",
> +			       displays[i].mode.name, ret);
> +	} else {
> +		printf("unsupported panel %s\n", panel);
> +		return -EINVAL;
> +	}
> +
> +	return 0;
> +}
> diff --git a/arch/arm/include/asm/imx-common/video.h b/arch/arm/include/asm/imx-common/video.h
> new file mode 100644
> index 0000000..e0c4ef4
> --- /dev/null
> +++ b/arch/arm/include/asm/imx-common/video.h
> @@ -0,0 +1,20 @@
> +/*
> + * SPDX-License-Identifier:	GPL-2.0+
> + */
> +
> +#ifndef __IMX_VIDEO_H_
> +#define __IMX_VIDEO_H_
> +
> +#include <linux/fb.h>
> +#include <ipu_pixfmt.h>
> +
> +struct display_info_t {
> +	int	bus;
> +	int	addr;
> +	int	pixfmt;
> +	int	(*detect)(struct display_info_t const *dev);
> +	void	(*enable)(struct display_info_t const *dev);
> +	struct	fb_videomode mode;
> +};
> +
> +#endif
> 

Nice clean up, thanks for this !

Acked-by: Stefano Babic <sbabic at denx.de>

Regards,
Stefano


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================