Re: [PATCH 1/4] ARM: shmobile: sh73a0: Set control-parent for all irqpin nodes

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Magnus,

On Thursday 18 December 2014 12:17:16 Magnus Damm wrote:
> On Thu, Dec 18, 2014 at 11:26 AM, Laurent Pinchart wrote:
> > The sh73a0 INTC can't mask interrupts properly most likely due to a
> > hardware bug. Set the control-parent property to delegate masking to the
> > parent interrupt controller.
> 
> Thanks for your patch. Are you sure it is needed for all interrupts?

I'm not as I can't test all interrupts, but given that it's needed for two 
interrupts on two different INTC instances (well, two different INTC DT nodes, 
but it's really the same IP core) I'm pretty confident that the workaround is 
needed for all INTC interrupts on sh73a0.

> If so, perhaps it we should consider to enable this in the driver based on
> the SoC compat string?

I was thinking about that. We could then remove the property from the DT 
bindings.

-- 
Regards,

Laurent Pinchart