Re: [PASEMI NEMO] Boot issue with the PowerPC updates 6.4-1

[Christian Zigotzky <chzigotzky@xenosoft.de>]

[-- Attachment #1: Type: text/plain, Size: 2652 bytes --]



> On 3. May 2023, at 19:28, Christian Zigotzky <chzigotzky@xenosoft.de> wrote:
> 
> 
> 
>>> On 3. May 2023, at 18:51, Rob Herring <robh@kernel.org> wrote:
>>> 
>>> On Wed, May 3, 2023 at 11:27 AM Christophe Leroy
>>> <christophe.leroy@csgroup.eu> wrote:
>>> 
>>> +Rob as he's the commit's Author.
>>> 
>>>> Le 03/05/2023 à 17:46, Christian Zigotzky a écrit :
>>>> On 02 May 2023 at 11:28 am, Michael Ellerman wrote:
>>>>> Christian Zigotzky <chzigotzky@xenosoft.de> writes:
>>>>>> Hello,
>>>>>> 
>>>>>> Our PASEMI Nemo board [1] doesn't boot with the PowerPC updates 6.4-1
>>>>>> [2].
>>>>>> 
>>>>>> The kernel hangs right after the booting Linux via __start() @
>>>>>> 0x0000000000000000 ...
>>>>>> 
>>>>>> I was able to revert the PowerPC updates 6.4-1 [2] with the following
>>>>>> command: git revert 70cc1b5307e8ee3076fdf2ecbeb89eb973aa0ff7 -m 1
>>>>>> 
>>>>>> After a re-compiling, the kernel boots without any problems without the
>>>>>> PowerPC updates 6.4-1 [2].
>>>>>> 
>>>>>> Could you please explain me, what you have done in the boot area?
>>>>> There's a few possibilities, but nothing obvious.
>>>>> 
>>>>> To begin with can you please test the following commits?
>>>>> 
>>>>> 77e69ee7ce07
>>>>> e4ab08be5b49
>>>>> eeac8ede1755
>>>>> 
>>>>> cheers
>>>> git revert e4ab08be5b4902e5b350b0e1e1a3c25eb21d76d4
>>>> 
>>>> [master 0086e2cbbec0] Revert "powerpc/isa-bridge: Remove open coded
>>>> "ranges" parsing"
>>>> 1 file changed, 129 insertions(+), 37 deletions(-)
>>>> 
>>>> After a recompiling it boots without any problems.
>>>> 
>>>> e4ab08be5b49 -- powerpc/isa-bridge: Remove open coded "ranges" parsing
>>>> is the bad commit.
>> 
>> Could I get a DT file for this board?
>> 
>> In the meantime, just revert this commit. I don't think I'll be able
>> to fix it before I'm out on sabbatical.
>> 
>> Rob
> 
> FYI:
> 
> Darren Stevens wrote:
> 
> The dtb passed by the CFE firmware has a number of issues, which up till
> now have been fixed by use of patches applied to the mainline kernel.
> This occasionally causes problems with changes made to mainline.
> 
> Patching the firmware to correct the dtb is not an option for the 
> following reasons:
> 
> It was modified by a 3rd party, and we don't have a copy of the source.
> 
> All versions of CFE used on the X1000 export the same dtb.
> 
> At least one machine suffered damage during a firmware upgrade attempt,
> many people will be unwilling to reflash their system if an upgrade is
> produced.
> 
> 

Nemo’s DT:  https://forum.hyperion-entertainment.com/viewtopic.php?p=54406#p54406

[-- Attachment #2: Type: text/html, Size: 10022 bytes --]