parport: Switch to some modern APIs

[resend, PATCH v1 0/3] parport: Switch to some modern APIs

Andy Shevchenko posted 3 patches 2 years, 2 months ago

Download series mbox

drivers/parport/procfs.c | 53 +++++++---------------------------------
drivers/parport/share.c  | 25 ++++++++-----------
include/linux/parport.h  |  2 --
3 files changed, 19 insertions(+), 61 deletions(-)

Expand all Fold all

[resend, PATCH v1 0/3] parport: Switch to some modern APIs

Posted by Andy Shevchenko 2 years, 2 months ago

This is a set of ad-hoc refactorings and cleanups to the parport driver.

In v1,resent:
- resent with Greg KH in Cc list

Initial contribution have been seen here 6 weeks ago:
https://lore.kernel.org/r/20230901134250.1172990-1-andriy.shevchenko@linux.intel.com

Andy Shevchenko (3):
  parport: Use kasprintf() instead of fixed buffer formatting
  parport: Use list_for_each() helper
  parport: Drop unneeded NULL or 0 assignments

 drivers/parport/procfs.c | 53 +++++++---------------------------------
 drivers/parport/share.c  | 25 ++++++++-----------
 include/linux/parport.h  |  2 --
 3 files changed, 19 insertions(+), 61 deletions(-)

-- 
2.40.0.1.gaa8946217a0b

Re: [resend, PATCH v1 0/3] parport: Switch to some modern APIs

Posted by Luis Chamberlain 2 years, 2 months ago

On Mon, Oct 16, 2023 at 04:31:32PM +0300, Andy Shevchenko wrote:
> This is a set of ad-hoc refactorings and cleanups to the parport driver.
> 
> In v1,resent:
> - resent with Greg KH in Cc list
> 
> Initial contribution have been seen here 6 weeks ago:
> https://lore.kernel.org/r/20230901134250.1172990-1-andriy.shevchenko@linux.intel.com
> 
> Andy Shevchenko (3):
>   parport: Use kasprintf() instead of fixed buffer formatting
>   parport: Use list_for_each() helper
>   parport: Drop unneeded NULL or 0 assignments

Nice cleanup!

Reviewed-by: Luis Chamberlain <mcgrof@kernel.org>

  Luis