Re: [PATCH v3 1/3] drm/nouveau: support for probing platform devices

[Martin Peres <martin.peres-GANU6spQydw@public.gmane.org>]

Le 26/06/2014 17:18, Alexandre Courbot a écrit :
> On Fri, Jun 27, 2014 at 12:10 AM, Martin Peres <martin.peres@free.fr> wrote:
>> Le 26/06/2014 16:58, Alexandre Courbot a écrit :
>>
>>> On Thu, Jun 26, 2014 at 6:30 PM, Roy Spliet <seven@nimrod-online.com>
>>> wrote:
>>>>
>>>> op 26-06-14 07:33, Alexandre Courbot schreef:
>>>>
>>>>> Add a platform driver for Nouveau devices declared using the device tree
>>>>> or platform data. This driver currently supports GK20A on Tegra
>>>>> platforms and is only compiled for these platforms if Nouveau is
>>>>> enabled.
>>>>>
>>>>> Nouveau will probe the chip type itself using the BOOT0 register, so all
>>>>> this driver really needs to do is to make sure the module is powered and
>>>>> its clocks active before calling nouveau_drm_platform_probe().
>>>>>
>>>>> Heavily based on work done by Thierry Reding.
>>>>>
>>>>> Signed-off-by: Thierry Reding <treding@nvidia.com>
>>>>> Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
>>>>> ---
>>>>>     drivers/gpu/drm/nouveau/Kconfig            |   8 ++
>>>>>     drivers/gpu/drm/nouveau/Makefile           |   3 +
>>>>>     drivers/gpu/drm/nouveau/nouveau_drm.c      |  53 ++++++---
>>>>>     drivers/gpu/drm/nouveau/nouveau_drm.h      |   8 ++
>>>>>     drivers/gpu/drm/nouveau/nouveau_platform.c | 182
>>>>> +++++++++++++++++++++++++++++
>>>>>     drivers/gpu/drm/nouveau/nouveau_platform.h |  49 ++++++++
>>>>>     6 files changed, 289 insertions(+), 14 deletions(-)
>>>>>     create mode 100644 drivers/gpu/drm/nouveau/nouveau_platform.c
>>>>>     create mode 100644 drivers/gpu/drm/nouveau/nouveau_platform.h
>>>>>
>>>>> diff --git a/drivers/gpu/drm/nouveau/Kconfig
>>>>> b/drivers/gpu/drm/nouveau/Kconfig
>>>>> index 637c29a33127..d4abaebfc35b 100644
>>>>> --- a/drivers/gpu/drm/nouveau/Kconfig
>>>>> +++ b/drivers/gpu/drm/nouveau/Kconfig
>>>>> @@ -25,6 +25,14 @@ config DRM_NOUVEAU
>>>>>           help
>>>>>             Choose this option for open-source nVidia support.
>>>>>     +config NOUVEAU_PLATFORM_DRIVER
>>>>> +       tristate "Nouveau (nVidia) integrated GPUs"
>>>>
>>>>
>>>> Maybe a little nit, but isn't the recommended capitalisation nowadays
>>>> NVIDIA
>>>> instead of nVidia?
>>>
>>>
>>> That's correct, I just copied that text from another Kconfig entry.
>>> This capitalization is also used elsewhere in Nouveau, so please allow
>>> me to fix this one in a separate patch. ;)
>>>
>>>> Also, integrated GPUs sounds like this is required for
>>>> the ION IGPs as well, although I reckon the dependencies on the next line
>>>> will hide it from the reader on x86.
>>>
>>>
>>> I don't know what word could better describe GK20A - we need to make
>>> the distinction because you can also use discrete GPUs on Tegra. Any
>>> better suggestion?
>>
>>
>> SoCs?
>
> SoC is the whole chip package, into which the GPU is, indeed, "integrated".
>

I meant replacing "integrated" with SoC, that would result in "NVIDIA 
SoCs' GPU" which could be appropriate.

Otherwise, what's wrong with using the name Tegra? NVIDIA Tegra GPUs 
seems perfectly fine, isn't it?

Don't mean to bikeshed on this, take these as suggestions. I'll be happy 
with whatever you decide.
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau