Re: [PATCH 1/5] PCI: add resizeable BAR infrastructure v2

From: Christian KÃnig
Date: Mon Mar 13 2017 - 05:44:00 EST


Am 06.03.2017 um 13:20 schrieb Andy Shevchenko:
On Mon, Mar 6, 2017 at 1:40 PM, Christian KÃnig <deathsimple@xxxxxxxxxxx> wrote:
+ for (i = 0; i < nbars; ++i, pos += 8) {
8 is defined somewhere in the spec? (Yes, I understand that is just 64
bits shift)

Yes, it is defined in the spec, see "Figure 7-x1 Resizable BAR Capability".

It just doesn't have a name or something if that's what you are asking for.

I've addressed all other comments in code and will send out the next version of the patch set today.

Thanks for the comments,
Christian.