On Thu, 08 Apr 2010 20:42:17 +0200
Florian Tobias Schandinat [off-list ref] wrote:

something I am wondering about is whether we can't simply do:
viaparinfo->memsize = pci_resource_len(pdev, 0);
I suppose that this is not possible meaning that the pci len can be 
longer than the actual memory but I just wanted to use the moment to 
make sure.

That would make sense.  But if somebody who is closer to the hardware than
I am doesn't take that approach, I'm nervous about changing it.  Harald?

This function was not designed to return an error (memsize is not 
checked). Either return a default value (let's say 8MB) or add a check 
for memsize.

That's a good point.  I put in a check.

Thanks,

jon