[flashrom] [PATCH]Hide hwaccess.h from public API

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Fri Jul 20 22:39:39 CEST 2012


Am 20.07.2012 20:51 schrieb Patrick Georgi:
> attached patch hides hwaccess.h (which is usually not necessary to use
> flashrom as library) from the public interface at flash.h.
> For the most part, this simply adds the header to driver .c files, but
> it also required a small workaround to handle libpci types used in a
> bunch of prototypes.
>
> Rationale: hwaccess.h at that place conflicts with libpayload which
> brings its own hardware access functions (which are wrapped properly
> to match flashrom's expectations in hwaccess.h already). They should
> also not matter in other contexts (and if someone requires them, they
> can still include hwaccess.h where needed).

Thanks for your patch.


> Librarification of flashrom requires some more patches, like pulling
> out some functions of the CLI code - even if doit() is by no means a
> pretty interface, it makes things simpler. But one step after the other...

I have some makefile patch pending which makes libflashrom usage easier.
I should fix that one and resend it.


> Signed-off-by: Patrick Georgi <patrick.georgi at secunet.com>

Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
and committed in r1549.

Admin note: Apparently patchwork didn't pick up this thread.

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the flashrom mailing list