search for: nouveau_platform_driv

...e Courbot <acourbot at nvidia.com> --- Vince, with this you won't have to export new functions for PM. I ended up writing it myself because I hit the same issue (nv_device() would now need to be exported on Ben's master branch). Be careful that when compiling against a kernel, CONFIG_NOUVEAU_PLATFORM_DRIVER now *must* be equal to 'y'. It is 'm' currently, so you will need to change it from tristate to bool in the kernel's Kconfig. For this reason it might be good to get this patch into -next quickly. drm/Kbuild | 3 +-- drm/Kconfig | 2 +- drm/nouveau_d...

...dia.com> > --- > Vince, with this you won't have to export new functions for PM. I ended > up writing it myself because I hit the same issue (nv_device() would now > need to be exported on Ben's master branch). > > Be careful that when compiling against a kernel, CONFIG_NOUVEAU_PLATFORM_DRIVER > now *must* be equal to 'y'. It is 'm' currently, so you will need to change it > from tristate to bool in the kernel's Kconfig. For this reason it might be good > to get this patch into -next quickly. I've got a huge chunk of churn (finally finished the "re...

This patch is needed by initramfs tools to detect the required firmware files for the module. This patch tests for either TEGRA_124_SOC or TEGRA_132_SOC for the firmwares related to the Tegra K1 generation. v2: move the MODULE_FIRMWARE to the nvidia_platform.c file. This will avoid to test for NOUVEAU_PLATFORM_DRIVER Signed-off-by: Nicolas Chauvet <kwizart at gmail.com> --- drivers/gpu/drm/nouveau/nouveau_platform.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/drivers/gpu/drm/nouveau/nouveau_platform.c b/drivers/gpu/drm/nouveau/nouveau_platform.c index 3eb6654..9013141 100644 --- a...

This patch is needed by initramfs tools to detect the required firmware files for the module. This patch tests for NOUVEAU_PLATFORM_DRIVER and either TEGRA_124_SOC or TEGRA_132_SOC for the firmwares related to the Tegra K1 generation. Signed-off-by: Nicolas Chauvet <kwizart at gmail.com> --- drivers/gpu/drm/nouveau/nouveau_drm.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/drivers/gpu/drm/nouveau/nouveau_...

...rivers/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? Also, integrated GPUs sounds like this is required for the ION IGPs as well, although I reckon the dependencies on the next line will...

...quired firmware files for the module. > > > > This patch tests for either TEGRA_124_SOC or TEGRA_132_SOC > > for the firmwares related to the Tegra K1 generation. > > > > v2: move the MODULE_FIRMWARE to the nvidia_platform.c file. > > This will avoid to test for NOUVEAU_PLATFORM_DRIVER > > Nice, thanks for doing this change! > > Reviewed-by: Alexandre Courbot <acourbot at nvidia.com> > Thx for the review. FYI I've tested the patch on top of 4.3+ kernels, and the initramfs generated with dracut works as expected. Is there any other concern with this pa...

....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 cap...

...--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" + depends on DRM_NOUVEAU && ARCH_TEGRA + default y + help + Support for Nouveau platform driver, used for integrated GPUs as found + on NVIDIA Tegra K1. + config NOUVEAU_DEBUG int "Maximum debug level" depends on D...

This series adds support for probing platform devices on Nouveau, as well as the DT bindings for GK20A. It doesn't enable the GPU yet on Tegra boards since a few extra things need to be supported before that. This version is mostly identical to v2 but fixes an important issue: the drvdata must be set to the drm_device for sysfs to work, so the platform device structure now includes the

This series adds support for probing platform devices on Nouveau, as well as the DT bindings for GK20A. It doesn't enable the GPU yet on Tegra boards since a few extra things need to be supported before that. Thanks to the input received for v1, this version is more self-contained and shares less stuff between nouveau_drm and nouveau_platform. The major change is that nouveau_platform is now

This patch series is the final (?) step towards the initial support of GK20A, allowing it to be probed and used (currently at a very slow speed, and for offscreen rendering only) on the Jetson TK1 and Venice 2 boards. The main piece if the first patch which adds platform devices probing support to Nouveau. There are probably lots of things that need to be discussed about it, e.g.: * The way the

...fs tools to detect > the required firmware files for the module. > > This patch tests for either TEGRA_124_SOC or TEGRA_132_SOC > for the firmwares related to the Tegra K1 generation. > > v2: move the MODULE_FIRMWARE to the nvidia_platform.c file. > This will avoid to test for NOUVEAU_PLATFORM_DRIVER Nice, thanks for doing this change! Reviewed-by: Alexandre Courbot <acourbot at nvidia.com>

...t nouveau_platform_remove(struct platform_device *pdev) +static void nouveau_platform_remove(struct platform_device *pdev) { struct drm_device *dev = platform_get_drvdata(pdev); nouveau_drm_device_remove(dev); - return 0; } #if IS_ENABLED(CONFIG_OF) @@ -93,5 +92,5 @@ struct platform_driver nouveau_platform_driver = { .of_match_table = of_match_ptr(nouveau_platform_match), }, .probe = nouveau_platform_probe, - .remove = nouveau_platform_remove, + .remove_new = nouveau_platform_remove, }; -- 2.39.2

...gt;> 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 elsew...

...;>>> --- 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...

...hutdown(struct platform_device *pdev) > +{ > + nouveau_drm_device_shutdown(platform_get_drvdata(pdev)); > +} > + > #if IS_ENABLED(CONFIG_OF) > static const struct nvkm_device_tegra_func gk20a_platform_data = { > .iommu_bit = 34, > @@ -94,4 +99,5 @@ struct platform_driver nouveau_platform_driver = { > }, > .probe = nouveau_platform_probe, > .remove = nouveau_platform_remove, > + .shutdown = nouveau_platform_shutdown, > }; -- Cheers, Lyude Paul (she/her) Software Engineer at Red Hat

Hi, This series includes some pieces of fixes to complete the GK20A power on/off sequences and add the suspend/resume support. The patches 1/11 - 4/11 are based on the linux-next-20141219. The patches 5/11 - 11/11 are based on the branch "linux-3.19" of Ben Skeggs's tree (http://cgit.freedesktop.org/~darktama/nouveau). Thanks, Vince Vince Hsu (4): (linux-next-20141219) ARM:

On 23/12/14 10:40, Vince Hsu wrote: > This patch adds some checks in the suspend/resume functions to distinguish > the dGPU and mobile GPU and exports some variables/functions so that the > nouveau platform device can reuse them. > Hi Vince, Afaiu one needs to export a symbol as it's used by another module or subsystem. With the follow up two patches you are not doing either one,

...-- diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig index 40afc69a3778..5ab13e7939db 100644 --- a/drivers/gpu/drm/nouveau/Kconfig +++ b/drivers/gpu/drm/nouveau/Kconfig @@ -26,7 +26,7 @@ config DRM_NOUVEAU Choose this option for open-source NVIDIA support. config NOUVEAU_PLATFORM_DRIVER - tristate "Nouveau (NVIDIA) SoC GPUs" + bool "Nouveau (NVIDIA) SoC GPUs" depends on DRM_NOUVEAU && ARCH_TEGRA default y help diff --git a/drivers/gpu/drm/nouveau/Makefile b/drivers/gpu/drm/nouveau/Makefile index ea40cd653c7c..13acf90c11c8 100644 --- a/drivers/gp...

These 4 patches fix two issues that existed on Tegra regarding DMA: 1) The bit indicating whether to use an IOMMU or not was hardcoded ; make this a platform property and use it in instmem 2) The DMA mask was not set for platform devices. Fix this by converting more pci_dma* to the DMA API, and use that more generic code to set the DMA mask properly for all platforms. Tested on both x86