[coreboot] [PATCH] Get rid of unaligned 32-bit config space read in mcd_d.c
Marc Jones
marcj303 at gmail.com
Mon Apr 12 23:16:24 CEST 2010
On Mon, Apr 12, 2010 at 12:52 AM, Arne Georg Gleditsch
<arne.gleditsch at numascale.com> wrote:
> Hi,
>
> This patch removes a non-32-bit aligned Get_NB32 from
> src/northbridge/amd/amdmct/mct/mct_d.c. The actual behavior should be
> unchanged for I/O-based config space reads, but previous behavior was
> wrong for MMCFG-based config space reads.
>
> Perhaps we want pci_mmio_*_config* to enforce natural alignment as well?
>
>
> Signed-off-by: Arne Georg Gleditsch <arne.gleditsch at numascale.com>
>
> --
> Arne.
Hi Arne,
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.
Acked-by: Marc Jones <marcj303 at gmail.com>
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.
Marc
--
http://se-eng.com
More information about the coreboot
mailing list