2014-06-11 15:24:13

by Alan Stern

[permalink] [raw]
Subject: Re: [PATCH v2] usb: host: uhci-grlib.c : use devm_ functions

On Tue, 10 Jun 2014, Himangi Saraogi wrote:

> The various devm_ functions allocate memory that is released when a
> driver detaches. This patch uses devm_ioremap_resource for data
> that is allocated in the probe function of a platform device and
> is only freed in the remove function. The corresponding free functions
> are removed and two labels are done away with. Also, linux/device.h
> is added to make sure the devm_*() routine declarations are
> unambiguously available.
>
> Signed-off-by: Himangi Saraogi <[email protected]>
> Acked-by: Julia Lawall <[email protected]>
> ---
> v2: fix error of devm_ioremap_resource and change label names.


> + hcd->regs = devm_ioremap_resource(&op->dev, hcd);

Obviously you haven't tried to compile this.

Alan Stern