26 Jan
2011
26 Jan
'11
9:06 a.m.
Dear Lei Wen,
In message AANLkTinN07gFCXTGqBVSAyQM_BDXJ6xq0+OzPmh+_DAb@mail.gmail.com you wrote:
Please fix. Also, please explain what MFP is. It may be obvious to you, it ain't so for me.
MFP is stand for multiply function pin setting, which servers for setting a pin of cpu to server for different functions on different design boards to better suit for its design.
multiply or multiple?
Maybe you can write something like "Add support for multiple function pin (MFP) for Marvell PANTHEON SoCs" in your commit message.
Otherwise people might wonder what Multi Function Peripherals or Multi Function Printers have to do with that.
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@denx.de
If I don't document something, it's usually either for a good reason,
or a bad reason. In this case it's a good reason. :-)
- Larry Wall in 1992Jan17.005405.16806@netlabs.com