PL011: fix reverse logic for interrupt mask register
authorAndre Przywara <andre.przywara@linaro.org>
Tue, 13 Aug 2013 15:12:35 +0000 (17:12 +0200)
committerIan Campbell <ian.campbell@citrix.com>
Wed, 21 Aug 2013 09:29:07 +0000 (10:29 +0100)
commit874f76a86adf1da70921884f5a868eec105cf8cd
treef4456f90dcdb8e792b2fc6cee897207b53759500
parent5263507b1b4ad5417871d8297f315d7b204426d4
PL011: fix reverse logic for interrupt mask register

The PL011 IMSC register description is somehow fuzzy in the
documentation; by comparing it with the Linux implementation one can
see that the logic is actually reversed to Xen's implementation:
A "0" in field means interrupt disabled, a "1" enables it.
Therefore we enabled all interrupts instead of disabling them in the
beginning and later on masked the wrong interrupts.
Unclear how this worked on the Versatile Express, but this fix is
needed to get Calxeda Midway running (and works on VExpress, too).

Signed-off-by: Andre Przywara <andre.przywara@linaro.org>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
xen/drivers/char/pl011.c