From mboxrd@z Thu Jan  1 00:00:00 1970
From: York Sun <yorksun@freescale.com>
Date: Thu, 14 Aug 2014 10:04:41 -0700
Subject: [U-Boot] [PATCH v4 02/18] ls102xa: i2c: Add i2c support for
	LS102xA
In-Reply-To: <1407998707-9632-3-git-send-email-b18965@freescale.com>
References: <1407998707-9632-1-git-send-email-b18965@freescale.com>
	<1407998707-9632-3-git-send-email-b18965@freescale.com>
Message-ID: <53ECEC29.6010207@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:44 PM, Alison Wang wrote:
> This patch is to add I2C 1,2,3 support for LS102xA.
> 

I think the commit message should say which existing I2C driver is compatible
with the controller of LS201x. From the change, you are using the same driver
for i.MX. But it is not clear if you are using the 8-bit driver or the 32-bit
driver. I can tell from your patch 01/18 that you enabled I2C_QUIRK_REG to use
8-bit driver. This should go into the commit message to help other follow the
suit if another SoC is added the same way.

York