Nouveau - Jun 2014 - [PATCH v3 0/3] drm/nouveau: support for probing platform devices

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 nouveau_device flattened into it to let us compute
the address of one from the other. Since the platform device resources (clocks,
regulators, ...) need to live longer than the nouveau_device, they are stored
into their own structure which is allocated separately.

Changes since v2:
* Allocate the nouveau_device flattened into the nouveau_platform_device to
  be able to compute one from another easily, without having to ressort on the
  drvdata which is required to store the drm_device.
* Only propose the platform driver if a supported arch (Tegra for now) is
  enabled.
* Move structure definitions into their own header file as some subdevs (e.g.
  clocks) will need to use them.

Changes since v1:
* Moved the platform device driver to its own module. This allows it to be more
  self-contained and saves us the need to export too many functions from
  nouveau_drm and nouveau_platform.
* Register the DRM device without using the platform helpers, which is made
  possible by drm_dev_set_unique(). This allows us to catch and register the
  DRM device during platform probe.
* Fixed the clock names in the DT bindings.
* Removed the patches enabling GK20A on Venice2 and Jetson TK1 as support
  is not complete yet.

Alexandre Courbot (2):
  drm/nouveau: support for probing platform devices
  ARM: tegra: of: add GK20A device tree binding

Thierry Reding (1):
  ARM: tegra: add GK20A GPU to Tegra124 DT

 .../devicetree/bindings/gpu/nvidia,gk20a.txt       |  43 +++++
 arch/arm/boot/dts/tegra124.dtsi                    |  15 ++
 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 ++++++
 8 files changed, 347 insertions(+), 14 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
 create mode 100644 drivers/gpu/drm/nouveau/nouveau_platform.c
 create mode 100644 drivers/gpu/drm/nouveau/nouveau_platform.h

-- 
2.0.0

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"
+	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 DRM_NOUVEAU
diff --git a/drivers/gpu/drm/nouveau/Makefile b/drivers/gpu/drm/nouveau/Makefile
index 8b307e143632..f55f0f34aef9 100644
--- a/drivers/gpu/drm/nouveau/Makefile
+++ b/drivers/gpu/drm/nouveau/Makefile
@@ -349,3 +349,6 @@ nouveau-$(CONFIG_DRM_NOUVEAU_BACKLIGHT) +=
nouveau_backlight.o
 nouveau-$(CONFIG_DEBUG_FS) += nouveau_debugfs.o
 
 obj-$(CONFIG_DRM_NOUVEAU)+= nouveau.o
+
+# platform driver
+obj-$(CONFIG_NOUVEAU_PLATFORM_DRIVER) += nouveau_platform.o
diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c
b/drivers/gpu/drm/nouveau/nouveau_drm.c
index ddd83756b9a2..78295261c350 100644
--- a/drivers/gpu/drm/nouveau/nouveau_drm.c
+++ b/drivers/gpu/drm/nouveau/nouveau_drm.c
@@ -494,10 +494,9 @@ nouveau_drm_unload(struct drm_device *dev)
 	return 0;
 }
 
-static void
-nouveau_drm_remove(struct pci_dev *pdev)
+void
+nouveau_drm_device_remove(struct drm_device *dev)
 {
-	struct drm_device *dev = pci_get_drvdata(pdev);
 	struct nouveau_drm *drm = nouveau_drm(dev);
 	struct nouveau_object *device;
 
@@ -508,6 +507,15 @@ nouveau_drm_remove(struct pci_dev *pdev)
 	nouveau_object_ref(NULL, &device);
 	nouveau_object_debug();
 }
+EXPORT_SYMBOL(nouveau_drm_device_remove);
+
+static void
+nouveau_drm_remove(struct pci_dev *pdev)
+{
+	struct drm_device *dev = pci_get_drvdata(pdev);
+
+	nouveau_drm_device_remove(dev);
+}
 
 static int
 nouveau_do_suspend(struct drm_device *dev, bool runtime)
@@ -1003,24 +1011,41 @@ nouveau_drm_pci_driver = {
 	.driver.pm = &nouveau_pm_ops,
 };
 
