* pSeries boot failure due to wrong interrupt controller
@ 2015-06-22 19:44 Jeremy Linton
[not found] ` <CAEFTgiyHkf8ooYdWW0nTLcdraJ_L9FpLSVZa4KS3CcdpGWJxew-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 10+ messages in thread
From: Jeremy Linton @ 2015-06-22 19:44 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA, paulus-eUNUBHrolfbYtjvyW6yDsg
The OF IRQ logic was refactored a while back, and it appears that it
broke the case where a device doesn't have an interrupt-map
(legacy/ISA platform peripherals). In particular on the older pSeries
platforms the console serial port (keyboard/etc) is attached via a
Winbond super I/O controller. With the refactored logic
of_irq_parse_raw() returns with the out_irq->np pointing at the bus,
rather than the interrupt controller. This results in the platform
logic getting confused about the appropriate interrupt host, and in
ics_rtas_host_match() it doesn't match "chrp,iic" and the xics takes
over and tries to use RTAS to map the interrupt. RTAS then crashes the
machine.
This tweaks the "-> no map, getting parent" behavior to behave as
before, and the machine boots.
Thanks
Signed-off-by: Jeremy Linton <lintonrjeremy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
diff --git a/drivers/of/irq.c b/drivers/of/irq.c
index 1a79806..78b4161b 100644
--- a/drivers/of/irq.c
+++ b/drivers/of/irq.c
@@ -252,7 +252,6 @@ int of_irq_parse_raw(const __be32 *addr, struct
of_phandle_args *out_irq)
* Successfully parsed an interrrupt-map translation; copy new
* interrupt specifier into the out_irq structure
*/
- out_irq->np = newpar;
match_array = imap - newaddrsize - newintsize;
for (i = 0; i < newintsize; i++)
@@ -262,6 +261,7 @@ int of_irq_parse_raw(const __be32 *addr, struct
of_phandle_args *out_irq)
skiplevel:
/* Iterate again with new parent */
+ out_irq->np=newpar;
pr_debug(" -> new parent: %s\n", of_node_full_name(newpar));
of_node_put(ipar);
ipar = newpar;
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: pSeries boot failure due to wrong interrupt controller
[not found] ` <CAEFTgiyHkf8ooYdWW0nTLcdraJ_L9FpLSVZa4KS3CcdpGWJxew-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2015-06-23 13:42 ` Rob Herring
[not found] ` <CAL_JsqJqkUL8XyqR6yjg+-thj4YOQ6cUohGHHid71tdRFft7vw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 10+ messages in thread
From: Rob Herring @ 2015-06-23 13:42 UTC (permalink / raw)
To: Jeremy Linton, Grant Likely
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, Paul Mackerras
+Grant
On Mon, Jun 22, 2015 at 2:44 PM, Jeremy Linton <lintonrjeremy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> The OF IRQ logic was refactored a while back, and it appears that it
> broke the case where a device doesn't have an interrupt-map
> (legacy/ISA platform peripherals). In particular on the older pSeries
> platforms the console serial port (keyboard/etc) is attached via a
> Winbond super I/O controller. With the refactored logic
> of_irq_parse_raw() returns with the out_irq->np pointing at the bus,
> rather than the interrupt controller. This results in the platform
> logic getting confused about the appropriate interrupt host, and in
> ics_rtas_host_match() it doesn't match "chrp,iic" and the xics takes
> over and tries to use RTAS to map the interrupt. RTAS then crashes the
> machine.
>
> This tweaks the "-> no map, getting parent" behavior to behave as
> before, and the machine boots.
I think the real question is why doesn't of_irq_find_parent return the
interrupt controller node instead of the bus node.
While I think this fix is correct, I think of_irq_parse_raw should be
a nop in the case of no interrupt-map.
Rob
>
> Thanks
>
> Signed-off-by: Jeremy Linton <lintonrjeremy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>
>
> diff --git a/drivers/of/irq.c b/drivers/of/irq.c
> index 1a79806..78b4161b 100644
> --- a/drivers/of/irq.c
> +++ b/drivers/of/irq.c
> @@ -252,7 +252,6 @@ int of_irq_parse_raw(const __be32 *addr, struct
> of_phandle_args *out_irq)
> * Successfully parsed an interrrupt-map translation; copy new
> * interrupt specifier into the out_irq structure
> */
> - out_irq->np = newpar;
>
> match_array = imap - newaddrsize - newintsize;
> for (i = 0; i < newintsize; i++)
> @@ -262,6 +261,7 @@ int of_irq_parse_raw(const __be32 *addr, struct
> of_phandle_args *out_irq)
>
> skiplevel:
> /* Iterate again with new parent */
> + out_irq->np=newpar;
> pr_debug(" -> new parent: %s\n", of_node_full_name(newpar));
> of_node_put(ipar);
> ipar = newpar;
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: pSeries boot failure due to wrong interrupt controller
2015-06-23 13:42 ` Rob Herring
@ 2015-06-26 17:54 ` Jeremy Linton
0 siblings, 0 replies; 10+ messages in thread
From: Jeremy Linton @ 2015-06-26 17:54 UTC (permalink / raw)
To: Rob Herring, Jeremy Linton, Grant Likely
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, Paul Mackerras,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ,
benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r
Ping? Anyone?
I'm adding ppc-dev to see if anyone there is interested in picking up the patch..
Thanks,
Jeremy
Ok so I messed up the formatting a little and was lazy and didn't checkpatch it
cause I mean, it was 1 line! If someone wants it resent I will do so.
On 6/23/2015 8:42 AM, Rob Herring wrote:
> +Grant
>
> On Mon, Jun 22, 2015 at 2:44 PM, Jeremy Linton <lintonrjeremy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> The OF IRQ logic was refactored a while back, and it appears that it
>> broke the case where a device doesn't have an interrupt-map
>> (legacy/ISA platform peripherals). In particular on the older pSeries
>> platforms the console serial port (keyboard/etc) is attached via a
>> Winbond super I/O controller. With the refactored logic
>> of_irq_parse_raw() returns with the out_irq->np pointing at the bus,
>> rather than the interrupt controller. This results in the platform
>> logic getting confused about the appropriate interrupt host, and in
>> ics_rtas_host_match() it doesn't match "chrp,iic" and the xics takes
>> over and tries to use RTAS to map the interrupt. RTAS then crashes the
>> machine.
>>
>> This tweaks the "-> no map, getting parent" behavior to behave as
>> before, and the machine boots.
>
> I think the real question is why doesn't of_irq_find_parent return the
> interrupt controller node instead of the bus node.
>
> While I think this fix is correct, I think of_irq_parse_raw should be
> a nop in the case of no interrupt-map.
>
> Rob
>
>>
>> Thanks
>>
>> Signed-off-by: Jeremy Linton <lintonrjeremy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>
>>
>> diff --git a/drivers/of/irq.c b/drivers/of/irq.c
>> index 1a79806..78b4161b 100644
>> --- a/drivers/of/irq.c
>> +++ b/drivers/of/irq.c
>> @@ -252,7 +252,6 @@ int of_irq_parse_raw(const __be32 *addr, struct
>> of_phandle_args *out_irq)
>> * Successfully parsed an interrrupt-map translation; copy new
>> * interrupt specifier into the out_irq structure
>> */
>> - out_irq->np = newpar;
>>
>> match_array = imap - newaddrsize - newintsize;
>> for (i = 0; i < newintsize; i++)
>> @@ -262,6 +261,7 @@ int of_irq_parse_raw(const __be32 *addr, struct
>> of_phandle_args *out_irq)
>>
>> skiplevel:
>> /* Iterate again with new parent */
>> + out_irq->np = newpar;
>> pr_debug(" -> new parent: %s\n", of_node_full_name(newpar));
>> of_node_put(ipar);
>> ipar = newpar;
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: pSeries boot failure due to wrong interrupt controller
@ 2015-06-26 17:54 ` Jeremy Linton
0 siblings, 0 replies; 10+ messages in thread
From: Jeremy Linton @ 2015-06-26 17:54 UTC (permalink / raw)
To: Rob Herring, Jeremy Linton, Grant Likely
Cc: devicetree, Paul Mackerras, linuxppc-dev, benh
Ping? Anyone?
I'm adding ppc-dev to see if anyone there is interested in picking up the patch..
Thanks,
Jeremy
Ok so I messed up the formatting a little and was lazy and didn't checkpatch it
cause I mean, it was 1 line! If someone wants it resent I will do so.
On 6/23/2015 8:42 AM, Rob Herring wrote:
> +Grant
>
> On Mon, Jun 22, 2015 at 2:44 PM, Jeremy Linton <lintonrjeremy@gmail.com> wrote:
>> The OF IRQ logic was refactored a while back, and it appears that it
>> broke the case where a device doesn't have an interrupt-map
>> (legacy/ISA platform peripherals). In particular on the older pSeries
>> platforms the console serial port (keyboard/etc) is attached via a
>> Winbond super I/O controller. With the refactored logic
>> of_irq_parse_raw() returns with the out_irq->np pointing at the bus,
>> rather than the interrupt controller. This results in the platform
>> logic getting confused about the appropriate interrupt host, and in
>> ics_rtas_host_match() it doesn't match "chrp,iic" and the xics takes
>> over and tries to use RTAS to map the interrupt. RTAS then crashes the
>> machine.
>>
>> This tweaks the "-> no map, getting parent" behavior to behave as
>> before, and the machine boots.
>
> I think the real question is why doesn't of_irq_find_parent return the
> interrupt controller node instead of the bus node.
>
> While I think this fix is correct, I think of_irq_parse_raw should be
> a nop in the case of no interrupt-map.
>
> Rob
>
>>
>> Thanks
>>
>> Signed-off-by: Jeremy Linton <lintonrjeremy@gmail.com>
>>
>>
>> diff --git a/drivers/of/irq.c b/drivers/of/irq.c
>> index 1a79806..78b4161b 100644
>> --- a/drivers/of/irq.c
>> +++ b/drivers/of/irq.c
>> @@ -252,7 +252,6 @@ int of_irq_parse_raw(const __be32 *addr, struct
>> of_phandle_args *out_irq)
>> * Successfully parsed an interrrupt-map translation; copy new
>> * interrupt specifier into the out_irq structure
>> */
>> - out_irq->np = newpar;
>>
>> match_array = imap - newaddrsize - newintsize;
>> for (i = 0; i < newintsize; i++)
>> @@ -262,6 +261,7 @@ int of_irq_parse_raw(const __be32 *addr, struct
>> of_phandle_args *out_irq)
>>
>> skiplevel:
>> /* Iterate again with new parent */
>> + out_irq->np = newpar;
>> pr_debug(" -> new parent: %s\n", of_node_full_name(newpar));
>> of_node_put(ipar);
>> ipar = newpar;
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: pSeries boot failure due to wrong interrupt controller
2015-06-26 17:54 ` Jeremy Linton
@ 2015-06-26 21:54 ` Benjamin Herrenschmidt
-1 siblings, 0 replies; 10+ messages in thread
From: Benjamin Herrenschmidt @ 2015-06-26 21:54 UTC (permalink / raw)
To: Jeremy Linton
Cc: Rob Herring, Jeremy Linton, Grant Likely,
devicetree-u79uwXL29TY76Z2rM5mHXA, Paul Mackerras,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ
On Fri, 2015-06-26 at 12:54 -0500, Jeremy Linton wrote:
> Ping? Anyone?
>
> I'm adding ppc-dev to see if anyone there is interested in picking up the patch..
Ah I missed that completely. From your discussion, it looks correct,
I'll see if I can find a machine to test, however I think we dropped
support for POWER3 so it might be non-trivial to do so.
And yes, when there is no map, the parent should be the interrupt-parent
with a fallback to the parent node if that doesn't exist.
Cheers,
Ben.
> Thanks,
> Jeremy
>
>
> Ok so I messed up the formatting a little and was lazy and didn't checkpatch it
> cause I mean, it was 1 line! If someone wants it resent I will do so.
>
>
>
>
>
> On 6/23/2015 8:42 AM, Rob Herring wrote:
> > +Grant
> >
> > On Mon, Jun 22, 2015 at 2:44 PM, Jeremy Linton <lintonrjeremy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> >> The OF IRQ logic was refactored a while back, and it appears that it
> >> broke the case where a device doesn't have an interrupt-map
> >> (legacy/ISA platform peripherals). In particular on the older pSeries
> >> platforms the console serial port (keyboard/etc) is attached via a
> >> Winbond super I/O controller. With the refactored logic
> >> of_irq_parse_raw() returns with the out_irq->np pointing at the bus,
> >> rather than the interrupt controller. This results in the platform
> >> logic getting confused about the appropriate interrupt host, and in
> >> ics_rtas_host_match() it doesn't match "chrp,iic" and the xics takes
> >> over and tries to use RTAS to map the interrupt. RTAS then crashes the
> >> machine.
> >>
> >> This tweaks the "-> no map, getting parent" behavior to behave as
> >> before, and the machine boots.
> >
> > I think the real question is why doesn't of_irq_find_parent return the
> > interrupt controller node instead of the bus node.
> >
> > While I think this fix is correct, I think of_irq_parse_raw should be
> > a nop in the case of no interrupt-map.
> >
> > Rob
> >
> >>
> >> Thanks
> >>
> >> Signed-off-by: Jeremy Linton <lintonrjeremy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> >>
> >>
> >> diff --git a/drivers/of/irq.c b/drivers/of/irq.c
> >> index 1a79806..78b4161b 100644
> >> --- a/drivers/of/irq.c
> >> +++ b/drivers/of/irq.c
> >> @@ -252,7 +252,6 @@ int of_irq_parse_raw(const __be32 *addr, struct
> >> of_phandle_args *out_irq)
> >> * Successfully parsed an interrrupt-map translation; copy new
> >> * interrupt specifier into the out_irq structure
> >> */
> >> - out_irq->np = newpar;
> >>
> >> match_array = imap - newaddrsize - newintsize;
> >> for (i = 0; i < newintsize; i++)
> >> @@ -262,6 +261,7 @@ int of_irq_parse_raw(const __be32 *addr, struct
> >> of_phandle_args *out_irq)
> >>
> >> skiplevel:
> >> /* Iterate again with new parent */
> >> + out_irq->np = newpar;
> >> pr_debug(" -> new parent: %s\n", of_node_full_name(newpar));
> >> of_node_put(ipar);
> >> ipar = newpar;
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: pSeries boot failure due to wrong interrupt controller
@ 2015-06-26 21:54 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 10+ messages in thread
From: Benjamin Herrenschmidt @ 2015-06-26 21:54 UTC (permalink / raw)
To: Jeremy Linton
Cc: Rob Herring, Jeremy Linton, Grant Likely, devicetree,
Paul Mackerras, linuxppc-dev
On Fri, 2015-06-26 at 12:54 -0500, Jeremy Linton wrote:
> Ping? Anyone?
>
> I'm adding ppc-dev to see if anyone there is interested in picking up the patch..
Ah I missed that completely. From your discussion, it looks correct,
I'll see if I can find a machine to test, however I think we dropped
support for POWER3 so it might be non-trivial to do so.
And yes, when there is no map, the parent should be the interrupt-parent
with a fallback to the parent node if that doesn't exist.
Cheers,
Ben.
> Thanks,
> Jeremy
>
>
> Ok so I messed up the formatting a little and was lazy and didn't checkpatch it
> cause I mean, it was 1 line! If someone wants it resent I will do so.
>
>
>
>
>
> On 6/23/2015 8:42 AM, Rob Herring wrote:
> > +Grant
> >
> > On Mon, Jun 22, 2015 at 2:44 PM, Jeremy Linton <lintonrjeremy@gmail.com> wrote:
> >> The OF IRQ logic was refactored a while back, and it appears that it
> >> broke the case where a device doesn't have an interrupt-map
> >> (legacy/ISA platform peripherals). In particular on the older pSeries
> >> platforms the console serial port (keyboard/etc) is attached via a
> >> Winbond super I/O controller. With the refactored logic
> >> of_irq_parse_raw() returns with the out_irq->np pointing at the bus,
> >> rather than the interrupt controller. This results in the platform
> >> logic getting confused about the appropriate interrupt host, and in
> >> ics_rtas_host_match() it doesn't match "chrp,iic" and the xics takes
> >> over and tries to use RTAS to map the interrupt. RTAS then crashes the
> >> machine.
> >>
> >> This tweaks the "-> no map, getting parent" behavior to behave as
> >> before, and the machine boots.
> >
> > I think the real question is why doesn't of_irq_find_parent return the
> > interrupt controller node instead of the bus node.
> >
> > While I think this fix is correct, I think of_irq_parse_raw should be
> > a nop in the case of no interrupt-map.
> >
> > Rob
> >
> >>
> >> Thanks
> >>
> >> Signed-off-by: Jeremy Linton <lintonrjeremy@gmail.com>
> >>
> >>
> >> diff --git a/drivers/of/irq.c b/drivers/of/irq.c
> >> index 1a79806..78b4161b 100644
> >> --- a/drivers/of/irq.c
> >> +++ b/drivers/of/irq.c
> >> @@ -252,7 +252,6 @@ int of_irq_parse_raw(const __be32 *addr, struct
> >> of_phandle_args *out_irq)
> >> * Successfully parsed an interrrupt-map translation; copy new
> >> * interrupt specifier into the out_irq structure
> >> */
> >> - out_irq->np = newpar;
> >>
> >> match_array = imap - newaddrsize - newintsize;
> >> for (i = 0; i < newintsize; i++)
> >> @@ -262,6 +261,7 @@ int of_irq_parse_raw(const __be32 *addr, struct
> >> of_phandle_args *out_irq)
> >>
> >> skiplevel:
> >> /* Iterate again with new parent */
> >> + out_irq->np = newpar;
> >> pr_debug(" -> new parent: %s\n", of_node_full_name(newpar));
> >> of_node_put(ipar);
> >> ipar = newpar;
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: pSeries boot failure due to wrong interrupt controller
2015-06-26 21:54 ` Benjamin Herrenschmidt
@ 2015-06-26 22:35 ` Jeremy Linton
-1 siblings, 0 replies; 10+ messages in thread
From: Jeremy Linton @ 2015-06-26 22:35 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: Rob Herring, Jeremy Linton, Grant Likely,
devicetree-u79uwXL29TY76Z2rM5mHXA, Paul Mackerras,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 6/26/2015 4:54 PM, Benjamin Herrenschmidt wrote:
> Ah I missed that completely. From your discussion, it looks correct, I'll
> see if I can find a machine to test, however I think we dropped support for
> POWER3 so it might be non-trivial to do so.
Thanks, the problem originally cropped up on a power4+, I was going to test
it on power5 too, but I don't have access anymore. I do have a power3 as well,
but I think that machine has other issues (the BUG_ON in the platform table
for starters).
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (MingW32)
iQEcBAEBAgAGBQJVjdPIAAoJEL5i86xrzcy7KA0H+QHVe3IFy12DlgxP0kbbaDtr
D5JLqh9OHBxZohotRzQz5errCIBYGxIYy93P6S2NvQSKS1gqZFFrvdyIs0f3UYo0
izzRVh/Gf10p0d3MJDI0EYC7OlxlP4hJ8lZKU2PNFQvhLT4eE1i/XJAySJLJsSpG
WD9CiP5Flixm/ECBPbyeg9nnc5c+EM4xPhE6J3Zp5pX7gU0cieJb2rzhz/7/TsI1
6AYMOPC4JpL3aK5z4eKuVgzvuJ/MyCjHdwii0TpNBhhhyidq8PEDoH3Y61CIOszq
Uu5HAQoToqBZdKM7siiZY0LJVQVpJaze4zhjexqhGa/Ll0iquj3V13xZ5qjj1eA=
=oeSx
-----END PGP SIGNATURE-----
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: pSeries boot failure due to wrong interrupt controller
@ 2015-06-26 22:35 ` Jeremy Linton
0 siblings, 0 replies; 10+ messages in thread
From: Jeremy Linton @ 2015-06-26 22:35 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: Rob Herring, Jeremy Linton, Grant Likely, devicetree,
Paul Mackerras, linuxppc-dev
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 6/26/2015 4:54 PM, Benjamin Herrenschmidt wrote:
> Ah I missed that completely. From your discussion, it looks correct, I'll
> see if I can find a machine to test, however I think we dropped support for
> POWER3 so it might be non-trivial to do so.
Thanks, the problem originally cropped up on a power4+, I was going to test
it on power5 too, but I don't have access anymore. I do have a power3 as well,
but I think that machine has other issues (the BUG_ON in the platform table
for starters).
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (MingW32)
iQEcBAEBAgAGBQJVjdPIAAoJEL5i86xrzcy7KA0H+QHVe3IFy12DlgxP0kbbaDtr
D5JLqh9OHBxZohotRzQz5errCIBYGxIYy93P6S2NvQSKS1gqZFFrvdyIs0f3UYo0
izzRVh/Gf10p0d3MJDI0EYC7OlxlP4hJ8lZKU2PNFQvhLT4eE1i/XJAySJLJsSpG
WD9CiP5Flixm/ECBPbyeg9nnc5c+EM4xPhE6J3Zp5pX7gU0cieJb2rzhz/7/TsI1
6AYMOPC4JpL3aK5z4eKuVgzvuJ/MyCjHdwii0TpNBhhhyidq8PEDoH3Y61CIOszq
Uu5HAQoToqBZdKM7siiZY0LJVQVpJaze4zhjexqhGa/Ll0iquj3V13xZ5qjj1eA=
=oeSx
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: pSeries boot failure due to wrong interrupt controller
2015-06-26 22:35 ` Jeremy Linton
@ 2015-06-26 22:38 ` Benjamin Herrenschmidt
-1 siblings, 0 replies; 10+ messages in thread
From: Benjamin Herrenschmidt @ 2015-06-26 22:38 UTC (permalink / raw)
To: Jeremy Linton
Cc: Rob Herring, Jeremy Linton, Grant Likely,
devicetree-u79uwXL29TY76Z2rM5mHXA, Paul Mackerras,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ
On Fri, 2015-06-26 at 17:35 -0500, Jeremy Linton wrote:
> On 6/26/2015 4:54 PM, Benjamin Herrenschmidt wrote:
> > Ah I missed that completely. From your discussion, it looks correct, I'll
> > see if I can find a machine to test, however I think we dropped support for
> > POWER3 so it might be non-trivial to do so.
>
> Thanks, the problem originally cropped up on a power4+, I was going to test
> it on power5 too, but I don't have access anymore. I do have a power3 as well,
> but I think that machine has other issues (the BUG_ON in the platform table
> for starters).
Ok, I don't think I still have one of these, but if you re-send the
patch with S-O-B etc... I'll review & ack it.
Cheers,
Ben.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: pSeries boot failure due to wrong interrupt controller
@ 2015-06-26 22:38 ` Benjamin Herrenschmidt
0 siblings, 0 replies; 10+ messages in thread
From: Benjamin Herrenschmidt @ 2015-06-26 22:38 UTC (permalink / raw)
To: Jeremy Linton
Cc: Rob Herring, Jeremy Linton, Grant Likely, devicetree,
Paul Mackerras, linuxppc-dev
On Fri, 2015-06-26 at 17:35 -0500, Jeremy Linton wrote:
> On 6/26/2015 4:54 PM, Benjamin Herrenschmidt wrote:
> > Ah I missed that completely. From your discussion, it looks correct, I'll
> > see if I can find a machine to test, however I think we dropped support for
> > POWER3 so it might be non-trivial to do so.
>
> Thanks, the problem originally cropped up on a power4+, I was going to test
> it on power5 too, but I don't have access anymore. I do have a power3 as well,
> but I think that machine has other issues (the BUG_ON in the platform table
> for starters).
Ok, I don't think I still have one of these, but if you re-send the
patch with S-O-B etc... I'll review & ack it.
Cheers,
Ben.
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2015-06-26 22:38 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-22 19:44 pSeries boot failure due to wrong interrupt controller Jeremy Linton
[not found] ` <CAEFTgiyHkf8ooYdWW0nTLcdraJ_L9FpLSVZa4KS3CcdpGWJxew-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-06-23 13:42 ` Rob Herring
[not found] ` <CAL_JsqJqkUL8XyqR6yjg+-thj4YOQ6cUohGHHid71tdRFft7vw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-06-26 17:54 ` Jeremy Linton
2015-06-26 17:54 ` Jeremy Linton
[not found] ` <558D91C1.6030004-WMdfZNuR64nqlBn2x/YWAg@public.gmane.org>
2015-06-26 21:54 ` Benjamin Herrenschmidt
2015-06-26 21:54 ` Benjamin Herrenschmidt
[not found] ` <1435355687.26815.8.camel-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2015-06-26 22:35 ` Jeremy Linton
2015-06-26 22:35 ` Jeremy Linton
[not found] ` <558DD3C8.6000800-WMdfZNuR64nqlBn2x/YWAg@public.gmane.org>
2015-06-26 22:38 ` Benjamin Herrenschmidt
2015-06-26 22:38 ` Benjamin Herrenschmidt
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.