Re: [PATCH 3/3] I2C: mediatek: Add driver for MediaTek I2C controller

Wolfram Sang
2014-10-30T13:25:05Z

[-- Attachment #1: Type: text/plain, Size: 382 bytes --]

> > +obj-$(CONFIG_I2C_MT65XX) += i2c-mt65xx.o
>
> As it works for mt65xx and mt81xx series, better rename it to i2c-mtk.c

That is a bit too generic IMO. It is likely that we will see some
totally different I2C core from MTK in the future. Plus, it is not
uncommon that a driver initially for SoC-X supports SoC-Y, too.

Thanks for pointing out the other remarks.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]