From mboxrd@z Thu Jan  1 00:00:00 1970
From: York Sun <yorksun@freescale.com>
Date: Thu, 14 Aug 2014 10:17:35 -0700
Subject: [U-Boot] [PATCH v4 16/18] video: dcu: Add Sii9022A HDMI
	Transmitter support
In-Reply-To: <1407998707-9632-17-git-send-email-b18965@freescale.com>
References: <1407998707-9632-1-git-send-email-b18965@freescale.com>
	<1407998707-9632-17-git-send-email-b18965@freescale.com>
Message-ID: <53ECEF2F.2060603@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 08/13/2014 11:45 PM, Alison Wang wrote:
> On LS1021ATWR, Silicon's Sii9022A HDMI Transmitter
> is used. This patch adds the common setting for this
> chip.
> 
> Signed-off-by: Alison Wang <alison.wang@freescale.com>
> ---
> Change log:
>  v4: Add commit messages.
>  v3: New file.
> 
>  board/freescale/common/Makefile       |   2 +
>  board/freescale/common/dcu_sii9022a.c | 153 ++++++++++++++++++++++++++++++++++
>  board/freescale/common/dcu_sii9022a.h |  13 +++
>  3 files changed, 168 insertions(+)
>  create mode 100644 board/freescale/common/dcu_sii9022a.c
>  create mode 100644 board/freescale/common/dcu_sii9022a.h

Shouldn't this driver stay under driver/video? It is for the HDMI transmitter,
not Freescale board specific, is it?

> 
> diff --git a/board/freescale/common/Makefile b/board/freescale/common/Makefile
> index 22b57cc..808ddd0 100644
> --- a/board/freescale/common/Makefile
> +++ b/board/freescale/common/Makefile
> @@ -34,6 +34,8 @@ ifndef CONFIG_RAMBOOT_PBL
>  obj-$(CONFIG_FSL_FIXED_MMC_LOCATION)	+= sdhc_boot.o
>  endif
>  
> +obj-$(CONFIG_FSL_DCU_SII9022A)    += dcu_sii9022a.o
> +
>  obj-$(CONFIG_MPC8541CDS)	+= cds_pci_ft.o
>  obj-$(CONFIG_MPC8548CDS)	+= cds_pci_ft.o
>  obj-$(CONFIG_MPC8555CDS)	+= cds_pci_ft.o
> diff --git a/board/freescale/common/dcu_sii9022a.c b/board/freescale/common/dcu_sii9022a.c
> new file mode 100644
> index 0000000..2da627e
> --- /dev/null
> +++ b/board/freescale/common/dcu_sii9022a.c
> @@ -0,0 +1,153 @@
> +/*
> + * Copyright 2014 Freescale Semiconductor, Inc.
> + *
> + * SPDX-License-Identifier:	GPL-2.0+
> + */
> +
> +#include <asm/io.h>
> +#include <common.h>
> +#include <fsl_dcu_fb.h>
>
I think you put the patches in wrong order. DCU driver has not been added before
this patch.

York