hello there,

[linux-4.5-rc4/arch/powerpc/boot/treeboot-akebono.c:90]: (style) A pointer can 
not be negative so it is either pointless or an error to check if it is not.

Source code is

    emac = finddevice("/plb/opb/ethernet");
    if (emac> 0) {

but

    void *emac;

Suggest new code

    emac = finddevice("/plb/opb/ethernet");
    if (emac != 0) {


Regards

David Binderman
                                          
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to