Hi Jim, Sorry I see nobody reviewed your patch. Jim Harris <james.r.harris at intel.com> wrote: > + memcpy(&dev2->id, &dev->id, sizeof(dev2->id)); [...] > + memcmp(&dev2->id, &dev->id, sizeof(dev2->id))) Why using memcpy and memcmp instead of simple assignment and comparison?