On Fri, Apr 18, 2014 at 05:41:01PM +0900, Masahiro Yamada wrote: > - Do not use a deep indentation. We have only 80-character > on each line and 1 indentation consumes 8 spaces. Before the > code moves far to the right, you should consider to > fix your code. See Linux Documentation/CodingStyle. > > - Add CONFIG_OF_STDOUT_VIA_ALIAS and OF_STDOUT_PATH macros > only to their definition. Do not add them to both > callee and caller. This is a tip to avoid using #ifdef > everywhere. > > - OF_STDOUT_PATH and CONFIG_OF_STDOUT_VIA_ALIAS are exclusive. > If both are defined, the former takes precedence. > Do not try to fix-up "linux,stdout-path" property twice. > > Signed-off-by: Masahiro Yamada <yamad...@jp.panasonic.com> > Acked-by: Simon Glass <s...@chromium.org>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot