Re: [PATCH] powerpc/dts: Removed fsl,msi property from dts.

[Diana CRACIUN <diana.craciun@freescale.com>]

On 03/16/2012 09:33 PM, Kumar Gala wrote:
> On Feb 9, 2012, at 7:41 AM, Diana Craciun wrote:
>
>> From: Diana CRACIUN<Diana.Craciun@freescale.com>
>>
>> The association in the decice tree between PCI and MSI
>> using fsl,msi property was an artificial one and it does
>> not reflect the actual hardware.
>>
>> Signed-off-by: Diana CRACIUN<Diana.Craciun@freescale.com>
>> ---
>> arch/powerpc/boot/dts/p2041rdb.dts |    3 ---
>> arch/powerpc/boot/dts/p3041ds.dts  |    4 ----
>> arch/powerpc/boot/dts/p3060qds.dts |    2 --
>> arch/powerpc/boot/dts/p4080ds.dts  |    3 ---
>> arch/powerpc/boot/dts/p5020ds.dts  |    4 ----
>> 5 files changed, 0 insertions(+), 16 deletions(-)
> Should we not also remove the code associated with this property?
>
> - k

No. The property may be present in the device tree when running under 
Freescale hypervisor and it is used to skip the MSI nodes which are not 
mapped in PAMU.

The code in kernel handles both situations: when the property is present 
and in this case it searches only the corresponding MSI device for 
available interrupts and when the property is not present and in this 
case it loops over all the MSI devices.

Diana