Messages in this thread | | | From | Andy Shevchenko <> | Date | Thu, 28 Jun 2018 03:00:58 +0300 | Subject | Re: [PATCH v5 09/10] gpio: Add a driver for Cadence I3C GPIO expander |
| |
On Thu, Jun 28, 2018 at 1:54 AM, Joe Perches <joe@perches.com> wrote: > On Wed, 2018-06-27 at 21:36 +0200, Boris Brezillon wrote: >> I mean, kzalloc() is not deprecated AFAIK and I don't really see the >> benefit of using kmalloc_array(), but if that makes you happy, let's go >> for kmalloc_array(). > > kcalloc
IIRC in the original code kmalloc(x*y) is used.
>> I do run checkpatch --strict and fix most of the thing reported except >> those hurting readability. I don't remember seeing checkpatch complain >> about kzalloc() usage, and I guess it's not smart enough to detect that >> for_each_bit_set() can be used to replace the "for() if (BIT(x) & val)" >> pattern. > > That would not an appropriate conversion suggestion in any case. > coccinelle could at least look at whether or not x is allocated > as a bitmap via DECLARE_BITMAP or bitmap_alloc
Huh?!
bitmap operations are working against unsigned long *. one long is also a bitmap. So, that coccinelle scripts must be fixed accordingly.
-- With Best Regards, Andy Shevchenko
| |