> @@ -3100,16 +3100,19 @@ char *mv88e6xxx_drv_probe(struct device *dsa_dev, > struct device *host_dev, > prod_num = (id & 0xfff0) >> 4; > rev = id & 0x000f; > > - name = mv88e6xxx_lookup_name(id, table, num); > - if (!name) > + info = mv88e6xxx_lookup_info(prod_num, table, num); > + if (!info) > return NULL; > > + name = (char *) info->name; > +
Hi Vivien Casting off the const is not so nice. How about adding in another patch to change the API to return a const char *. Andrew