On 16/11/2016 22:57, york sun wrote:
On 11/15/2016 02:20 PM, Alexander Graf wrote:
On 15/11/2016 23:11, york sun wrote:
On 11/15/2016 12:36 PM, Alexander Graf wrote:
Hi York,
Do you want to take that patch set?
Yes, I can. I saw it assigned to you in patchwork so I didn't act.
I think since it's really hardware enablement for your platform, I'd
prefer if you pull them all in.
Alex,
I got compiling warnings
Btw, speaking of warnings, you might want to compile the code base with
gcc6 :)
drivers/ddr/fsl/options.c:80:33: warning: 'single_S' defined but not
used [-Wunused-const-variable=]
static const struct dynamic_odt single_S[4] = {
^~~~~~~~
drivers/ddr/fsl/options.c:63:33: warning: 'single_D' defined but not
used [-Wunused-const-variable=]
static const struct dynamic_odt single_D[4] = {
^~~~~~~~
drivers/ddr/fsl/options.c:36:33: warning: 'single_Q' defined but not
used [-Wunused-const-variable=]
static const struct dynamic_odt single_Q[4] = {
^~~~~~~~
drivers/net/e1000.c: In function 'e1000_initialize_hardware_bits':
drivers/net/e1000.c:1526:2: warning: this 'if' clause does not guard...
[-Wmisleading-indentation]
if (hw->mac_type == e1000_igb)
^~
drivers/net/e1000.c:1529:3: note: ...this statement, but the latter is
misleadingly indented as if it is guarded by the 'if'
switch (hw->mac_type) {
^~~~~~
drivers/net/ldpaa_eth/ldpaa_eth.c: In function 'ldpaa_eth_open':
drivers/net/ldpaa_eth/ldpaa_eth.c:423:2: warning: this 'if' clause does
not guard... [-Wmisleading-indentation]
if (priv->phydev)
^~
drivers/net/ldpaa_eth/ldpaa_eth.c:425:3: note: ...this statement, but
the latter is misleadingly indented as if it is guarded by the 'if'
if (err) {
^~
I'm seeing the warnings too now with the nand target. I'll fix them up
and send v7.
Alex
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
