On 05/07/18 11:33, David Miller wrote: > From: Colin King <colin.king@xxxxxxxxxxxxx> > Date: Thu, 5 Jul 2018 10:37:32 +0100 > >> From: Colin Ian King <colin.king@xxxxxxxxxxxxx> >> >> Variables adv and lpa are being assigned but are never used hence they >> are redundant and can be removed. >> >> Cleans up clang warnings: >> warning: variable 'lpa' set but not used [-Wunused-but-set-variable] >> warning: variable 'adv' set but not used [-Wunused-but-set-variable] >> >> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > I think you can safely remove the register reads too, so please > do so. Just to clarify, just the MII_ADVERTISE and MII_LPA, or both these AND also MII_ESTATUS too? > > Thanks. > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
