lkml.org 
[lkml]   [2008]   [Mar]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [patch] pci: revert "PCI: remove transparent bridge sizing"
From
Date

On Thu, 2008-03-27 at 02:29 +0300, Ivan Kokshaysky wrote:

> Well, at this point (pdev_sort_resources call) bridge resource->start
> has nothing to do with a bus address, it just a temporary storage for
> required alignment, filled by sizing routines (and 0 is definitely
> invalid here).

Allright, that sounds better then.

> I know, this is quite confusing, but I didn't want to add extra
> fields to existing structures or create temporary per-bus trees...
> But after pci_assign_resource() that resource can certainly be at 0,
> depending on PCIBIOS_MIN_{IO,MEM} and free slots in the resource tree.

In fact, this is the resource assignment code, and PCIBIOS_MIN_* is set
to non-0 for powerpc too so we don't try to assign things down at 0
anyway, so we don't have a problem. We only care about things at 0 that
have already been put there by the firmware and that we decide not to
reassign.

So I don't have any objection anymore.

Cheers,
Ben.




\
 
 \ /
  Last update: 2008-03-27 00:51    [W:0.168 / U:0.408 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site