All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2 04/14] imx: imx-common: move lcdif structure and macro definition to imx-common
Date: Tue, 20 Oct 2015 15:15:19 +0200	[thread overview]
Message-ID: <56263E67.2050801@denx.de> (raw)
In-Reply-To: <1445341184-22272-5-git-send-email-Peng.Fan@freescale.com>

Hi Peng,

On 20/10/2015 13:39, Peng Fan wrote:
> Move 'struct mxs_lcdif_regs' and lcdif related macro definitions to
> arch/arm/include/asm/imx-common/regs-lcdif.h.
> 
> i.MX6SX/UL/28 have similar register layout and bit definitions of registers.
> 

I do not like this approach. Please split this patch into two: the first
patch moves only into imx-common (and please use -M in format-patch to
detect renaming), and the second one changes what you need to change.
There is then no trace about changes in commit.

> Signed-off-by: Peng Fan <Peng.Fan@freescale.com>
> Cc: Stefano Babic <sbabic@denx.de>
> ---
> 
> V2:
>  none
> 
>  arch/arm/include/asm/arch-mxs/regs-lcdif.h   | 201 +-----------------------
>  arch/arm/include/asm/imx-common/regs-lcdif.h | 222 +++++++++++++++++++++++++++
>  2 files changed, 223 insertions(+), 200 deletions(-)
>  create mode 100644 arch/arm/include/asm/imx-common/regs-lcdif.h
> 
> diff --git a/arch/arm/include/asm/arch-mxs/regs-lcdif.h b/arch/arm/include/asm/arch-mxs/regs-lcdif.h
> index 8915d84..bee1f9a 100644
> --- a/arch/arm/include/asm/arch-mxs/regs-lcdif.h
> +++ b/arch/arm/include/asm/arch-mxs/regs-lcdif.h
> @@ -13,205 +13,6 @@
>  #ifndef __MX28_REGS_LCDIF_H__
>  #define __MX28_REGS_LCDIF_H__
>  
> -#include <asm/imx-common/regs-common.h>
> -
> -#ifndef	__ASSEMBLY__
> -struct mxs_lcdif_regs {
> -	mxs_reg_32(hw_lcdif_ctrl)		/* 0x00 */
> -	mxs_reg_32(hw_lcdif_ctrl1)		/* 0x10 */
> -#if defined(CONFIG_MX28)
> -	mxs_reg_32(hw_lcdif_ctrl2)		/* 0x20 */
> -#endif
> -	mxs_reg_32(hw_lcdif_transfer_count)	/* 0x20/0x30 */
> -	mxs_reg_32(hw_lcdif_cur_buf)		/* 0x30/0x40 */
> -	mxs_reg_32(hw_lcdif_next_buf)		/* 0x40/0x50 */
> -
> -#if defined(CONFIG_MX23)
> -	uint32_t	reserved1[4];
> -#endif
> -
> -	mxs_reg_32(hw_lcdif_timing)		/* 0x60 */
> -	mxs_reg_32(hw_lcdif_vdctrl0)		/* 0x70 */
> -	mxs_reg_32(hw_lcdif_vdctrl1)		/* 0x80 */
> -	mxs_reg_32(hw_lcdif_vdctrl2)		/* 0x90 */
> -	mxs_reg_32(hw_lcdif_vdctrl3)		/* 0xa0 */
> -	mxs_reg_32(hw_lcdif_vdctrl4)		/* 0xb0 */
> -	mxs_reg_32(hw_lcdif_dvictrl0)		/* 0xc0 */
> -	mxs_reg_32(hw_lcdif_dvictrl1)		/* 0xd0 */
> -	mxs_reg_32(hw_lcdif_dvictrl2)		/* 0xe0 */
> -	mxs_reg_32(hw_lcdif_dvictrl3)		/* 0xf0 */
> -	mxs_reg_32(hw_lcdif_dvictrl4)		/* 0x100 */
> -	mxs_reg_32(hw_lcdif_csc_coeffctrl0)	/* 0x110 */
> -	mxs_reg_32(hw_lcdif_csc_coeffctrl1)	/* 0x120 */
> -	mxs_reg_32(hw_lcdif_csc_coeffctrl2)	/* 0x130 */
> -	mxs_reg_32(hw_lcdif_csc_coeffctrl3)	/* 0x140 */
> -	mxs_reg_32(hw_lcdif_csc_coeffctrl4)	/* 0x150 */
> -	mxs_reg_32(hw_lcdif_csc_offset)	/* 0x160 */
> -	mxs_reg_32(hw_lcdif_csc_limit)		/* 0x170 */
> -
> -#if defined(CONFIG_MX23)
> -	uint32_t	reserved2[12];
> -#endif
> -	mxs_reg_32(hw_lcdif_data)		/* 0x1b0/0x180 */
> -	mxs_reg_32(hw_lcdif_bm_error_stat)	/* 0x1c0/0x190 */
> -#if defined(CONFIG_MX28)
> -	mxs_reg_32(hw_lcdif_crc_stat)		/* 0x1a0 */
> -#endif
> -	mxs_reg_32(hw_lcdif_lcdif_stat)		/* 0x1d0/0x1b0 */
> -	mxs_reg_32(hw_lcdif_version)		/* 0x1e0/0x1c0 */
> -	mxs_reg_32(hw_lcdif_debug0)		/* 0x1f0/0x1d0 */
> -	mxs_reg_32(hw_lcdif_debug1)		/* 0x200/0x1e0 */
> -	mxs_reg_32(hw_lcdif_debug2)		/* 0x1f0 */
> -};
> -#endif
> -
> -#define	LCDIF_CTRL_SFTRST					(1 << 31)
> -#define	LCDIF_CTRL_CLKGATE					(1 << 30)
> -#define	LCDIF_CTRL_YCBCR422_INPUT				(1 << 29)
> -#define	LCDIF_CTRL_READ_WRITEB					(1 << 28)
> -#define	LCDIF_CTRL_WAIT_FOR_VSYNC_EDGE				(1 << 27)
> -#define	LCDIF_CTRL_DATA_SHIFT_DIR				(1 << 26)
> -#define	LCDIF_CTRL_SHIFT_NUM_BITS_MASK				(0x1f << 21)
> -#define	LCDIF_CTRL_SHIFT_NUM_BITS_OFFSET			21
> -#define	LCDIF_CTRL_DVI_MODE					(1 << 20)
> -#define	LCDIF_CTRL_BYPASS_COUNT					(1 << 19)
> -#define	LCDIF_CTRL_VSYNC_MODE					(1 << 18)
> -#define	LCDIF_CTRL_DOTCLK_MODE					(1 << 17)
> -#define	LCDIF_CTRL_DATA_SELECT					(1 << 16)
> -#define	LCDIF_CTRL_INPUT_DATA_SWIZZLE_MASK			(0x3 << 14)
> -#define	LCDIF_CTRL_INPUT_DATA_SWIZZLE_OFFSET			14
> -#define	LCDIF_CTRL_CSC_DATA_SWIZZLE_MASK			(0x3 << 12)
> -#define	LCDIF_CTRL_CSC_DATA_SWIZZLE_OFFSET			12
> -#define	LCDIF_CTRL_LCD_DATABUS_WIDTH_MASK			(0x3 << 10)
> -#define	LCDIF_CTRL_LCD_DATABUS_WIDTH_OFFSET			10
> -#define	LCDIF_CTRL_LCD_DATABUS_WIDTH_16BIT			(0 << 10)
> -#define	LCDIF_CTRL_LCD_DATABUS_WIDTH_8BIT			(1 << 10)
> -#define	LCDIF_CTRL_LCD_DATABUS_WIDTH_18BIT			(2 << 10)
> -#define	LCDIF_CTRL_LCD_DATABUS_WIDTH_24BIT			(3 << 10)
> -#define	LCDIF_CTRL_WORD_LENGTH_MASK				(0x3 << 8)
> -#define	LCDIF_CTRL_WORD_LENGTH_OFFSET				8
> -#define	LCDIF_CTRL_WORD_LENGTH_16BIT				(0 << 8)
> -#define	LCDIF_CTRL_WORD_LENGTH_8BIT				(1 << 8)
> -#define	LCDIF_CTRL_WORD_LENGTH_18BIT				(2 << 8)
> -#define	LCDIF_CTRL_WORD_LENGTH_24BIT				(3 << 8)
> -#define	LCDIF_CTRL_RGB_TO_YCBCR422_CSC				(1 << 7)
> -#define	LCDIF_CTRL_LCDIF_MASTER					(1 << 5)
> -#define	LCDIF_CTRL_DATA_FORMAT_16_BIT				(1 << 3)
> -#define	LCDIF_CTRL_DATA_FORMAT_18_BIT				(1 << 2)
> -#define	LCDIF_CTRL_DATA_FORMAT_24_BIT				(1 << 1)
> -#define	LCDIF_CTRL_RUN						(1 << 0)
> -
> -#define	LCDIF_CTRL1_COMBINE_MPU_WR_STRB				(1 << 27)
> -#define	LCDIF_CTRL1_BM_ERROR_IRQ_EN				(1 << 26)
> -#define	LCDIF_CTRL1_BM_ERROR_IRQ				(1 << 25)
> -#define	LCDIF_CTRL1_RECOVER_ON_UNDERFLOW			(1 << 24)
> -#define	LCDIF_CTRL1_INTERLACE_FIELDS				(1 << 23)
> -#define	LCDIF_CTRL1_START_INTERLACE_FROM_SECOND_FIELD		(1 << 22)
> -#define	LCDIF_CTRL1_FIFO_CLEAR					(1 << 21)
> -#define	LCDIF_CTRL1_IRQ_ON_ALTERNATE_FIELDS			(1 << 20)
> -#define	LCDIF_CTRL1_BYTE_PACKING_FORMAT_MASK			(0xf << 16)
> -#define	LCDIF_CTRL1_BYTE_PACKING_FORMAT_OFFSET			16
> -#define	LCDIF_CTRL1_OVERFLOW_IRQ_EN				(1 << 15)
> -#define	LCDIF_CTRL1_UNDERFLOW_IRQ_EN				(1 << 14)
> -#define	LCDIF_CTRL1_CUR_FRAME_DONE_IRQ_EN			(1 << 13)
> -#define	LCDIF_CTRL1_VSYNC_EDGE_IRQ_EN				(1 << 12)
> -#define	LCDIF_CTRL1_OVERFLOW_IRQ				(1 << 11)
> -#define	LCDIF_CTRL1_UNDERFLOW_IRQ				(1 << 10)
> -#define	LCDIF_CTRL1_CUR_FRAME_DONE_IRQ				(1 << 9)
> -#define	LCDIF_CTRL1_VSYNC_EDGE_IRQ				(1 << 8)
> -#define	LCDIF_CTRL1_BUSY_ENABLE					(1 << 2)
> -#define	LCDIF_CTRL1_MODE86					(1 << 1)
> -#define	LCDIF_CTRL1_RESET					(1 << 0)
> -
> -#define	LCDIF_CTRL2_OUTSTANDING_REQS_MASK			(0x7 << 21)
> -#define	LCDIF_CTRL2_OUTSTANDING_REQS_OFFSET			21
> -#define	LCDIF_CTRL2_OUTSTANDING_REQS_REQ_1			(0x0 << 21)
> -#define	LCDIF_CTRL2_OUTSTANDING_REQS_REQ_2			(0x1 << 21)
> -#define	LCDIF_CTRL2_OUTSTANDING_REQS_REQ_4			(0x2 << 21)
> -#define	LCDIF_CTRL2_OUTSTANDING_REQS_REQ_8			(0x3 << 21)
> -#define	LCDIF_CTRL2_OUTSTANDING_REQS_REQ_16			(0x4 << 21)
> -#define	LCDIF_CTRL2_BURST_LEN_8					(1 << 20)
> -#define	LCDIF_CTRL2_ODD_LINE_PATTERN_MASK			(0x7 << 16)
> -#define	LCDIF_CTRL2_ODD_LINE_PATTERN_OFFSET			16
> -#define	LCDIF_CTRL2_ODD_LINE_PATTERN_RGB			(0x0 << 16)
> -#define	LCDIF_CTRL2_ODD_LINE_PATTERN_RBG			(0x1 << 16)
> -#define	LCDIF_CTRL2_ODD_LINE_PATTERN_GBR			(0x2 << 16)
> -#define	LCDIF_CTRL2_ODD_LINE_PATTERN_GRB			(0x3 << 16)
> -#define	LCDIF_CTRL2_ODD_LINE_PATTERN_BRG			(0x4 << 16)
> -#define	LCDIF_CTRL2_ODD_LINE_PATTERN_BGR			(0x5 << 16)
> -#define	LCDIF_CTRL2_EVEN_LINE_PATTERN_MASK			(0x7 << 12)
> -#define	LCDIF_CTRL2_EVEN_LINE_PATTERN_OFFSET			12
> -#define	LCDIF_CTRL2_EVEN_LINE_PATTERN_RGB			(0x0 << 12)
> -#define	LCDIF_CTRL2_EVEN_LINE_PATTERN_RBG			(0x1 << 12)
> -#define	LCDIF_CTRL2_EVEN_LINE_PATTERN_GBR			(0x2 << 12)
> -#define	LCDIF_CTRL2_EVEN_LINE_PATTERN_GRB			(0x3 << 12)
> -#define	LCDIF_CTRL2_EVEN_LINE_PATTERN_BRG			(0x4 << 12)
> -#define	LCDIF_CTRL2_EVEN_LINE_PATTERN_BGR			(0x5 << 12)
> -#define	LCDIF_CTRL2_READ_PACK_DIR				(1 << 10)
> -#define	LCDIF_CTRL2_READ_MODE_OUTPUT_IN_RGB_FORMAT		(1 << 9)
> -#define	LCDIF_CTRL2_READ_MODE_6_BIT_INPUT			(1 << 8)
> -#define	LCDIF_CTRL2_READ_MODE_NUM_PACKED_SUBWORDS_MASK		(0x7 << 4)
> -#define	LCDIF_CTRL2_READ_MODE_NUM_PACKED_SUBWORDS_OFFSET	4
> -#define	LCDIF_CTRL2_INITIAL_DUMMY_READ_MASK			(0x7 << 1)
> -#define	LCDIF_CTRL2_INITIAL_DUMMY_READ_OFFSET			1
> -
> -#define	LCDIF_TRANSFER_COUNT_V_COUNT_MASK			(0xffff << 16)
> -#define	LCDIF_TRANSFER_COUNT_V_COUNT_OFFSET			16
> -#define	LCDIF_TRANSFER_COUNT_H_COUNT_MASK			(0xffff << 0)
> -#define	LCDIF_TRANSFER_COUNT_H_COUNT_OFFSET			0
> -
> -#define	LCDIF_CUR_BUF_ADDR_MASK					0xffffffff
> -#define	LCDIF_CUR_BUF_ADDR_OFFSET				0
> -
> -#define	LCDIF_NEXT_BUF_ADDR_MASK				0xffffffff
> -#define	LCDIF_NEXT_BUF_ADDR_OFFSET				0
> -
> -#define	LCDIF_TIMING_CMD_HOLD_MASK				(0xff << 24)
> -#define	LCDIF_TIMING_CMD_HOLD_OFFSET				24
> -#define	LCDIF_TIMING_CMD_SETUP_MASK				(0xff << 16)
> -#define	LCDIF_TIMING_CMD_SETUP_OFFSET				16
> -#define	LCDIF_TIMING_DATA_HOLD_MASK				(0xff << 8)
> -#define	LCDIF_TIMING_DATA_HOLD_OFFSET				8
> -#define	LCDIF_TIMING_DATA_SETUP_MASK				(0xff << 0)
> -#define	LCDIF_TIMING_DATA_SETUP_OFFSET				0
> -
> -#define	LCDIF_VDCTRL0_VSYNC_OEB					(1 << 29)
> -#define	LCDIF_VDCTRL0_ENABLE_PRESENT				(1 << 28)
> -#define	LCDIF_VDCTRL0_VSYNC_POL					(1 << 27)
> -#define	LCDIF_VDCTRL0_HSYNC_POL					(1 << 26)
> -#define	LCDIF_VDCTRL0_DOTCLK_POL				(1 << 25)
> -#define	LCDIF_VDCTRL0_ENABLE_POL				(1 << 24)
> -#define	LCDIF_VDCTRL0_VSYNC_PERIOD_UNIT				(1 << 21)
> -#define	LCDIF_VDCTRL0_VSYNC_PULSE_WIDTH_UNIT			(1 << 20)
> -#define	LCDIF_VDCTRL0_HALF_LINE					(1 << 19)
> -#define	LCDIF_VDCTRL0_HALF_LINE_MODE				(1 << 18)
> -#define	LCDIF_VDCTRL0_VSYNC_PULSE_WIDTH_MASK			0x3ffff
> -#define	LCDIF_VDCTRL0_VSYNC_PULSE_WIDTH_OFFSET			0
> -
> -#define	LCDIF_VDCTRL1_VSYNC_PERIOD_MASK				0xffffffff
> -#define	LCDIF_VDCTRL1_VSYNC_PERIOD_OFFSET			0
> -
> -#if defined(CONFIG_MX23)
> -#define	LCDIF_VDCTRL2_HSYNC_PULSE_WIDTH_MASK			(0xff << 24)
> -#define	LCDIF_VDCTRL2_HSYNC_PULSE_WIDTH_OFFSET			24
> -#elif defined(CONFIG_MX28)
> -#define	LCDIF_VDCTRL2_HSYNC_PULSE_WIDTH_MASK			(0x3fff << 18)
> -#define	LCDIF_VDCTRL2_HSYNC_PULSE_WIDTH_OFFSET			18
> -#endif
> -#define	LCDIF_VDCTRL2_HSYNC_PERIOD_MASK				0x3ffff
> -#define	LCDIF_VDCTRL2_HSYNC_PERIOD_OFFSET			0
> -
> -#define	LCDIF_VDCTRL3_MUX_SYNC_SIGNALS				(1 << 29)
> -#define	LCDIF_VDCTRL3_VSYNC_ONLY				(1 << 28)
> -#define	LCDIF_VDCTRL3_HORIZONTAL_WAIT_CNT_MASK			(0xfff << 16)
> -#define	LCDIF_VDCTRL3_HORIZONTAL_WAIT_CNT_OFFSET		16
> -#define	LCDIF_VDCTRL3_VERTICAL_WAIT_CNT_MASK			(0xffff << 0)
> -#define	LCDIF_VDCTRL3_VERTICAL_WAIT_CNT_OFFSET			0
> -
> -#define	LCDIF_VDCTRL4_DOTCLK_DLY_SEL_MASK			(0x7 << 29)
> -#define	LCDIF_VDCTRL4_DOTCLK_DLY_SEL_OFFSET			29
> -#define	LCDIF_VDCTRL4_SYNC_SIGNALS_ON				(1 << 18)
> -#define	LCDIF_VDCTRL4_DOTCLK_H_VALID_DATA_CNT_MASK		0x3ffff
> -#define	LCDIF_VDCTRL4_DOTCLK_H_VALID_DATA_CNT_OFFSET		0
> +#include <asm/imx-common/regs-lcdif.h>
>  
>  #endif /* __MX28_REGS_LCDIF_H__ */
> diff --git a/arch/arm/include/asm/imx-common/regs-lcdif.h b/arch/arm/include/asm/imx-common/regs-lcdif.h
> new file mode 100644
> index 0000000..b077003
> --- /dev/null
> +++ b/arch/arm/include/asm/imx-common/regs-lcdif.h
> @@ -0,0 +1,222 @@
> +/*
> + * Freescale i.MX28/6SX/6UL LCDIF Register Definitions
> + *
> + * SPDX-License-Identifier:	GPL-2.0+
> + */
> +#ifndef __IMX_REGS_LCDIF_H__
> +#define __IMX_REGS_LCDIF_H__
> +
> +#if !(defined(__KERNEL_STRICT_NAMES) || defined(__ASSEMBLY__))
> +#include <linux/types.h>
> +#include <asm/imx-common/regs-common.h>
> +
> +/* eLCDIF controller registers */
> +struct mxs_lcdif_regs {
> +	mxs_reg_32(hw_lcdif_ctrl)
> +	mxs_reg_32(hw_lcdif_ctrl1)
> +#if defined(CONFIG_MX28) || defined(CONFIG_MX6SX) || defined(CONFIG_MX6UL)
> +	mxs_reg_32(hw_lcdif_ctrl2)
> +#endif
> +	mxs_reg_32(hw_lcdif_transfer_count)
> +	mxs_reg_32(hw_lcdif_cur_buf)
> +	mxs_reg_32(hw_lcdif_next_buf)
> +#if defined(CONFIG_MX23)
> +	uint32_t	reserved1[4];
> +#endif
> +	mxs_reg_32(hw_lcdif_timing)
> +	mxs_reg_32(hw_lcdif_vdctrl0)
> +	mxs_reg_32(hw_lcdif_vdctrl1)
> +	mxs_reg_32(hw_lcdif_vdctrl2)
> +	mxs_reg_32(hw_lcdif_vdctrl3)
> +	mxs_reg_32(hw_lcdif_vdctrl4)
> +	mxs_reg_32(hw_lcdif_dvictrl0)
> +	mxs_reg_32(hw_lcdif_dvictrl1)
> +	mxs_reg_32(hw_lcdif_dvictrl2)
> +	mxs_reg_32(hw_lcdif_dvictrl3)
> +	mxs_reg_32(hw_lcdif_dvictrl4)
> +	mxs_reg_32(hw_lcdif_csc_coeffctrl0)
> +	mxs_reg_32(hw_lcdif_csc_coeffctrl1)
> +	mxs_reg_32(hw_lcdif_csc_coeffctrl2)
> +	mxs_reg_32(hw_lcdif_csc_coeffctrl3)
> +	mxs_reg_32(hw_lcdif_csc_coeffctrl4)
> +	mxs_reg_32(hw_lcdif_csc_offset)
> +	mxs_reg_32(hw_lcdif_csc_limit)
> +#if defined(CONFIG_MX23)
> +	uint32_t	reserved2[12];
> +#endif
> +	mxs_reg_32(hw_lcdif_data)
> +	mxs_reg_32(hw_lcdif_bm_error_stat)
> +#if defined(CONFIG_MX28) || defined(CONFIG_MX6SX) || defined(CONFIG_MX6UL)
> +	mxs_reg_32(hw_lcdif_crc_stat)
> +#endif
> +	mxs_reg_32(hw_lcdif_lcdif_stat)
> +	mxs_reg_32(hw_lcdif_version)
> +	mxs_reg_32(hw_lcdif_debug0)
> +	mxs_reg_32(hw_lcdif_debug1)
> +	mxs_reg_32(hw_lcdif_debug2)
> +#if defined(CONFIG_MX6SX) || defined(CONFIG_MX6UL)
> +	mxs_reg_32(hw_lcdif_thres)
> +	mxs_reg_32(hw_lcdif_as_ctrl)
> +	mxs_reg_32(hw_lcdif_as_buf)
> +	mxs_reg_32(hw_lcdif_as_next_buf)
> +	mxs_reg_32(hw_lcdif_as_clrkeylow)
> +	mxs_reg_32(hw_lcdif_as_clrkeyhigh)
> +	mxs_reg_32(hw_lcdif_as_sync_delay)
> +	mxs_reg_32(hw_lcdif_as_debug3)
> +	mxs_reg_32(hw_lcdif_as_debug4)
> +	mxs_reg_32(hw_lcdif_as_debug5)
> +#endif
> +};
> +#endif
> +
> +#define LCDIF_CTRL_SFTRST                                       (1 << 31)
> +#define LCDIF_CTRL_CLKGATE                                      (1 << 30)
> +#define LCDIF_CTRL_YCBCR422_INPUT                               (1 << 29)
> +#define LCDIF_CTRL_READ_WRITEB                                  (1 << 28)
> +#define LCDIF_CTRL_WAIT_FOR_VSYNC_EDGE                          (1 << 27)
> +#define LCDIF_CTRL_DATA_SHIFT_DIR                               (1 << 26)
> +#define LCDIF_CTRL_SHIFT_NUM_BITS_MASK                          (0x1f << 21)
> +#define LCDIF_CTRL_SHIFT_NUM_BITS_OFFSET                        21
> +#define LCDIF_CTRL_DVI_MODE                                     (1 << 20)
> +#define LCDIF_CTRL_BYPASS_COUNT                                 (1 << 19)
> +#define LCDIF_CTRL_VSYNC_MODE                                   (1 << 18)
> +#define LCDIF_CTRL_DOTCLK_MODE                                  (1 << 17)
> +#define LCDIF_CTRL_DATA_SELECT                                  (1 << 16)
> +#define LCDIF_CTRL_INPUT_DATA_SWIZZLE_MASK                      (0x3 << 14)
> +#define LCDIF_CTRL_INPUT_DATA_SWIZZLE_OFFSET                    14
> +#define LCDIF_CTRL_CSC_DATA_SWIZZLE_MASK                        (0x3 << 12)
> +#define LCDIF_CTRL_CSC_DATA_SWIZZLE_OFFSET                      12
> +#define LCDIF_CTRL_LCD_DATABUS_WIDTH_MASK                       (0x3 << 10)
> +#define LCDIF_CTRL_LCD_DATABUS_WIDTH_OFFSET                     10
> +#define LCDIF_CTRL_LCD_DATABUS_WIDTH_16BIT                      (0 << 10)
> +#define LCDIF_CTRL_LCD_DATABUS_WIDTH_8BIT                       (1 << 10)
> +#define LCDIF_CTRL_LCD_DATABUS_WIDTH_18BIT                      (2 << 10)
> +#define LCDIF_CTRL_LCD_DATABUS_WIDTH_24BIT                      (3 << 10)
> +#define LCDIF_CTRL_WORD_LENGTH_MASK                             (0x3 << 8)
> +#define LCDIF_CTRL_WORD_LENGTH_OFFSET                           8
> +#define LCDIF_CTRL_WORD_LENGTH_16BIT                            (0 << 8)
> +#define LCDIF_CTRL_WORD_LENGTH_8BIT                             (1 << 8)
> +#define LCDIF_CTRL_WORD_LENGTH_18BIT                            (2 << 8)
> +#define LCDIF_CTRL_WORD_LENGTH_24BIT                            (3 << 8)
> +#define LCDIF_CTRL_RGB_TO_YCBCR422_CSC                          (1 << 7)
> +#define LCDIF_CTRL_LCDIF_MASTER                                 (1 << 5)
> +#define LCDIF_CTRL_DATA_FORMAT_16_BIT                           (1 << 3)
> +#define LCDIF_CTRL_DATA_FORMAT_18_BIT                           (1 << 2)
> +#define LCDIF_CTRL_DATA_FORMAT_24_BIT                           (1 << 1)
> +#define LCDIF_CTRL_RUN						(1 << 0)
> +
> +#define LCDIF_CTRL1_COMBINE_MPU_WR_STRB                         (1 << 27)
> +#define LCDIF_CTRL1_BM_ERROR_IRQ_EN                             (1 << 26)
> +#define LCDIF_CTRL1_BM_ERROR_IRQ                                (1 << 25)
> +#define LCDIF_CTRL1_RECOVER_ON_UNDERFLOW                        (1 << 24)
> +#define LCDIF_CTRL1_INTERLACE_FIELDS                            (1 << 23)
> +#define LCDIF_CTRL1_START_INTERLACE_FROM_SECOND_FIELD           (1 << 22)
> +#define LCDIF_CTRL1_FIFO_CLEAR                                  (1 << 21)
> +#define LCDIF_CTRL1_IRQ_ON_ALTERNATE_FIELDS                     (1 << 20)
> +#define LCDIF_CTRL1_BYTE_PACKING_FORMAT_MASK                    (0xf << 16)
> +#define LCDIF_CTRL1_BYTE_PACKING_FORMAT_OFFSET                  16
> +#define LCDIF_CTRL1_OVERFLOW_IRQ_EN                             (1 << 15)
> +#define LCDIF_CTRL1_UNDERFLOW_IRQ_EN                            (1 << 14)
> +#define LCDIF_CTRL1_CUR_FRAME_DONE_IRQ_EN                       (1 << 13)
> +#define LCDIF_CTRL1_VSYNC_EDGE_IRQ_EN                           (1 << 12)
> +#define LCDIF_CTRL1_OVERFLOW_IRQ                                (1 << 11)
> +#define LCDIF_CTRL1_UNDERFLOW_IRQ                               (1 << 10)
> +#define LCDIF_CTRL1_CUR_FRAME_DONE_IRQ                          (1 << 9)
> +#define LCDIF_CTRL1_VSYNC_EDGE_IRQ                              (1 << 8)
> +#define LCDIF_CTRL1_BUSY_ENABLE                                 (1 << 2)
> +#define LCDIF_CTRL1_MODE86                                      (1 << 1)
> +#define LCDIF_CTRL1_RESET                                       (1 << 0)
> +
> +#define LCDIF_CTRL2_OUTSTANDING_REQS_MASK                       (0x7 << 21)
> +#define LCDIF_CTRL2_OUTSTANDING_REQS_OFFSET                     21
> +#define LCDIF_CTRL2_OUTSTANDING_REQS_REQ_1                      (0x0 << 21)
> +#define LCDIF_CTRL2_OUTSTANDING_REQS_REQ_2                      (0x1 << 21)
> +#define LCDIF_CTRL2_OUTSTANDING_REQS_REQ_4                      (0x2 << 21)
> +#define LCDIF_CTRL2_OUTSTANDING_REQS_REQ_8                      (0x3 << 21)
> +#define LCDIF_CTRL2_OUTSTANDING_REQS_REQ_16                     (0x4 << 21)
> +#define LCDIF_CTRL2_BURST_LEN_8                                 (1 << 20)
> +#define LCDIF_CTRL2_ODD_LINE_PATTERN_MASK                       (0x7 << 16)
> +#define LCDIF_CTRL2_ODD_LINE_PATTERN_OFFSET                     16
> +#define LCDIF_CTRL2_ODD_LINE_PATTERN_RGB                        (0x0 << 16)
> +#define LCDIF_CTRL2_ODD_LINE_PATTERN_RBG                        (0x1 << 16)
> +#define LCDIF_CTRL2_ODD_LINE_PATTERN_GBR                        (0x2 << 16)
> +#define LCDIF_CTRL2_ODD_LINE_PATTERN_GRB                        (0x3 << 16)
> +#define LCDIF_CTRL2_ODD_LINE_PATTERN_BRG                        (0x4 << 16)
> +#define LCDIF_CTRL2_ODD_LINE_PATTERN_BGR                        (0x5 << 16)
> +#define LCDIF_CTRL2_EVEN_LINE_PATTERN_MASK                      (0x7 << 12)
> +#define LCDIF_CTRL2_EVEN_LINE_PATTERN_OFFSET                    12
> +#define LCDIF_CTRL2_EVEN_LINE_PATTERN_RGB                       (0x0 << 12)
> +#define LCDIF_CTRL2_EVEN_LINE_PATTERN_RBG                       (0x1 << 12)
> +#define LCDIF_CTRL2_EVEN_LINE_PATTERN_GBR                       (0x2 << 12)
> +#define LCDIF_CTRL2_EVEN_LINE_PATTERN_GRB                       (0x3 << 12)
> +#define LCDIF_CTRL2_EVEN_LINE_PATTERN_BRG                       (0x4 << 12)
> +#define LCDIF_CTRL2_EVEN_LINE_PATTERN_BGR                       (0x5 << 12)
> +#define LCDIF_CTRL2_READ_PACK_DIR                               (1 << 10)
> +#define LCDIF_CTRL2_READ_MODE_OUTPUT_IN_RGB_FORMAT              (1 << 9)
> +#define LCDIF_CTRL2_READ_MODE_6_BIT_INPUT                       (1 << 8)
> +#define LCDIF_CTRL2_READ_MODE_NUM_PACKED_SUBWORDS_MASK          (0x7 << 4)
> +#define LCDIF_CTRL2_READ_MODE_NUM_PACKED_SUBWORDS_OFFSET        4
> +#define LCDIF_CTRL2_INITIAL_DUMMY_READ_MASK                     (0x7 << 1)
> +#define LCDIF_CTRL2_INITIAL_DUMMY_READ_OFFSET                   1
> +
> +#define LCDIF_TRANSFER_COUNT_V_COUNT_MASK                       (0xffff << 16)
> +#define LCDIF_TRANSFER_COUNT_V_COUNT_OFFSET                     16
> +#define LCDIF_TRANSFER_COUNT_H_COUNT_MASK                       (0xffff << 0)
> +#define LCDIF_TRANSFER_COUNT_H_COUNT_OFFSET                     0
> +
> +#define LCDIF_CUR_BUF_ADDR_MASK                                 0xffffffff
> +#define LCDIF_CUR_BUF_ADDR_OFFSET                               0
> +
> +#define LCDIF_NEXT_BUF_ADDR_MASK                                0xffffffff
> +#define LCDIF_NEXT_BUF_ADDR_OFFSET                              0
> +
> +#define LCDIF_TIMING_CMD_HOLD_MASK                              (0xff << 24)
> +#define LCDIF_TIMING_CMD_HOLD_OFFSET                            24
> +#define LCDIF_TIMING_CMD_SETUP_MASK                             (0xff << 16)
> +#define LCDIF_TIMING_CMD_SETUP_OFFSET                           16
> +#define LCDIF_TIMING_DATA_HOLD_MASK                             (0xff << 8)
> +#define LCDIF_TIMING_DATA_HOLD_OFFSET                           8
> +#define LCDIF_TIMING_DATA_SETUP_MASK                            (0xff << 0)
> +#define LCDIF_TIMING_DATA_SETUP_OFFSET                          0
> +
> +#define LCDIF_VDCTRL0_VSYNC_OEB                                 (1 << 29)
> +#define LCDIF_VDCTRL0_ENABLE_PRESENT                            (1 << 28)
> +#define LCDIF_VDCTRL0_VSYNC_POL                                 (1 << 27)
> +#define LCDIF_VDCTRL0_HSYNC_POL                                 (1 << 26)
> +#define LCDIF_VDCTRL0_DOTCLK_POL                                (1 << 25)
> +#define LCDIF_VDCTRL0_ENABLE_POL                                (1 << 24)
> +#define LCDIF_VDCTRL0_VSYNC_PERIOD_UNIT                         (1 << 21)
> +#define LCDIF_VDCTRL0_VSYNC_PULSE_WIDTH_UNIT                    (1 << 20)
> +#define LCDIF_VDCTRL0_HALF_LINE                                 (1 << 19)
> +#define LCDIF_VDCTRL0_HALF_LINE_MODE                            (1 << 18)
> +#define LCDIF_VDCTRL0_VSYNC_PULSE_WIDTH_MASK                    0x3ffff
> +#define LCDIF_VDCTRL0_VSYNC_PULSE_WIDTH_OFFSET                  0
> +
> +#define LCDIF_VDCTRL1_VSYNC_PERIOD_MASK                         0xffffffff
> +#define LCDIF_VDCTRL1_VSYNC_PERIOD_OFFSET                       0
> +
> +#if defined(CONFIG_MX23)
> +#define LCDIF_VDCTRL2_HSYNC_PULSE_WIDTH_MASK                    (0xff << 24)
> +#define LCDIF_VDCTRL2_HSYNC_PULSE_WIDTH_OFFSET                  24
> +#else
> +#define LCDIF_VDCTRL2_HSYNC_PULSE_WIDTH_MASK                    (0x3fff << 18)
> +#define LCDIF_VDCTRL2_HSYNC_PULSE_WIDTH_OFFSET                  18
> +#endif
> +
> +#define LCDIF_VDCTRL2_HSYNC_PERIOD_MASK                         0x3ffff
> +#define LCDIF_VDCTRL2_HSYNC_PERIOD_OFFSET                       0
> +
> +#define LCDIF_VDCTRL3_MUX_SYNC_SIGNALS                          (1 << 29)
> +#define LCDIF_VDCTRL3_VSYNC_ONLY                                (1 << 28)
> +#define LCDIF_VDCTRL3_HORIZONTAL_WAIT_CNT_MASK                  (0xfff << 16)
> +#define LCDIF_VDCTRL3_HORIZONTAL_WAIT_CNT_OFFSET                16
> +#define LCDIF_VDCTRL3_VERTICAL_WAIT_CNT_MASK                    (0xffff << 0)
> +#define LCDIF_VDCTRL3_VERTICAL_WAIT_CNT_OFFSET                  0
> +
> +#define LCDIF_VDCTRL4_DOTCLK_DLY_SEL_MASK                       (0x7 << 29)
> +#define LCDIF_VDCTRL4_DOTCLK_DLY_SEL_OFFSET                     29
> +#define LCDIF_VDCTRL4_SYNC_SIGNALS_ON                           (1 << 18)
> +#define LCDIF_VDCTRL4_DOTCLK_H_VALID_DATA_CNT_MASK              0x3ffff
> +#define LCDIF_VDCTRL4_DOTCLK_H_VALID_DATA_CNT_OFFSET            0
> +
> +#endif /* __IMX_REGS_LCDIF_H__ */
> 

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
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
=====================================================================

  reply	other threads:[~2015-10-20 13:15 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-20 11:39 [U-Boot] [PATCH V2 00/14] imx: mx6/7: support lcdif Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 01/14] mxs: add parameter base_addr for mxs_set_lcdclk Peng Fan
