[U-Boot] [PATCH 1/9] Tegra: T30: Add include files

Tom Warren twarren.nvidia at gmail.com
Thu Sep 13 23:10:12 CEST 2012


Tom,

On Thu, Sep 13, 2012 at 11:06 AM, Tom Rini <trini at ti.com> wrote:
> On Wed, Sep 12, 2012 at 03:10:47PM -0700, Tom Warren wrote:
>
>> Signed-off-by: Tom Warren <twarren at nvidia.com>
>
> A few things:
> - I see some #define FOO[space][space]val that should be [tab]

Probably copied over from Tegra20 files. I'll turn on whitespace
highlighting in my editor and fix 'em up.

> - I didn't checkpatch.pl this (nor the whole series) but please do and
>   let us know if it's clean or why the warnings are false positives.

I always run checkpath before submitting. I'll put a notice to that
affect in the next version. Checkpatch ran clean w/only 1
false-positive about 'macros with complex values should be enclosed in
parenthesis' for the "#define CONFIG_DEFAULT_DEVICE_TREE
tegra30-cardhu" line in cardhu.h.

> - My preference is to bring in includes and C files and Makefiles and so
>   on all at once, when each is needed / useful.  YMMV and not a big
>   deal.
> - But please make sure that you aren't adding defines / structs / etc
>   that aren't used at some point by the end of the series at least.
>   Removing (and correcting!) structs and defines was one of the things I
>   had to do on am33xx.  If it wasn't added until the corresponding
>   driver work was being pushed, it'd have saved me some time.
>

Sure, and that's good advice. I took a couple of passes during the
port to try and remove vestigial and/or useless/unsupported files,
features, and code, but I'm sure I missed some (as Stephen has already
pointed out). I'll address those in V2.

Thanks,

Tom
> --
> Tom


More information about the U-Boot mailing list