From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Mon, 20 Oct 2014 16:46:52 +0200
Subject: [U-Boot] [PATCH] arm: socfpga: Add I2C support to SoCFPGA
In-Reply-To: <1413787548-25438-1-git-send-email-sr@denx.de>
References: <1413787548-25438-1-git-send-email-sr@denx.de>
Message-ID: <201410201646.52990.marex@denx.de>
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 Monday, October 20, 2014 at 08:45:48 AM, Stefan Roese wrote:

Hi all,

> This patch adds I2C support for the SoCFPGA. Using the designware I2C
> controller driver. It supports all 4 I2C busses on the SoCFPGA.
> 
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Vince Briders <vbridger@altera.com>
> Cc: Chin Liang See <clsee@altera.com>
> Cc: Dinh Nguyen <dinguyen@altera.com>

+CC Heiko, the I2C custodian.

> ---
>  include/configs/socfpga_common.h | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
> 
> diff --git a/include/configs/socfpga_common.h
> b/include/configs/socfpga_common.h index 49504dc..ba1e072 100644
> --- a/include/configs/socfpga_common.h
> +++ b/include/configs/socfpga_common.h
> @@ -157,6 +157,28 @@
>  #define CONFIG_BAUDRATE			115200
> 
>  /*
> + * I2C support
> + */
> +#define CONFIG_HARD_I2C
> +#define CONFIG_DW_I2C
> +#define CONFIG_I2C_MULTI_BUS

I presume this should be converted to CONFIG_SYS_I2C framework instead of using 
this I2C multibus horror, no?

> +#define CONFIG_SYS_I2C_BUS_MAX		4
> +#define CONFIG_SYS_I2C_BASE		SOCFPGA_I2C0_ADDRESS
> +#define CONFIG_SYS_I2C_BASE1		SOCFPGA_I2C1_ADDRESS
> +#define CONFIG_SYS_I2C_BASE2		SOCFPGA_I2C2_ADDRESS
> +#define CONFIG_SYS_I2C_BASE3		SOCFPGA_I2C3_ADDRESS
> +/* using standard mode which the speed up to 100Kb/s */
> +#define CONFIG_SYS_I2C_SPEED		100000
> +/* address of device when used as slave */
> +#define CONFIG_SYS_I2C_SLAVE		0x02
> +#ifndef __ASSEMBLY__
> +/* clock supplied to I2C controller in unit of MHz */
> +unsigned int cm_get_l4_sp_clk_hz(void);

Include clock_manager.h instead maybe ?

> +#define IC_CLK				(cm_get_l4_sp_clk_hz() / 1000000)
> +#endif
> +#define CONFIG_CMD_I2C
> +
> +/*
>   * U-Boot environment
>   */
>  #define CONFIG_SYS_CONSOLE_IS_IN_ENV

Best regards,
Marek Vasut