[PATCH] Updated Legacy code in arm rtl22xx
Sebastian Huber
sebastian.huber at embedded-brains.de
Fri Jul 12 19:30:39 UTC 2013
On 12/07/13 17:30, Vipul Nayyar wrote:
> Ok, So before sending the updated patch I'll list in advance what I'm gonna change, so that we don't have to go through this again.
>
> 1) Change cs8900_isr to static. The sample code would be:
> static void cs8900_isr(void *arg)
> {
> cs8900_device *cs = arg;
> cs8900_interrupt(LPC22xx_INTERRUPT_EINT2, cs);
> }
This is all right in principle, but you don't need the intermediate
variable.
Now the questions are:
1. Who will call this function?
2. Where does the actual arg argument value come from?
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone : +49 89 189 47 41-16
Fax : +49 89 189 47 41-09
E-Mail : sebastian.huber at embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
More information about the devel
mailing list