On Thu, 2019-02-14 at 10:52 -0800, James Smart wrote: > > On 2/13/2019 5:51 PM, YueHaibing wrote: > > Fixes gcc '-Wunused-but-set-variable' warning: > > > > drivers/scsi/lpfc/lpfc_init.c: In function > > 'lpfc_cpu_affinity_check': > > drivers/scsi/lpfc/lpfc_init.c:10599:19: warning: > > variable 'phys_id' set but not used [-Wunused-but-set-variable] > > > > It never used since introduction in commit 6a828b0f6192 ("scsi: > > lpfc: Support > > non-uniform allocation of MSIX vectors to hardware queues") > > > > Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx> > > --- > > drivers/scsi/lpfc/lpfc_init.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > > > Looks fine. Thanks > > Signed-off-by: James Smart <james.smart@xxxxxxxxxxxx> Under the DCO this can't be a Signed-off-by tag: signoffs track the patch transmission path under the DCO, so unless you send it you can't add your signoff. If you just want Martin to apply it now, and you don't want to gather and resend it with your other lpfc patches, I think the tag you want is Acked-by. James