On Fri, May 08, 2009 at 06:29:12PM +0200, Carl-Daniel Hailfinger wrote:
Signed-off-by: Carl-Daniel Hailfinger c-d.hailfinger.devel.2006@gmx.net
With the change below: Acked-by: Uwe Hermann uwe@hermann-uwe.de
Index: flashrom-external_functionpointer/flash.h
--- flashrom-external_functionpointer/flash.h (Revision 473) +++ flashrom-external_functionpointer/flash.h (Arbeitskopie) @@ -76,34 +76,64 @@
[...]
+extern const struct programmer_entry programmer_table[];
+static inline int programmer_init(void)
I'd drop all those 'inline's, they're pretty useless and they're not enforced by the compiler anyway. And we couldn't care less about potential speed improvements here anyway.
Let's postpone that to another patch though.
+/* internal.c */ +int internal_init(void); +int internal_shutdown(void); +void internal_chip_writeb(uint8_t b, volatile void *addr); +void internal_chip_writew(uint16_t b, volatile void *addr); +void internal_chip_writel(uint32_t b, volatile void *addr);
b -> val
Index: flashrom-external_functionpointer/flashrom.c
--- flashrom-external_functionpointer/flashrom.c (Revision 473)
- ret = programmer_init();
Unused right now, we should add some error handling in another patch.
Uwe.