On Mon, Apr 17, 2023 at 11:04:04AM +0300, Dan Carpenter wrote: > On Sat, Apr 15, 2023 at 04:10:44PM +0200, Luke Koch wrote: > > + for (int i = 0; i < 8; i++) { > > + if (count > i && > > + DOT11_RATE5_ISBASIC_GET(item->supprates[i])) { > > + req->basicrate[i] .data = item->supprates[i]; > ^ > Could you remove these spaces before the .data and .status? Otherwise > it looks good to me. Sure, the spaces must have been leftovers from the token pasting and I unfortunately overlooked them. Thanks for reviewing! Best regards, Luke Koch