Messages in this thread | | | Date | Wed, 17 Mar 2021 12:33:37 +0100 | From | Borislav Petkov <> | Subject | Re: [PATCH] EDAC: use NULL instead of using plain integer as pointer |
| |
On Wed, Mar 17, 2021 at 03:47:25PM +0800, Yang Li wrote: > This fixes the following sparse warnings: > drivers/edac/sb_edac.c:433:10: warning: Using plain integer as NULL > pointer > drivers/edac/sb_edac.c:499:10: warning: Using plain integer as NULL > pointer > drivers/edac/sb_edac.c:570:10: warning: Using plain integer as NULL > pointer > drivers/edac/sb_edac.c:614:10: warning: Using plain integer as NULL > pointer > drivers/edac/sb_edac.c:680:10: warning: Using plain integer as NULL > pointer > > Reported-by: Abaci Robot <abaci@linux.alibaba.com> > Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> > --- > drivers/edac/sb_edac.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/edac/sb_edac.c b/drivers/edac/sb_edac.c > index 93daa42..a99df55 100644 > --- a/drivers/edac/sb_edac.c > +++ b/drivers/edac/sb_edac.c > @@ -433,7 +433,7 @@ struct sbridge_pvt { > > static const struct pci_id_table pci_dev_descr_sbridge_table[] = { > PCI_ID_TABLE_ENTRY(pci_dev_descr_sbridge, ARRAY_SIZE(pci_dev_descr_sbridge), 1, SANDY_BRIDGE), > - {0,} /* 0 terminated list. */ > + {NULL,} /* 0 terminated list. */
Why does this matter for list terminators?
-- Regards/Gruss, Boris.
https://people.kernel.org/tglx/notes-about-netiquette
| |