pci issue - wrong detection of pci ressources

[Christian Ehrhardt]

Sergei Shtylyov wrote:
> Hello.
> 
> Christian Ehrhardt wrote:
> 
[...]
> 
>> The Documentation of the 440EPx core lists these spaces:
>> PCI 1 Memory     1 8000 0000     1 BFFF FFFF     1GB
>> I/O              1 E800 0000     1 E800 FFFF     64KB
>> I/O              1 E880 0000     1 EBFF FFFF     56MB
> 
> Having 2 I/O spaces looks just wrong. Actually, PCs do well with only 
> 64K of I/O space.
> 

ok - I just wanted to be complete.
I removed the 56M section from the new dts file patch.

[...]

>> radeonfb: EDID probed
>> Parsing EDID data for panel info
>> Setting up default mode based on panel info
>> radeonfb (0000:00:0a.0): ATI Radeon Y`
> 
>    Hm, what's that Y`?

Thats the final message in the radeonfb driver ater initializing everything.
   printk ("radeonfb (%s): %s\n", pci_name(rinfo->pdev), rinfo->name);
I wonder why that rinfo->name is clobbered - maybe another issue, I have to keep that in mind.

[...]
> 
>    I think you'd better use Ben's patch that he's just posted:
> 
> http://patchwork.ozlabs.org/linuxppc/patch?id=18034
> 
> WBR, Sergei

yep - I use Ben's patch now which reduces my patch to the actual dts fix.
Updated patch attached.

-- 

Grüsse / regards, 
Christian Ehrhardt
IBM Linux Technology Center, Open Virtualization
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: fix-sequoia-pci-memory-dts
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20080422/ce796b80/attachment.txt>