-int nouveau_drm_platform_probe(struct platform_device *pdev)
+struct drm_device *
+nouveau_platform_device_create_(struct platform_device *pdev, int size,
+				void **pobject)
 {
-	struct nouveau_device *device;
-	int ret;
+	struct drm_device *drm;
+	int err;
 
-	ret = nouveau_device_create(pdev, NOUVEAU_BUS_PLATFORM,
+	err = nouveau_device_create_(pdev, NOUVEAU_BUS_PLATFORM,
 				    nouveau_platform_name(pdev),
 				    dev_name(&pdev->dev), nouveau_config,
-				    nouveau_debug, &device);
-
-	ret = drm_platform_init(&driver, pdev);
-	if (ret) {
-		nouveau_object_ref(NULL, (struct nouveau_object **)&device);
-		return ret;
+				    nouveau_debug, size, pobject);
+	if (err)
+		return ERR_PTR(err);
+
+	drm = drm_dev_alloc(&driver, &pdev->dev);
+	if (!drm) {
+		err = -ENOMEM;
+		goto err_free;
 	}
 
-	return ret;
+	err = drm_dev_set_unique(drm, "%s", dev_name(&pdev->dev));
+	if (err < 0)
+		goto err_free;
+
+	drm->platformdev = pdev;
+	platform_set_drvdata(pdev, drm);
+
+	return drm;
+
+err_free:
+	nouveau_object_ref(NULL, (struct nouveau_object **)pobject);
+
+	return ERR_PTR(err);
 }
+EXPORT_SYMBOL(nouveau_platform_device_create_);
 
 static int __init
 nouveau_drm_init(void)
diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.h
b/drivers/gpu/drm/nouveau/nouveau_drm.h
index 7efbafaf7c1d..dc342232182d 100644
--- a/drivers/gpu/drm/nouveau/nouveau_drm.h
+++ b/drivers/gpu/drm/nouveau/nouveau_drm.h
@@ -39,6 +39,7 @@
 #include <drm/ttm/ttm_page_alloc.h>
 
 struct nouveau_channel;
+struct platform_device;
 
 #define DRM_FILE_PAGE_OFFSET (0x100000000ULL >> PAGE_SHIFT)
 
@@ -157,6 +158,13 @@ nouveau_dev(struct drm_device *dev)
 int nouveau_pmops_suspend(struct device *);
 int nouveau_pmops_resume(struct device *);
 
+#define nouveau_platform_device_create(p, u)                                  
\
+	nouveau_platform_device_create_(p, sizeof(**u), (void **)u)
+struct drm_device *
+nouveau_platform_device_create_(struct platform_device *pdev,
+				int size, void **pobject);
+void nouveau_drm_device_remove(struct drm_device *dev);
+
 #define NV_FATAL(cli, fmt, args...) nv_fatal((cli), fmt, ##args)
 #define NV_ERROR(cli, fmt, args...) nv_error((cli), fmt, ##args)
 #define NV_WARN(cli, fmt, args...) nv_warn((cli), fmt, ##args)
diff --git a/drivers/gpu/drm/nouveau/nouveau_platform.c
b/drivers/gpu/drm/nouveau/nouveau_platform.c
new file mode 100644
index 000000000000..72ab0519abf6
--- /dev/null
+++ b/drivers/gpu/drm/nouveau/nouveau_platform.c
@@ -0,0 +1,182 @@
+/*
+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the
"Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ */
+
+#include <linux/clk.h>
+#include <linux/io.h>
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/of.h>
+#include <linux/reset.h>
+#include <linux/regulator/consumer.h>
+#include <linux/tegra-powergate.h>
+
+#include "nouveau_drm.h"
+#include "nouveau_platform.h"
+
+static int nouveau_platform_power_up(struct nouveau_platform_gpu *gpu)
+{
+	int err;
+
+	err = regulator_enable(gpu->vdd);
+	if (err)
+		goto err_power;
+
+	err = clk_prepare_enable(gpu->clk);
+	if (err)
+		goto err_clk;
+	err = clk_prepare_enable(gpu->clk_pwr);
+	if (err)
+		goto err_clk_pwr;
+	clk_set_rate(gpu->clk_pwr, 204000000);
+	udelay(10);
+
+	reset_control_assert(gpu->rst);
+	udelay(10);
+
+	err = tegra_powergate_remove_clamping(TEGRA_POWERGATE_3D);
+	if (err)
+		goto err_clamp;
+	udelay(10);
+
+	reset_control_deassert(gpu->rst);
+	udelay(10);
+
+	return 0;
+
+err_clamp:
+	clk_disable_unprepare(gpu->clk_pwr);
+err_clk_pwr:
+	clk_disable_unprepare(gpu->clk);
+err_clk:
+	regulator_disable(gpu->vdd);
+err_power:
+	return err;
+}
+
+static int nouveau_platform_power_down(struct nouveau_platform_gpu *gpu)
+{
+	int err;
+
+	reset_control_assert(gpu->rst);
+	udelay(10);
+
+	clk_disable_unprepare(gpu->clk_pwr);
+	clk_disable_unprepare(gpu->clk);
+	udelay(10);
+
+	err = regulator_disable(gpu->vdd);
+	if (err)
+		return err;
+
+	return 0;
+}
+
+static int nouveau_platform_probe(struct platform_device *pdev)
+{
+	struct nouveau_platform_gpu *gpu;
+	struct nouveau_platform_device *device;
+	struct drm_device *drm;
+	int err;
+
+	gpu = devm_kzalloc(&pdev->dev, sizeof(*gpu), GFP_KERNEL);
+	if (!gpu)
+		return -ENOMEM;
+
+	gpu->vdd = devm_regulator_get(&pdev->dev, "vdd");
+	if (IS_ERR(gpu->vdd))
+		return PTR_ERR(gpu->vdd);
+
+	gpu->rst = devm_reset_control_get(&pdev->dev, "gpu");
+	if (IS_ERR(gpu->rst))
+		return PTR_ERR(gpu->rst);
+
+	gpu->clk = devm_clk_get(&pdev->dev, "gpu");
+	if (IS_ERR(gpu->clk))
+		return PTR_ERR(gpu->clk);
+
+	gpu->clk_pwr = devm_clk_get(&pdev->dev, "pwr");
+	if (IS_ERR(gpu->clk_pwr))
+		return PTR_ERR(gpu->clk_pwr);
+
+	err = nouveau_platform_power_up(gpu);
+	if (err)
+		return err;
+
+	drm = nouveau_platform_device_create(pdev, &device);
+	if (IS_ERR(drm)) {
+		err = PTR_ERR(drm);
+		goto power_down;
+	}
+
+	device->gpu = gpu;
+
+	err = drm_dev_register(drm, 0);
+	if (err < 0)
+		goto err_unref;
+
+	return 0;
+
+err_unref:
+	drm_dev_unref(drm);
+
+	return 0;
+
+power_down:
+	nouveau_platform_power_down(gpu);
+
+	return err;
+}
+
+static int nouveau_platform_remove(struct platform_device *pdev)
+{
+	struct drm_device *drm_dev = platform_get_drvdata(pdev);
+	struct nouveau_device *device = nouveau_dev(drm_dev);
+	struct nouveau_platform_gpu *gpu = nv_device_to_platform(device)->gpu;
+
+	nouveau_drm_device_remove(drm_dev);
+
+	return nouveau_platform_power_down(gpu);
+}
+
+#if IS_ENABLED(CONFIG_OF)
+static const struct of_device_id nouveau_platform_match[] = {
+	{ .compatible = "nvidia,gk20a" },
+	{ }
+};
+
+MODULE_DEVICE_TABLE(of, nouveau_platform_match);
+#endif
+
+struct platform_driver nouveau_platform_driver = {
+	.driver = {
+		.name = "nouveau",
+		.of_match_table = of_match_ptr(nouveau_platform_match),
+	},
+	.probe = nouveau_platform_probe,
+	.remove = nouveau_platform_remove,
+};
+
+module_platform_driver(nouveau_platform_driver);
+
+MODULE_AUTHOR(DRIVER_AUTHOR);
+MODULE_DESCRIPTION(DRIVER_DESC);
+MODULE_LICENSE("GPL and additional rights");
diff --git a/drivers/gpu/drm/nouveau/nouveau_platform.h
b/drivers/gpu/drm/nouveau/nouveau_platform.h
new file mode 100644
index 000000000000..91f66504900e
--- /dev/null
+++ b/drivers/gpu/drm/nouveau/nouveau_platform.h
@@ -0,0 +1,49 @@
+/*
+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the
"Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ */
+
+#ifndef __NOUVEAU_PLATFORM_H__
+#define __NOUVEAU_PLATFORM_H__
+
+#include "core/device.h"
+
+struct reset_control;
+struct clk;
+struct regulator;
+
+struct nouveau_platform_gpu {
+	struct reset_control *rst;
+	struct clk *clk;
+	struct clk *clk_pwr;
+
+	struct regulator *vdd;
+};
+
+struct nouveau_platform_device {
+	struct nouveau_device device;
+
+	struct nouveau_platform_gpu *gpu;
+};
+
+#define nv_device_to_platform(d)                                              
\
+	container_of(d, struct nouveau_platform_device, device)
+
+#endif
-- 
2.0.0

Add the device tree binding documentation for the GK20A GPU used in
Tegra K1 SoCs.

Signed-off-by: Alexandre Courbot <acourbot at nvidia.com>
Acked-by: Stephen Warren <swarren at nvidia.com>
---
 .../devicetree/bindings/gpu/nvidia,gk20a.txt       | 43 ++++++++++++++++++++++
 1 file changed, 43 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt

diff --git a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
new file mode 100644
index 000000000000..23bfe8e1f7cc
--- /dev/null
+++ b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
@@ -0,0 +1,43 @@
+NVIDIA GK20A Graphics Processing Unit
+
+Required properties:
+- compatible: "nvidia,<chip>-<gpu>"
+  Currently recognized values:
+  - nvidia,tegra124-gk20a
+- reg: Physical base address and length of the controller's registers.
+  Must contain two entries:
+  - first entry for bar0
+  - second entry for bar1
+- interrupts: Must contain an entry for each entry in interrupt-names.
+  See ../interrupt-controller/interrupts.txt for details.
+- interrupt-names: Must include the following entries:
+  - stall
+  - nonstall
+- vdd-supply: regulator for supply voltage.
+- clocks: Must contain an entry for each entry in clock-names.
+  See ../clocks/clock-bindings.txt for details.
+- clock-names: Must include the following entries:
+  - gpu
+  - pwr
+- resets: Must contain an entry for each entry in reset-names.
+  See ../reset/reset.txt for details.
+- reset-names: Must include the following entries:
+  - gpu
+
+Example:
+
+	gpu at 0,57000000 {
+		compatible = "nvidia,gk20a";
+		reg = <0x0 0x57000000 0x0 0x01000000>,
+		      <0x0 0x58000000 0x0 0x01000000>;
+		interrupts = <GIC_SPI 157 IRQ_TYPE_LEVEL_HIGH>,
+			     <GIC_SPI 158 IRQ_TYPE_LEVEL_HIGH>;
+		interrupt-names = "stall", "nonstall";
+		vdd-supply = <&vdd_gpu>;
+		clocks = <&tegra_car TEGRA124_CLK_GPU>,
+			 <&tegra_car TEGRA124_CLK_PLL_P_OUT5>;
+		clock-names = "gpu", "pwr";
+		resets = <&tegra_car 184>;
+		reset-names = "gpu";
+		status = "disabled";
+	};
-- 
2.0.0

From: Thierry Reding <treding at nvidia.com>

Add the GK20A device node to Tegra124's device tree.

Signed-off-by: Thierry Reding <treding at nvidia.com>
Signed-off-by: Alexandre Courbot <acourbot at nvidia.com>
---
 arch/arm/boot/dts/tegra124.dtsi | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi
index d675186d8eba..cc9e7504c2da 100644
--- a/arch/arm/boot/dts/tegra124.dtsi
+++ b/arch/arm/boot/dts/tegra124.dtsi
@@ -102,6 +102,21 @@
 			(GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>;
 	};
 
+	gpu at 0,57000000 {
+		compatible = "nvidia,gk20a";
+		reg = <0x0 0x57000000 0x0 0x01000000>,
+		      <0x0 0x58000000 0x0 0x01000000>;
+		interrupts = <GIC_SPI 157 IRQ_TYPE_LEVEL_HIGH>,
+			     <GIC_SPI 158 IRQ_TYPE_LEVEL_HIGH>;
+		interrupt-names = "stall", "nonstall";
+		clocks = <&tegra_car TEGRA124_CLK_GPU>,
+			 <&tegra_car TEGRA124_CLK_PLL_P_OUT5>;
+		clock-names = "gpu", "pwr";
+		resets = <&tegra_car 184>;
+		reset-names = "gpu";
+		status = "disabled";
+	};
+
 	timer at 0,60005000 {
 		compatible = "nvidia,tegra124-timer",
"nvidia,tegra20-timer";
 		reg = <0x0 0x60005000 0x0 0x400>;
-- 
2.0.0

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? 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.
> +	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 DRM_NOUVEAU
> diff --git a/drivers/gpu/drm/nouveau/Makefile
b/drivers/gpu/drm/nouveau/Makefile
> index 8b307e143632..f55f0f34aef9 100644
> --- a/drivers/gpu/drm/nouveau/Makefile
> +++ b/drivers/gpu/drm/nouveau/Makefile
> @@ -349,3 +349,6 @@ nouveau-$(CONFIG_DRM_NOUVEAU_BACKLIGHT) +=
nouveau_backlight.o
>   nouveau-$(CONFIG_DEBUG_FS) += nouveau_debugfs.o
>   
>   obj-$(CONFIG_DRM_NOUVEAU)+= nouveau.o
> +
> +# platform driver
> +obj-$(CONFIG_NOUVEAU_PLATFORM_DRIVER) += nouveau_platform.o
> diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c
b/drivers/gpu/drm/nouveau/nouveau_drm.c
> index ddd83756b9a2..78295261c350 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_drm.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_drm.c
> @@ -494,10 +494,9 @@ nouveau_drm_unload(struct drm_device *dev)
>   	return 0;
>   }
>   
> -static void
> -nouveau_drm_remove(struct pci_dev *pdev)
> +void
> +nouveau_drm_device_remove(struct drm_device *dev)
>   {
> -	struct drm_device *dev = pci_get_drvdata(pdev);
>   	struct nouveau_drm *drm = nouveau_drm(dev);
>   	struct nouveau_object *device;
>   
> @@ -508,6 +507,15 @@ nouveau_drm_remove(struct pci_dev *pdev)
>   	nouveau_object_ref(NULL, &device);
>   	nouveau_object_debug();
>   }
> +EXPORT_SYMBOL(nouveau_drm_device_remove);
> +
> +static void
> +nouveau_drm_remove(struct pci_dev *pdev)
> +{
> +	struct drm_device *dev = pci_get_drvdata(pdev);
> +
> +	nouveau_drm_device_remove(dev);
> +}
>   
>   static int
>   nouveau_do_suspend(struct drm_device *dev, bool runtime)
> @@ -1003,24 +1011,41 @@ nouveau_drm_pci_driver = {
>   	.driver.pm = &nouveau_pm_ops,
>   };
>   
> -int nouveau_drm_platform_probe(struct platform_device *pdev)
> +struct drm_device *
> +nouveau_platform_device_create_(struct platform_device *pdev, int size,
> +				void **pobject)
>   {
> -	struct nouveau_device *device;
> -	int ret;
> +	struct drm_device *drm;
> +	int err;
>   
> -	ret = nouveau_device_create(pdev, NOUVEAU_BUS_PLATFORM,
> +	err = nouveau_device_create_(pdev, NOUVEAU_BUS_PLATFORM,
>   				    nouveau_platform_name(pdev),
>   				    dev_name(&pdev->dev), nouveau_config,
> -				    nouveau_debug, &device);
> -
> -	ret = drm_platform_init(&driver, pdev);
> -	if (ret) {
> -		nouveau_object_ref(NULL, (struct nouveau_object **)&device);
> -		return ret;
> +				    nouveau_debug, size, pobject);
> +	if (err)
> +		return ERR_PTR(err);
> +
> +	drm = drm_dev_alloc(&driver, &pdev->dev);
> +	if (!drm) {
> +		err = -ENOMEM;
> +		goto err_free;
>   	}
>   
> -	return ret;
> +	err = drm_dev_set_unique(drm, "%s",
dev_name(&pdev->dev));
> +	if (err < 0)
> +		goto err_free;
> +
> +	drm->platformdev = pdev;
> +	platform_set_drvdata(pdev, drm);
> +
> +	return drm;
> +
> +err_free:
> +	nouveau_object_ref(NULL, (struct nouveau_object **)pobject);
> +
> +	return ERR_PTR(err);
>   }
> +EXPORT_SYMBOL(nouveau_platform_device_create_);
>   
>   static int __init
>   nouveau_drm_init(void)
> diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.h
b/drivers/gpu/drm/nouveau/nouveau_drm.h
> index 7efbafaf7c1d..dc342232182d 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_drm.h
> +++ b/drivers/gpu/drm/nouveau/nouveau_drm.h
> @@ -39,6 +39,7 @@
>   #include <drm/ttm/ttm_page_alloc.h>
>   
>   struct nouveau_channel;
> +struct platform_device;
>   
>   #define DRM_FILE_PAGE_OFFSET (0x100000000ULL >> PAGE_SHIFT)
>   
> @@ -157,6 +158,13 @@ nouveau_dev(struct drm_device *dev)
>   int nouveau_pmops_suspend(struct device *);
>   int nouveau_pmops_resume(struct device *);
>   
> +#define nouveau_platform_device_create(p, u)                              
\
> +	nouveau_platform_device_create_(p, sizeof(**u), (void **)u)
> +struct drm_device *
> +nouveau_platform_device_create_(struct platform_device *pdev,
> +				int size, void **pobject);
> +void nouveau_drm_device_remove(struct drm_device *dev);
> +
>   #define NV_FATAL(cli, fmt, args...) nv_fatal((cli), fmt, ##args)
>   #define NV_ERROR(cli, fmt, args...) nv_error((cli), fmt, ##args)
>   #define NV_WARN(cli, fmt, args...) nv_warn((cli), fmt, ##args)
> diff --git a/drivers/gpu/drm/nouveau/nouveau_platform.c
b/drivers/gpu/drm/nouveau/nouveau_platform.c
> new file mode 100644
> index 000000000000..72ab0519abf6
> --- /dev/null
> +++ b/drivers/gpu/drm/nouveau/nouveau_platform.c
> @@ -0,0 +1,182 @@
> +/*
> + * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the
"Software"),
> + * to deal in the Software without restriction, including without
limitation
> + * the rights to use, copy, modify, merge, publish, distribute,
sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included
in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY
KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> + * DEALINGS IN THE SOFTWARE.
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/io.h>
> +#include <linux/module.h>
> +#include <linux/platform_device.h>
> +#include <linux/of.h>
> +#include <linux/reset.h>
> +#include <linux/regulator/consumer.h>
> +#include <linux/tegra-powergate.h>
> +
> +#include "nouveau_drm.h"
> +#include "nouveau_platform.h"
> +
> +static int nouveau_platform_power_up(struct nouveau_platform_gpu *gpu)
> +{
> +	int err;
> +
> +	err = regulator_enable(gpu->vdd);
> +	if (err)
> +		goto err_power;
> +
> +	err = clk_prepare_enable(gpu->clk);
> +	if (err)
> +		goto err_clk;
> +	err = clk_prepare_enable(gpu->clk_pwr);
> +	if (err)
> +		goto err_clk_pwr;
> +	clk_set_rate(gpu->clk_pwr, 204000000);
> +	udelay(10);
> +
> +	reset_control_assert(gpu->rst);
> +	udelay(10);
> +
> +	err = tegra_powergate_remove_clamping(TEGRA_POWERGATE_3D);
> +	if (err)
> +		goto err_clamp;
> +	udelay(10);
> +
> +	reset_control_deassert(gpu->rst);
> +	udelay(10);
> +
> +	return 0;
> +
> +err_clamp:
> +	clk_disable_unprepare(gpu->clk_pwr);
> +err_clk_pwr:
> +	clk_disable_unprepare(gpu->clk);
> +err_clk:
> +	regulator_disable(gpu->vdd);
> +err_power:
> +	return err;
> +}
> +
> +static int nouveau_platform_power_down(struct nouveau_platform_gpu *gpu)
> +{
> +	int err;
> +
> +	reset_control_assert(gpu->rst);
> +	udelay(10);
> +
> +	clk_disable_unprepare(gpu->clk_pwr);
> +	clk_disable_unprepare(gpu->clk);
> +	udelay(10);
> +
> +	err = regulator_disable(gpu->vdd);
> +	if (err)
> +		return err;
> +
> +	return 0;
> +}
> +
> +static int nouveau_platform_probe(struct platform_device *pdev)
> +{
> +	struct nouveau_platform_gpu *gpu;
> +	struct nouveau_platform_device *device;
> +	struct drm_device *drm;
> +	int err;
> +
> +	gpu = devm_kzalloc(&pdev->dev, sizeof(*gpu), GFP_KERNEL);
> +	if (!gpu)
> +		return -ENOMEM;
> +
> +	gpu->vdd = devm_regulator_get(&pdev->dev, "vdd");
> +	if (IS_ERR(gpu->vdd))
> +		return PTR_ERR(gpu->vdd);
> +
> +	gpu->rst = devm_reset_control_get(&pdev->dev, "gpu");
> +	if (IS_ERR(gpu->rst))
> +		return PTR_ERR(gpu->rst);
> +
> +	gpu->clk = devm_clk_get(&pdev->dev, "gpu");
> +	if (IS_ERR(gpu->clk))
> +		return PTR_ERR(gpu->clk);
> +
> +	gpu->clk_pwr = devm_clk_get(&pdev->dev, "pwr");
> +	if (IS_ERR(gpu->clk_pwr))
> +		return PTR_ERR(gpu->clk_pwr);
> +
> +	err = nouveau_platform_power_up(gpu);
> +	if (err)
> +		return err;
> +
> +	drm = nouveau_platform_device_create(pdev, &device);
> +	if (IS_ERR(drm)) {
> +		err = PTR_ERR(drm);
> +		goto power_down;
> +	}
> +
> +	device->gpu = gpu;
> +
> +	err = drm_dev_register(drm, 0);
> +	if (err < 0)
> +		goto err_unref;
> +
> +	return 0;
> +
> +err_unref:
> +	drm_dev_unref(drm);
> +
> +	return 0;
> +
> +power_down:
> +	nouveau_platform_power_down(gpu);
> +
> +	return err;
> +}
> +
> +static int nouveau_platform_remove(struct platform_device *pdev)
> +{
> +	struct drm_device *drm_dev = platform_get_drvdata(pdev);
> +	struct nouveau_device *device = nouveau_dev(drm_dev);
> +	struct nouveau_platform_gpu *gpu = nv_device_to_platform(device)->gpu;
> +
> +	nouveau_drm_device_remove(drm_dev);
> +
> +	return nouveau_platform_power_down(gpu);
> +}
> +
> +#if IS_ENABLED(CONFIG_OF)
> +static const struct of_device_id nouveau_platform_match[] = {
> +	{ .compatible = "nvidia,gk20a" },
> +	{ }
> +};
> +
> +MODULE_DEVICE_TABLE(of, nouveau_platform_match);
> +#endif
> +
> +struct platform_driver nouveau_platform_driver = {
> +	.driver = {
> +		.name = "nouveau",
> +		.of_match_table = of_match_ptr(nouveau_platform_match),
> +	},
> +	.probe = nouveau_platform_probe,
> +	.remove = nouveau_platform_remove,
> +};
> +
> +module_platform_driver(nouveau_platform_driver);
> +
> +MODULE_AUTHOR(DRIVER_AUTHOR);
> +MODULE_DESCRIPTION(DRIVER_DESC);
> +MODULE_LICENSE("GPL and additional rights");
> diff --git a/drivers/gpu/drm/nouveau/nouveau_platform.h
b/drivers/gpu/drm/nouveau/nouveau_platform.h
> new file mode 100644
> index 000000000000..91f66504900e
> --- /dev/null
> +++ b/drivers/gpu/drm/nouveau/nouveau_platform.h
> @@ -0,0 +1,49 @@
> +/*
> + * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the
"Software"),
> + * to deal in the Software without restriction, including without
limitation
> + * the rights to use, copy, modify, merge, publish, distribute,
sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included
in
> + * all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY
KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> + * DEALINGS IN THE SOFTWARE.
> + */
> +
> +#ifndef __NOUVEAU_PLATFORM_H__
> +#define __NOUVEAU_PLATFORM_H__
> +
> +#include "core/device.h"
> +
> +struct reset_control;
> +struct clk;
> +struct regulator;
> +
> +struct nouveau_platform_gpu {
> +	struct reset_control *rst;
> +	struct clk *clk;
> +	struct clk *clk_pwr;
> +
> +	struct regulator *vdd;
> +};
> +
> +struct nouveau_platform_device {
> +	struct nouveau_device device;
> +
> +	struct nouveau_platform_gpu *gpu;
> +};
> +
> +#define nv_device_to_platform(d)                                          
\
> +	container_of(d, struct nouveau_platform_device, device)
> +
> +#endif

On Thu, Jun 26, 2014 at 2:33 PM, Alexandre Courbot <acourbot at
nvidia.com> wrote:
> 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 nouveau_device flattened into it to let us
compute
> the address of one from the other. Since the platform device resources
(clocks,
> regulators, ...) need to live longer than the nouveau_device, they are
stored
> into their own structure which is allocated separately.
>
> Changes since v2:
> * Allocate the nouveau_device flattened into the nouveau_platform_device to
>   be able to compute one from another easily, without having to ressort on
the
>   drvdata which is required to store the drm_device.
> * Only propose the platform driver if a supported arch (Tegra for now) is
>   enabled.
> * Move structure definitions into their own header file as some subdevs
(e.g.
>   clocks) will need to use them.
>
> Changes since v1:
> * Moved the platform device driver to its own module. This allows it to be
more
>   self-contained and saves us the need to export too many functions from
>   nouveau_drm and nouveau_platform.
> * Register the DRM device without using the platform helpers, which is made
>   possible by drm_dev_set_unique(). This allows us to catch and register
the
>   DRM device during platform probe.
> * Fixed the clock names in the DT bindings.
> * Removed the patches enabling GK20A on Venice2 and Jetson TK1 as support
>   is not complete yet.
Stephen, Thierry,

Ben has merged the first patch of this series into the Nouveau tree.
Could you get patches 2 and 3 through the Tegra tree? Thanks!

On 07/02/2014 03:09 AM, Alexandre Courbot wrote:
> On Thu, Jun 26, 2014 at 2:33 PM, Alexandre Courbot <acourbot at
nvidia.com> wrote:
>> 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 nouveau_device flattened into it to let us
compute
>> the address of one from the other. Since the platform device resources
(clocks,
>> regulators, ...) need to live longer than the nouveau_device, they are
stored
>> into their own structure which is allocated separately.
...
> Stephen, Thierry,
> 
> Ben has merged the first patch of this series into the Nouveau tree.
> Could you get patches 2 and 3 through the Tegra tree? Thanks!
I've applied patches 2 and 3 to Tegra's for-3.17/dt branch.