[U-Boot] [PATCH] gpio: generalize for all generic gpio providers

Mike Frysinger vapier at gentoo.org
Tue Apr 12 05:14:19 CEST 2011


On Monday, April 11, 2011 15:34:17 Andreas Pretzsch wrote:
> Am Sonntag, den 03.04.2011, 04:43 -0400 schrieb Mike Frysinger:
> > +	return port_base + simple_strtoul(name, NULL, 10);
> 
> Remark: Leads to an oom access when exceeding the processor number of
> GPIOs, e.g. PF48 on a BF561. IMHO, no problem but only a cosmetic issue,
> not worth adding an additional per-cpu check.
> In the end, no difference to other user errors like an invalid memory
> address.

err, oom ?  i guess you mean oob ?

simple to fix by having the return of gpio_request() get checked ...
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20110411/942891a6/attachment.pgp 


More information about the U-Boot mailing list