Re: [PATCH] pinctrl: sh-pfc: Kill unused variable in sh_pfc_remove()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Tuesday 26 April 2016 04:00 PM, Geert Uytterhoeven wrote:
If CONFIG_PINCTRL_SH_PFC_GPIO=n:

     drivers/pinctrl/sh-pfc/core.c: In function 'sh_pfc_remove':
     drivers/pinctrl/sh-pfc/core.c:649:17: warning: unused variable 'pfc' [-Wunused-variable]

Fixes: 67ec8d7b48463904 ("pinctrl: ish-pfc: Use devm_pinctrl_register() for pinctrl registration")
Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
---
I won't queue this up in sh-pfc-for-v4.7, as the offending commit is not in
that branch.


Ideally, you dont need the call of sh_pfc_unregister_gpiochip() if you use the devm_gpiochip_add_data() which is already available on all subsystem.

int sh_pfc_unregister_gpiochip(struct sh_pfc *pfc)
{
        gpiochip_remove(&pfc->gpio->gpio_chip);
#ifdef CONFIG_SUPERH
        gpiochip_remove(&pfc->func->gpio_chip);
#endif
        return 0;
}

So you can get rid of this calls and lines.


-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux