gpio: at91: add sanity check for the NULL pointer
We need check the NULL pointer as at91_pio_get_port() may return NULL. Also print a error message when at91_pio_get_port() failed otherwise we cannot notice the failure. Signed-off-by: Josh Wu <josh.wu@atmel.com> Signed-off-by: Andreas Bießmann <andreas.devel@googlemail.com>
This commit is contained in:
parent
4952759251
commit
7d82d89772
|
@ -34,6 +34,7 @@ static struct at91_port *at91_pio_get_port(unsigned port)
|
||||||
#endif
|
#endif
|
||||||
#endif
|
#endif
|
||||||
default:
|
default:
|
||||||
|
printf("Error: at91_gpio: Fail to get PIO base!\n");
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -200,7 +201,7 @@ int at91_set_pio_output(unsigned port, u32 pin, int value)
|
||||||
struct at91_port *at91_port = at91_pio_get_port(port);
|
struct at91_port *at91_port = at91_pio_get_port(port);
|
||||||
u32 mask;
|
u32 mask;
|
||||||
|
|
||||||
if ((port < ATMEL_PIO_PORTS) && (pin < 32)) {
|
if (at91_port && (port < ATMEL_PIO_PORTS) && (pin < 32)) {
|
||||||
mask = 1 << pin;
|
mask = 1 << pin;
|
||||||
writel(mask, &at91_port->idr);
|
writel(mask, &at91_port->idr);
|
||||||
writel(mask, &at91_port->pudr);
|
writel(mask, &at91_port->pudr);
|
||||||
|
|
Loading…
Reference in New Issue