[PATCH v6 2/7] wl1271: propagate set_power's return value
Luciano Coelho
luciano.coelho at nokia.com
Thu Sep 16 15:56:51 EDT 2010
On Thu, 2010-09-16 at 21:53 +0200, ext Ohad Ben-Cohen wrote:
> On Thu, Sep 16, 2010 at 9:40 PM, Luciano Coelho
> <luciano.coelho at nokia.com> wrote:
> >> + int ret = wl->if_ops->power(wl, true);
> >
> > I think it look nicer if you keep the "int ret" in one line by itself
> > and then do a ret = wl->if_ops... on another one.
>
> Fixed.
>
> >> +static int wl1271_sdio_power_on(struct wl1271 *wl)
> >> {
> >> struct sdio_func *func = wl_to_func(wl);
> >>
> >> sdio_claim_host(func);
> >> sdio_enable_func(func);
> >> sdio_release_host(func);
> >> +
> >> + return 0;
> >> }
> >
> > You seem to always return 0, so the whole chain to pass the value up
> > seems unnecessary. Is this just a preparation for a future patch?
>
> Yes, it's soon going to be:
>
> static int wl1271_sdio_power_on(struct wl1271 *wl)
> {
> struct sdio_func *func = wl_to_func(wl);
> int ret;
>
> ret = pm_runtime_get_sync(&func->dev);
> if (ret)
> goto out;
>
> sdio_claim_host(func);
> sdio_enable_func(func);
> sdio_release_host(func);
>
> out:
> return ret;
> }
>
Ok, that was the only explanation I could think of ;)
Acked-by: Luciano Coelho <luciano.coelho at nokia.com>
--
Cheers,
Luca.
More information about the linux-arm-kernel
mailing list