[PATCHv3] mx31: add support for the bugbase 1.3 from buglabs
Uwe Kleine-König
u.kleine-koenig at pengutronix.de
Sun Feb 13 16:18:55 EST 2011
Hi Denis,
On Sat, Feb 12, 2011 at 10:16:59PM +0100, Denis 'GNUtoo' Carikli wrote:
> This work was based on bug-linux-2.6.30.patch that can be found
> in buglabs's svn here:
> svn://bugcamp.net/bug/branches/izzy/experimental
>
> Note that the hardware schematics and documentations can be obtained
> here: http://www.bugcommunity.com/wiki/index.php/BUGbase
>
> Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo at no-log.org>
The patch looks good to me now. If you want to make me still more
happy, there are still two minor things to optimize:
- don't reference the history of the patch in the commit log. This is
quite useless once the commit made it into Linus' tree. (The info
might still be interesting, but then put it after the tripple dash
below. Then it won't make it into git.)
- there is a trailing empty line in bug_board_init()
Maybe Sascha can fix this up when taking the patch? (Assuming the
others are happy now, too of course.)
And I wonder there is no copyright statement by you in
arch/arm/mach-mx3/mach-bug.c, maybe because you consider your changes
too minor to be relevant? (INAL and I don't know who to value your
contribution.)
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
More information about the linux-arm-kernel
mailing list