From mboxrd@z Thu Jan  1 00:00:00 1970
From: Masahiro Yamada <yamada.m@jp.panasonic.com>
Date: Tue, 02 Dec 2014 13:35:51 +0900
Subject: [U-Boot] [PATCH v3 01/10] dm: i2c: Add a uclass for I2C
In-Reply-To: <CAPnjgZ3Uq1Xy+jKoj7_Fk=5fbNK0dL9kH0WUV1uHPxV2uf1FWQ@mail.gmail.com>
References: <20141201204700.67C8.AA925319@jp.panasonic.com>
	<CAPnjgZ3Uq1Xy+jKoj7_Fk=5fbNK0dL9kH0WUV1uHPxV2uf1FWQ@mail.gmail.com>
Message-ID: <20141202133550.67DF.AA925319@jp.panasonic.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

Hi Simon,





On Mon, 1 Dec 2014 21:31:29 -0700
Simon Glass <sjg@chromium.org> wrote:

> +Heiko - are you OK with the new msg-based approach?

This approach looks much better to me.




> >> +     .name           = "i2c_generic_drv",
> >> +     .id             = UCLASS_I2C_GENERIC,
> >> +};
> >
> >
> > Can we move "i2c_generic" to a different file?
> > maybe, drivers/i2c/i2c-generic.c or drivers/i2c/i2c-generic-chip.c ?
> >
> > UCLASS_DRIVER(i2c) is a bus, whereas UCLASS_DRIVER(i2c_generic) is a chip.
> >
> > Mixing up a bus and a chip-device together in the same file
> > looks confusing to me.


I take back this comment.

I thought it again, and
"i2c_generic" is short enough to be included in i2c-uclass.c.
I am fine with it as is.


Best Regards
Masahiro Yamada