Hi Wolfgang,
On Wed, Jan 26, 2011 at 4:06 PM, Wolfgang Denk wd@denx.de wrote:
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?
Sorry for my poor English. It is 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.
Agree. Fix it in next patch set...
Best regards, Lei