On Thu, May 19, 2016 at 12:00 PM, Simon Glass <[email protected]> wrote: > On 11 May 2016 at 08:44, Bin Meng <[email protected]> wrote: >> PIRQ routing table checksum is fixed up in copy_pirq_routing_table(), >> which is fine if we only write the configuration table once. But with >> the SeaBIOS case, when we write the table for the second time, the >> checksum will be fixed up to zero per the checksum algorithm, which >> is caused by the checksum field not being zero before fix up, since >> the checksum has already been calculated in the first run. >> >> To fix this, move the checksum fixup to create_pirq_routing_table(), >> so that copy_pirq_routing_table() only does what its function name >> suggests: copy the table to somewhere else. >> >> Signed-off-by: Bin Meng <[email protected]> >> --- >> >> Changes in v2: None >> >> arch/x86/cpu/irq.c | 4 ++++ >> arch/x86/lib/pirq_routing.c | 4 ---- >> 2 files changed, 4 insertions(+), 4 deletions(-) > > Reviewed-by: Simon Glass <[email protected]>
applied to u-boot-x86, thanks! _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
