| Message ID | 20230419111443.231151-3-raghuhack78@gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Fixing check patch styling issues | expand |
Raghu H wrote: > Styling errors due to linux of unwanted spaces in the definition, > modified the definition to use tab to fix the styling issue. > > Signed-off-by: Raghu H <raghuhack78@gmail.com> NAK This is unnecessary churn. The formatting of the file is fine and very readable. Ira > --- > drivers/cxl/core/port.c | 18 +++++++++--------- > 1 file changed, 9 insertions(+), 9 deletions(-) > > diff --git a/drivers/cxl/core/port.c b/drivers/cxl/core/port.c > index 8ee6b6e2e2a4..7c3aaed180ca 100644 > --- a/drivers/cxl/core/port.c > +++ b/drivers/cxl/core/port.c > @@ -95,15 +95,15 @@ static ssize_t size_show(struct device *dev, struct device_attribute *attr, > } > static DEVICE_ATTR_RO(size); > > -#define CXL_DECODER_FLAG_ATTR(name, flag) \ > -static ssize_t name##_show(struct device *dev, \ > - struct device_attribute *attr, char *buf) \ > -{ \ > - struct cxl_decoder *cxld = to_cxl_decoder(dev); \ > - \ > - return sysfs_emit(buf, "%s\n", \ > - (cxld->flags & (flag)) ? "1" : "0"); \ > -} \ > +#define CXL_DECODER_FLAG_ATTR(name, flag) \ > +static ssize_t name##_show(struct device *dev, \ > + struct device_attribute *attr, char *buf) \ > +{ \ > + struct cxl_decoder *cxld = to_cxl_decoder(dev); \ > + \ > + return sysfs_emit(buf, "%s\n", \ > + (cxld->flags & (flag)) ? "1" : "0"); \ > +} \ > static DEVICE_ATTR_RO(name) > > CXL_DECODER_FLAG_ATTR(cap_pmem, CXL_DECODER_F_PMEM); > -- > 2.39.2 >
diff --git a/drivers/cxl/core/port.c b/drivers/cxl/core/port.c index 8ee6b6e2e2a4..7c3aaed180ca 100644 --- a/drivers/cxl/core/port.c +++ b/drivers/cxl/core/port.c @@ -95,15 +95,15 @@ static ssize_t size_show(struct device *dev, struct device_attribute *attr, } static DEVICE_ATTR_RO(size); -#define CXL_DECODER_FLAG_ATTR(name, flag) \ -static ssize_t name##_show(struct device *dev, \ - struct device_attribute *attr, char *buf) \ -{ \ - struct cxl_decoder *cxld = to_cxl_decoder(dev); \ - \ - return sysfs_emit(buf, "%s\n", \ - (cxld->flags & (flag)) ? "1" : "0"); \ -} \ +#define CXL_DECODER_FLAG_ATTR(name, flag) \ +static ssize_t name##_show(struct device *dev, \ + struct device_attribute *attr, char *buf) \ +{ \ + struct cxl_decoder *cxld = to_cxl_decoder(dev); \ + \ + return sysfs_emit(buf, "%s\n", \ + (cxld->flags & (flag)) ? "1" : "0"); \ +} \ static DEVICE_ATTR_RO(name) CXL_DECODER_FLAG_ATTR(cap_pmem, CXL_DECODER_F_PMEM);
Styling errors due to linux of unwanted spaces in the definition, modified the definition to use tab to fix the styling issue. Signed-off-by: Raghu H <raghuhack78@gmail.com> --- drivers/cxl/core/port.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-)