Patch set 6:Code-Review +1
7 comments:
File src/drivers/aspeed/common/ast_i2c.c:
Patch Set #6, Line 30: * Authors: Dave Airlie <airlied@redhat.com>
why not just put these two lines below the copyright above?
*/
#include
nit: newline before include
File src/drivers/aspeed/common/ast_mode.c:
/*
* Authors: Dave Airlie <airlied@redhat.com>
* Authors: 9Elements Agency GmbH <patrick.rudolph@9elements.com>
move up below copyright?
*/
#include
newline after comment
why two newlines?
File src/drivers/aspeed/common/ast_mode_corebootfb.c:
*/
/*
* Authors: Dave Airlie <airlied@redhat.com>
* Authors: 9Elements Agency GmbH <patrick.rudolph@9elements.com>
move below Copyright?
newline
To view, visit change 35726. To unsubscribe, or for help writing mail filters, visit settings.