| Message ID | 1615967245-35819-1-git-send-email-yang.lee@linux.alibaba.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | EDAC: use NULL instead of using plain integer as pointer | expand |
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?
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. */ }; /* This changes depending if 1HA or 2HA: @@ -499,7 +499,7 @@ struct sbridge_pvt { static const struct pci_id_table pci_dev_descr_ibridge_table[] = { PCI_ID_TABLE_ENTRY(pci_dev_descr_ibridge, 12, 2, IVY_BRIDGE), - {0,} /* 0 terminated list. */ + {NULL,} /* 0 terminated list. */ }; /* Haswell support */ @@ -570,7 +570,7 @@ struct sbridge_pvt { static const struct pci_id_table pci_dev_descr_haswell_table[] = { PCI_ID_TABLE_ENTRY(pci_dev_descr_haswell, 13, 2, HASWELL), - {0,} /* 0 terminated list. */ + {NULL,} /* 0 terminated list. */ }; /* Knight's Landing Support */ @@ -614,7 +614,7 @@ struct sbridge_pvt { static const struct pci_id_table pci_dev_descr_knl_table[] = { PCI_ID_TABLE_ENTRY(pci_dev_descr_knl, ARRAY_SIZE(pci_dev_descr_knl), 1, KNIGHTS_LANDING), - {0,} + {NULL,} }; /* @@ -680,7 +680,7 @@ struct sbridge_pvt { static const struct pci_id_table pci_dev_descr_broadwell_table[] = { PCI_ID_TABLE_ENTRY(pci_dev_descr_broadwell, 10, 2, BROADWELL), - {0,} /* 0 terminated list. */ + {NULL,} /* 0 terminated list. */ };
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(-)