lkml.org 
[lkml]   [2021]   [Nov]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v10 0/5] Add Xilinx AMS Driver
Date
Hi Andy,

Thanks for the review.

> -----Original Message-----
> From: Andy Shevchenko <andy.shevchenko@gmail.com>
> Sent: Wednesday 17 November 2021 5:14 PM
> To: Anand Ashok Dumbre <ANANDASH@xilinx.com>
> Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>; Jonathan
> Cameron <jic23@kernel.org>; Lars-Peter Clausen <lars@metafoo.de>; linux-
> iio <linux-iio@vger.kernel.org>; git <git@xilinx.com>; Michal Simek
> <michals@xilinx.com>; Greg Kroah-Hartman <gregkh@linuxfoundation.org>;
> Rafael J. Wysocki <rafael@kernel.org>; ACPI Devel Maling List <linux-
> acpi@vger.kernel.org>; Andy Shevchenko
> <andriy.shevchenko@linux.intel.com>; Krogerus, Heikki
> <heikki.krogerus@linux.intel.com>
> Subject: Re: [PATCH v10 0/5] Add Xilinx AMS Driver
>
> On Wed, Nov 17, 2021 at 6:46 PM Anand Ashok Dumbre
> <anand.ashok.dumbre@xilinx.com> wrote:
> >
> > Add Xilinx AMS driver which is used for Xilinx's ZynqMP AMS controller.
> > This AMS driver is used to report various interface voltages and
> > temperatures across the system.
> > This driver will be used by iio-hwmon to repport voltages and
> > temperatures across the system by using various channel interfaces.
> > This driver handles AMS module including PS-Sysmon & PL-Sysmon. The
> > binding documentation is added for understanding of AMS, PS, PL Sysmon
> Channels.
>
> Thanks and wait a bit, please. No need to send patches so often. We haven't
> settled down yet on v9. Here you effectively ignored my comment given
> again patch 5/5. Why?
>

Apologies for sending patch too soon. I sent the patch before I saw your response.

> ...
>
> > Changes in v10:
> > - Fixed licence in zynqmp.dtsi.
> > - Changed the macros to use BIT().
> > - Realign some code to fit within 100 chars.
> > - Modified readl_poll_timeout usage.
>
> > - Usage of array_size() instead of check_mul_overflow().
>
> Why? I have asked you why you haven't used kcalloc there as well.

Yes, I saw the implementation of kcalloc after reading your comment and
It does check_mul_overflow internally so this is not needed.

I will wait a couple of days before sending new patches.

>
> > - Usage of dev_err_probe() instead of dev_err().
> > - Usage of kcalloc instead of kzalloc()
>
> --
> With Best Regards,
> Andy Shevchenko

Thanks,
Anand
\
 
 \ /
  Last update: 2021-11-18 16:04    [W:0.144 / U:0.400 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site