Re: [coreboot] [PATCH] Get rid of unaligned 32-bit config space read in mcd_d.c

[Arne Georg Gleditsch]

Marc Jones  writes:
> That code is doing something a little ugly to make the two cs
> registers use the one mask register. It was ported from a routine that
> assumed that GET_NB32() fixed the alignment. Maybe add a comment about
> why that is happening.

I assumed as much.  I'll see if I can't put a comment in.
> Acked-by: Marc Jones
>
> I would also ack a change to:
> u32 Get_NB32(u32 dev, u32 reg)
> u32 Get_NB32(u32 dev, u32 reg)
>
> or to the  pci_mmio_*_config* to do it since the pci_io* versions do.

Thanks, I'll include this as well.  The current situation with different
semantics on unaligned accesses isn't really helping anyone, as far as I
can see.