Re: RFC PATCH patch-13-mm1-hwmon-adm9240-whitespace-1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Grant,

> Patches are planned to make chassis_clear write data =
> <dont_care>,

With what benefit?

> and rework sysfs interface to reduce driver size.

Are you referring to Yani Ioannou's dynamic sysfs callbacks? That would
be very welcome, thanks for looking into that.

> This patch removes mixed tab/space indenting,

There is nothing wrong with mixed tab/space indenting. Additionally,
this part of the code will be completely modified again if you convert
the driver to use dynamic callbacks. So I would like you not to include
these changes in this cleanup patch.

> breaks long lines,

Good.

> removed a trailing space

Good too.

Thanks,
-- 
Jean Delvare




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux