[flashrom] [PATCH 2/3] Add support for SPARC (maybe).

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Sun Jan 25 03:04:09 CET 2015


On 19.01.2015 21:39, Stefan Tauner wrote:
> Does (cross-)compile but is not run-tested.
>
> Signed-off-by: Stefan Tauner <stefan.tauner at alumni.tuwien.ac.at>
> Acked-by: Stefan Tauner <stefan.tauner at alumni.tuwien.ac.at>
> ---
>  Makefile   | 2 +-
>  hwaccess.h | 8 ++++++++
>  platform.h | 5 ++++-
>  3 files changed, 13 insertions(+), 2 deletions(-)

hwaccess.c needs to be patched as well. Specifically,
static inline void sync_primitive(void)
needs either a comment why sync_primitive is unneeded or a a code
snippet with correct code.

Not sure if I read
http://lxr.free-electrons.com/source/arch/sparc/include/asm/barrier_64.h#L47
correctly, but we may need

__asm__ __volatile__("membar #StoreLoad":::"memory")


Mh. That might not be needed if /dev/mem uses a special access mode
which enforces memory access ordering in Sparc. Someone else with Sparc
knowledge needs to check this.

That said, the patch looks correct apart from the missing hwaccess.c
stuff mentioned above.


Regards,
Carl-Daniel





More information about the flashrom mailing list