[Skiboot] [PATCH] Revert "core/ipmi: Set interrupt-parent property"

Alistair Popple alistair at popple.id.au
Fri Nov 11 11:42:49 AEDT 2016


The problem is that pre 4.2 kernels lack the patch to create the irqdomain for 
the interrupt-parent so when the generic kernel code tries to register the IRQ 
it can't find the domain and barfs spurious warnings.

4.2+ kernels already detect the case where the irqdomain exists but the 
interrupt-parent property isn't set and fix things up appropriately which 
makes this patch nice to have rather than necessary.

Acked-By: Alistair Popple <alistair at popple.id.au>

On Fri, 11 Nov 2016 11:34:04 AM Stewart Smith wrote:
> This reverts commit d997e482705d9fdff8e25fcbe07fb56008f96ae1.
> 
> A problem was found with pre 4.2 linux kernels where a spurious WARNING
> would be emitted. This change doesn't matter enough to scare users
> so we can just revert it.
> 
> Reported-by: Pridhiviraj Paidipeddi <ppaidipe at in.ibm.com>
> Signed-off-by: Stewart Smith <stewart at linux.vnet.ibm.com>
> ---
>  core/ipmi-opal.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/core/ipmi-opal.c b/core/ipmi-opal.c
> index c86f20d..1b28aa6 100644
> --- a/core/ipmi-opal.c
> +++ b/core/ipmi-opal.c
> @@ -127,7 +127,7 @@ out_unlock:
>  
>  void ipmi_opal_init(void)
>  {
> -	struct dt_node *opal_ipmi, *opal_event;
> +	struct dt_node *opal_ipmi;
>  
>  	opal_ipmi = dt_new(opal_node, "ipmi");
>  	dt_add_property_strings(opal_ipmi, "compatible", "ibm,opal-ipmi");
> @@ -135,10 +135,6 @@ void ipmi_opal_init(void)
>  			      IPMI_DEFAULT_INTERFACE);
>  	dt_add_property_cells(opal_ipmi, "interrupts",
>  			      ilog2(ipmi_backend->opal_event_ipmi_recv));
> -	opal_event = dt_find_by_name(opal_node, "event");
> -	if (opal_event)
> -		dt_add_property_cells(opal_ipmi, "interrupt-parent",
> -				      opal_event->phandle);
>  
>  	opal_register(OPAL_IPMI_SEND, opal_ipmi_send, 3);
>  	opal_register(OPAL_IPMI_RECV, opal_ipmi_recv, 3);
> 



More information about the Skiboot mailing list