Hi Gregory, > + > + /* update unicast mapping */ > + mvneta_set_rx_mode(pp->dev); I know it may be an ultimate level of nitpicking, but can you start a comment with capital letter?:) Best regards, Marcin