2015-10-20 13:05   ` Stefano Babic
2015-10-26  2:57     ` Peng Fan
2015-10-26 15:52       ` Stefano Babic
2015-10-27  5:28         ` Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 02/14] sandisk: sfp: correct function name Peng Fan
2015-10-20 13:09   ` Stefano Babic
2015-10-20 11:39 ` [U-Boot] [PATCH V2 03/14] xfi3: " Peng Fan
2015-10-20 13:10   ` Stefano Babic
2015-10-26  3:00     ` Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 04/14] imx: imx-common: move lcdif structure and macro definition to imx-common Peng Fan
2015-10-20 13:15   ` Stefano Babic [this message]
2015-10-26  3:06     ` Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 05/14] imx: mx6: fix register address Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 06/14] imx: mx6: crm_reg: add LCDIF related macros Peng Fan
2015-10-20 13:19   ` Stefano Babic
2015-10-20 11:39 ` [U-Boot] [PATCH V2 07/14] imx: mx6: add clock api for lcdif Peng Fan
2015-10-20 13:39   ` Stefano Babic
2015-10-26  3:13     ` Peng Fan
2015-10-27  7:36     ` Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 08/14] imx: mx6ul_14x14_evk: support lcdif display Peng Fan
2015-10-20 13:41   ` Stefano Babic
2015-10-26  3:18     ` Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 09/14] video: mxsfb: introduce lcdif_power_down Peng Fan
2015-10-20 13:59   ` Stefano Babic
2015-10-26  3:24     ` Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 10/14] imx: mx6: implement reset_misc Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 11/14] imx: imx-common: power down lcdif before boot os Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 12/14] imx: mx7: compile misc.c for mx7 Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 13/14] imx: mx7 use the common lcdif register structure Peng Fan
2015-10-20 11:39 ` [U-Boot] [PATCH V2 14/14] imx: mx7dsabresd: support lcdif Peng Fan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56263E67.2050801@denx.de \
    --to=sbabic@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.