[U-Boot] [PATCH v1 3/4] x86: tangier: Fix off-by-one error when preparing CSRT

Bin Meng bmeng.cn at gmail.com
Tue Sep 10 08:21:09 UTC 2019


On Tue, Sep 10, 2019 at 4:14 PM Bin Meng <bmeng.cn at gmail.com> wrote:
>
> On Thu, Aug 29, 2019 at 10:04 PM Andy Shevchenko
> <andriy.shevchenko at linux.intel.com> wrote:
> >
> > Intel iDMA 32-bit controller has 17 bits for the maximum block size value.
> > Due to nature of the binary number representation the maximum value is
> > 2^17 - 1. The original code misses the latter part in equation.
> >
> > Fixes: 5e99fde34a77 ("x86: tangier: Populate CSRT for shared DMA controller")
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> > ---
> >  arch/x86/cpu/tangier/acpi.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
>
> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>

applied to u-boot-x86, thanks!


More information about the U-Boot mailing list