On Thu, Dec 30, 2021 at 03:39:00PM +0000, Colin Ian King wrote: > The pointer link is being re-assigned the same value that it was > initialized with in the previous declaration statement. The > re-assignment is redundant and can be removed. > > Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx> It would be better to add Fixes statement in the end of message. Fixes: 387707fdf486 ("net/smc: convert static link ID to dynamic references") And [PATCH net ...] is fine to point out the target. The patch itself looks good to me, thanks. Reviewed-by: Tony Lu <tonylu@xxxxxxxxxxxxxxxxx> Thanks, Tony Lu