On 07.09.2020 17:24, Kristof Havasi wrote: > Dear Ciubatariu, > > as I am not familiar with the linux development workflow, I am > contacting you directly as the author of the upstream patch: > af199a1a9cb02ec0194804bd46c174b6db262075 > > I noticed that your addition there was applied twice into 5.4 [1] > > d9b8206e5323ae3c9b5b4177478a1224108642f7 v5.4.51-45-gd9b8206e5323 > d55dad8b1d893fae0c4e778abf2ace048bcbad86 v5.4.52-13-gd55dad8b1d89 > > resulting in a non-harmful, but unnecessary double setting of the variable. > > /* set the real number of ports */ > dev->ds->num_ports = dev->port_cnt; > > /* set the real number of ports */ > dev->ds->num_ports = dev->port_cnt; > > return 0; > > Could you notify the stable maintainers to apply your patch correctly? > > Best regards, > Kristóf Havasi > > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/drivers/net/dsa/microchip/ksz8795.c?h=v5.4.63#n1274 > Hello, Kristóf discovered that one patch of mine was applied twice. What is the best way to address this? Thank you Kristóf for finding this. Best regards, Codrin
