On Wed, 18 Dec 2024 16:37:44 +0200 Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx> wrote: > Loop variable i counting from 0 upwards does not need to be signed so > make it unsigned int. > > Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx> Whilst I find it hard to care, it is harmless so up to Bjorn if he wants the churn or not. Reviewed-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > --- > drivers/pci/pcie/tlp.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/pci/pcie/tlp.c b/drivers/pci/pcie/tlp.c > index 2bf15749cd31..65ac7b5d8a87 100644 > --- a/drivers/pci/pcie/tlp.c > +++ b/drivers/pci/pcie/tlp.c > @@ -24,7 +24,8 @@ > int pcie_read_tlp_log(struct pci_dev *dev, int where, > struct pcie_tlp_log *log) > { > - int i, ret; > + unsigned int i; > + int ret; > > memset(log, 0, sizeof(*log)); >