Re: [U-Boot] [PATCH 0/2] ARM: tegra: enable GPU DT node

[Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]

On 08/05/2015 08:24 AM, Tom Warren wrote:
> Alex/Andreas,
>
>> -----Original Message-----
>> From: Tom Warren
>> Sent: Tuesday, August 04, 2015 8:41 AM
>> To: 'Andreas Färber'; u-boot-0aAXYlwwYIKGBzrmiIFOJg@public.gmane.org
>> Cc: Alex Courbot; linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org; Stephen
>> Warren; Guillaume Gardet
>> Subject: RE: [U-Boot] [PATCH 0/2] ARM: tegra: enable GPU DT node
>>
>> Andreas,
>>
>>> -----Original Message-----
>>> From: Andreas Färber [mailto:afaerber-l3A5Bk7waGM@public.gmane.org]
>>> Sent: Tuesday, August 04, 2015 6:56 AM
>>> To: u-boot-0aAXYlwwYIKGBzrmiIFOJg@public.gmane.org
>>> Cc: Alex Courbot; linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org;
>>> Stephen Warren; Tom Warren; Guillaume Gardet
>>> Subject: Re: [U-Boot] [PATCH 0/2] ARM: tegra: enable GPU DT node
>>>
>>> Am 23.07.2015 um 13:46 schrieb Andreas Färber:
>>>> Am 09.07.2015 um 09:32 schrieb Alexandre Courbot:
>>>>> Tegra124 requires the bootloader to perform VPR initialization,
>>>>> otherwise the GPU cannot be used by the system. Since using the GPU
>>>>> without that initialization results in a hang, the GPU DT node is
>>>>> left disabled, and it is the task of the bootloader to enable it
>>>>> after ensuring it is safe to use the GPU.
>>>>>
>>>>> VPR init is already performed since patch df3443dfa449, but the
>>>>> device tree was left untouched. This patch series performs this
>>>>> last step and prepares the GPU intialization code to receive more
>>>>> code for
>>> newer chips.
>>>>
>>>> Tested-by: Andreas Färber <afaerber-l3A5Bk7waGM@public.gmane.org>
>>>>
>>>> I've tested this patchset on v2015.07 with 4.2.0-rc3-00115-gc5dfd65
>>>> - with these two patches I get a console login on HDMI again.
>>>
>>> Ping! Independent of the Linux and X11 discussions this thread has
>>> drifted off into, these two patches are still missing in v2015.10-rc1
>>> and don't apply any more (as reported by Guillaume). Can you please rebase
>> and merge them?
>> Is that request for me? Sorry, lost track of these patches since it appeared that
>> there was an on-going discussion.
>> If you're sure they're OK within the context of Tegra U-Boot, I'll apply them and
>> send them with the next PR.
>>
>> Tom
> Applied to u-boot-tegra/next (along with some other pending Tegra patches for clocks/PLL/SPI/ums/etc.). I added T210/P2571 support to this patchset.
>
> PTAL.  Also, there doesn't appear to be a 'gpu@0,57000000' property in any t124/t210 DT file. Is that coming in another patch or one I missed?

Thanks Tom! I have tried your branch and can confirm it is booting 
Jetson TK1 as expected (i.e. the GPU is in a usable state).

The GPU node is not needed in U-boot's DT files (if that's what you 
meant). It is present in the kernel though, and that's the DT my patches 
will modify.

Alex.