RE: [PATCH 2/4] pinctrl: Fix pinmux_hog_maps when ctrl_dev_name isnot set

From: Stephen Warren
Date: Tue Jan 24 2012 - 19:48:06 EST


Tony Lindgren wrote at Tuesday, January 24, 2012 5:24 PM:
...
> The ctrl_dev_name is optional for struct pinmux_map assuming
> that ctrl_dev is set. Without this patch we can get:
>
> Unable to handle kernel NULL pointer dereference at virtual address 00000000
> ...
> (pinmux_hog_maps+0xa4/0x20c)
> (pinctrl_register+0x2a4/0x378)
> ...
>
> Fix this by adding adding a test for map->ctrl_dev.
> Additionally move the test for map->ctrl_dev earlier
> to optimize out the loop a bit.
>
> Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>

Acked-by: Stephen Warren <swarren@xxxxxxxxxx>

--
nvpublic

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/