[PATCH v5 2/3] wl18xx: add basic device-tree support

Eliad Peller eliad at wizery.com
Wed Mar 11 04:50:47 PDT 2015

On Wed, Mar 11, 2015 at 11:51 AM, Arnd Bergmann <arnd at arndb.de> wrote:
>> > +static void wlcore_del_platform_data(struct wl12xx_platform_data *pdata)
>> > +{
>> > +       kfree(pdata);
>> > +}
>> > +
>> This function seems to be an unnecessary, why not just call kfree() directly?
>> Or better, maybe the resource-managed devm_*() functions can be used
>> so the data doesn't have to be explicitly freed?
> As I said earlier, I think it would be best not to dynamically allocate anything
> here at all. As Eliad explained, the data is used by two different drivers:
> wl12xx and wl18xx, and only the latter is converted for now, but after the
> conversion, it should not need the platform data structure any more, only
> the irq number that gets passed in from DT.
sure, i'll try taking care of it (probably with additional patch after
the conversion)


More information about the linux-arm-kernel mailing list