Re: [PATCH] pinctrl: amd: Add support for additional GPIO

[Linus Walleij <linus.walleij@xxxxxxxxxx>]

On Thu, Dec 1, 2016 at 9:13 AM, Shah, Nehal-bakulchandra
<Nehal-Bakulchandra.shah@xxxxxxx> wrote:

> This patch adds support for new Bank and add IRQCHIP_SKIP_SET_WAKE flag.
>
> Reviewed-by: S-k, Shyam-sundar <Shyam-sundar.S-k@xxxxxxx>
> Signed-off-by: Nehal Shah <Nehal-bakulchandra.Shah@xxxxxxx>

(...)
>  static int amd_gpio_probe(struct platform_device *pdev)
>  {
> +       int hwnum;
>         int ret = 0;
>         int irq_base;
>         struct resource *res;
>         struct amd_gpio *gpio_dev;
> +       struct acpi_device *adev = ACPI_COMPANION(&pdev->dev);
> +
> +       ret = kstrtoint(acpi_device_uid(adev), 2, &hwnum);
>
>         gpio_dev = devm_kzalloc(&pdev->dev,
>                                 sizeof(struct amd_gpio), GFP_KERNEL);
> @@ -763,16 +771,17 @@ static int amd_gpio_probe(struct platform_device *pdev)
>         gpio_dev->gc.set                        = amd_gpio_set_value;
>         gpio_dev->gc.set_debounce       = amd_gpio_set_debounce;
>         gpio_dev->gc.dbg_show           = amd_gpio_dbg_show;
> +       gpio_dev->gc.base               = -1;

Good!

But now hwnum is an unused variable.

Yours,
Linus Walleij
--
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