> > +u64 read_mac_stats(struct eth_com *ecom, unsigned int addr)
> > +{
> > +   u32 data_l, data_h;
> > +
> > +   if (eth_com_read_reg(ecom, addr, &data_l) ||
> > +       eth_com_read_reg(ecom, addr + 1, &data_h))
> > +           return 0xffffffffffffffffULL;

> return -1; ?

Since this is a u64 function, i expect you get a compiler
warning. Maybe only with W=1. It is better to use U64_MAX.

         Andrew

Reply via email to