Matthew Wilcox wrote:
> On Mon, Jan 31, 2005 at 10:52:29PM -0600, Brian King wrote:
> 
>>@@ -62,8 +72,11 @@ static int rtas_read_config(struct devic
>> 		return PCIBIOS_DEVICE_NOT_FOUND;
>> 	if (where & (size - 1))
>> 		return PCIBIOS_BAD_REGISTER_NUMBER;
> 
> 
> You should probably delete this redundant test at the same time ...

Done. The new patch below also adds some address checking to iSeries 
config accessor functions. Additionally, this patch should address 
Arnd's concern, as it now looks for the "ibm,pci-config-space-type" 
property on the device itself rather than on the bridge.


-- 
Brian King
eServer Storage I/O
IBM Linux Technology Center