[U-Boot] [PATCH 0/5] add i2c support to pantheon and aramada100

Wolfgang Denk wd at denx.de
Tue Mar 15 09:12:28 CET 2011


Dear Lei Wen,

In message <1300160443-12552-1-git-send-email-leiwen at marvell.com> you wrote:
> V2:
> rename the previous pxa_i2c to mvi2c, since this driver would be shared
> by many other Marvell platforms.
> 
> Lei Wen (5):
>   pxa: move i2c driver to the common place
>   mvi2c: use structure to replace the direclty define
>   I2C: add i2c support for Pantheon platform
>   I2C: mvi2c: add multi bus support
>   I2C: add i2c support for Armada100 platform

Checkpatch says:

[U-Boot] [PATCH V2 1/5] pxa: move i2c driver to the common place
total: 57 errors, 7 warnings, 497 lines checked
[U-Boot] [PATCH v2 2/5] mvi2c: use structure to replace the
total: 2 errors, 1 warnings, 333 lines checked
[U-Boot] [PATCH v2 3/5] I2C: add i2c support for Pantheon platform
total: 2 errors, 1 warnings, 64 lines checked
[U-Boot] [PATCH v2 4/5] I2C: mvi2c: add multi bus support
total: 2 errors, 0 warnings, 51 lines checked
[U-Boot] [PATCH v2 5/5] I2C: add i2c support for Armada100 platform
total: 2 errors, 1 warnings, 65 lines checked

Please cleanup!

and next time please run checkpatch _before_ submitting!

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
There is, however, a strange, musty smell in the air that reminds  me
of something...hmm...yes...I've got it...there's a VMS nearby, or I'm
a Blit.          - Larry Wall in Configure from the perl distribution


More information about the U-Boot mailing list