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 <e...@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 <sba...@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: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot