| Message ID | 20190902123216.9809-3-rrichter@marvell.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | EDAC: Small cleanups and fixes | expand |
On Mon, Sep 02, 2019 at 12:33:43PM +0000, Robert Richter wrote: > Struct member dev_ch_attribute->channel is always used as unsigned > int. Change type to unsigned int to avoid type casts. > > Signed-off-by: Robert Richter <rrichter@marvell.com> > --- > drivers/edac/edac_mc_sysfs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/edac/edac_mc_sysfs.c b/drivers/edac/edac_mc_sysfs.c > index 640b9419623e..4eb8c5ceb973 100644 > --- a/drivers/edac/edac_mc_sysfs.c > +++ b/drivers/edac/edac_mc_sysfs.c > @@ -131,7 +131,7 @@ static const char * const edac_caps[] = { > > struct dev_ch_attribute { > struct device_attribute attr; > - int channel; > + unsigned int channel; > }; > > #define DEVICE_CHANNEL(_name, _mode, _show, _store, _var) \ > -- Merged this one into 1/5.
diff --git a/drivers/edac/edac_mc_sysfs.c b/drivers/edac/edac_mc_sysfs.c index 640b9419623e..4eb8c5ceb973 100644 --- a/drivers/edac/edac_mc_sysfs.c +++ b/drivers/edac/edac_mc_sysfs.c @@ -131,7 +131,7 @@ static const char * const edac_caps[] = { struct dev_ch_attribute { struct device_attribute attr; - int channel; + unsigned int channel; }; #define DEVICE_CHANNEL(_name, _mode, _show, _store, _var) \
Struct member dev_ch_attribute->channel is always used as unsigned int. Change type to unsigned int to avoid type casts. Signed-off-by: Robert Richter <rrichter@marvell.com> --- drivers/edac/edac_mc_sysfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)