Martin Peres
2014-Jun-26 15:10 UTC
[Nouveau] [PATCH v3 1/3] drm/nouveau: support for probing platform devices
Le 26/06/2014 16:58, Alexandre Courbot a ?crit :> On Thu, Jun 26, 2014 at 6:30 PM, Roy Spliet <seven at 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 at nvidia.com> >>> Signed-off-by: Alexandre Courbot <acourbot at 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?
Alexandre Courbot
2014-Jun-26 15:18 UTC
[Nouveau] [PATCH v3 1/3] drm/nouveau: support for probing platform devices
On Fri, Jun 27, 2014 at 12:10 AM, Martin Peres <martin.peres at 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 at 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 at nvidia.com> >>>> Signed-off-by: Alexandre Courbot <acourbot at 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".
Martin Peres
2014-Jun-26 16:31 UTC
[Nouveau] [PATCH v3 1/3] drm/nouveau: support for probing platform devices
Le 26/06/2014 17:18, Alexandre Courbot a ?crit :> On Fri, Jun 27, 2014 at 12:10 AM, Martin Peres <martin.peres at 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 at 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 at nvidia.com> >>>>> Signed-off-by: Alexandre Courbot <acourbot at 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.
Apparently Analagous Threads
- [PATCH v3 1/3] drm/nouveau: support for probing platform devices
- [PATCH v3 1/3] drm/nouveau: support for probing platform devices
- [PATCH v3 1/3] drm/nouveau: support for probing platform devices
- [PATCH v3 1/3] drm/nouveau: support for probing platform devices
- [PATCH v3 0/3] drm/nouveau: support for probing platform devices