Skip to contentSkip to conversation

drm/nouveau: Document nouveau support for Tegra in DRIVER_DESC

PATCH Superseded Rhys Kidd 5 participants 6 messages Aug 11, 2017 – Oct 12, 2017 1 message not in the archive

Subsystems (3)
lore
Share & tools
lore

Rhys Kidd

author
#0 lore raw

PATCH 1/1 drm/nouveau: Document nouveau support for Tegra in DRIVER_DESC

nouveau supports the Tegra K1 and higher after the SoC-based GPUs converged with the main GeForce GPU families.

---
 drivers/gpu/drm/nouveau/nouveau_drv.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/gpu/drm/nouveau/nouveau_drv.h +1 -1
index 822fe1d4d35e..fbe42ec0a5f1 100644
@@ -5,7 +5,7 @@
 #define DRIVER_EMAIL		"[email protected]"
 
 #define DRIVER_NAME		"nouveau"
-#define DRIVER_DESC		"nVidia Riva/TNT/GeForce/Quadro/Tesla"
+#define DRIVER_DESC		"nVidia Riva/TNT/GeForce/Quadro/Tesla/Tegra"
 #define DRIVER_DATE		"20120801"
 
 #define DRIVER_MAJOR		1
Signature · 6 lines
2.11.0

_______________________________________________
Nouveau mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/nouveau
#1 lore raw
Quoted 13 lines · Rhys Kidd

nouveau supports the Tegra K1 and higher after the SoC-based GPUs converged with the main GeForce GPU families.

---
 drivers/gpu/drm/nouveau/nouveau_drv.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/gpu/drm/nouveau/nouveau_drv.h +0 -0
index 822fe1d4d35e..fbe42ec0a5f1 100644

drivers/gpu/drm/nouveau/nouveau_drv.h line 8

Quoted 5 lines
drivers/gpu/drm/nouveau/nouveau_drv.h +1 -1
@@ -5,7 +5,7 @@
 #define DRIVER_EMAIL		"[email protected]"
 
 #define DRIVER_NAME		"nouveau"
-#define DRIVER_DESC		"nVidia Riva/TNT/GeForce/Quadro/Tesla"
+#define DRIVER_DESC		"nVidia Riva/TNT/GeForce/Quadro/Tesla/Tegra"

The problem is that we do not support all Tegras. How about Tegra TK1+?

I really do not care much about this string, but I guess I probably should a bit.

With the TK1 added, this is:

Reviewed-by" Martin Peres <[email protected]>

drivers/gpu/drm/nouveau/nouveau_drv.h line 11

Quoted 4 lines
drivers/gpu/drm/nouveau/nouveau_drv.h +0 -0
 #define DRIVER_DATE		"20120801"
 
 #define DRIVER_MAJOR		1

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Acked-by on v1 1/1
#2 lore raw
In reply to Rhys Kidd

You are right, Tegra should probably be part of the description, even if we do not support pre-K1 Tegras.

Quoted 28 lines · Rhys Kidd

nouveau supports the Tegra K1 and higher after the SoC-based GPUs converged with the main GeForce GPU families.

---
 drivers/gpu/drm/nouveau/nouveau_drv.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/gpu/drm/nouveau/nouveau_drv.h +1 -1
index 822fe1d4d35e..fbe42ec0a5f1 100644
@@ -5,7 +5,7 @@
 #define DRIVER_EMAIL		"[email protected]"
 
 #define DRIVER_NAME		"nouveau"
-#define DRIVER_DESC		"nVidia Riva/TNT/GeForce/Quadro/Tesla"
+#define DRIVER_DESC		"nVidia Riva/TNT/GeForce/Quadro/Tesla/Tegra"
 #define DRIVER_DATE		"20120801"
 
 #define DRIVER_MAJOR		1
Signature · 6 lines
2.11.0

_______________________________________________
Nouveau mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/nouveau

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

A message that is not in the archive (sent off-list or not yet received): 20170811105616.9727-1-rhyskidd-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org

author
#4 lore raw

PATCH v2 1/1 drm/nouveau: Document nouveau support for Tegra in DRIVER_DESC

nouveau supports the Tegra K1 and higher after the SoC-based GPUs converged with the main GeForce GPU families.

v2:
- Qualify that support is Tegra K1+ (Martin Peres)

---
 drivers/gpu/drm/nouveau/nouveau_drv.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/gpu/drm/nouveau/nouveau_drv.h +1 -1
index 822fe1d4d35e..ec60d52790e3 100644
@@ -5,7 +5,7 @@
 #define DRIVER_EMAIL		"[email protected]"
 
 #define DRIVER_NAME		"nouveau"
-#define DRIVER_DESC		"nVidia Riva/TNT/GeForce/Quadro/Tesla"
+#define DRIVER_DESC		"nVidia Riva/TNT/GeForce/Quadro/Tesla/Tegra K1+"
 #define DRIVER_DATE		"20120801"
 
 #define DRIVER_MAJOR		1
Signature · 6 lines
2.11.0

_______________________________________________
Nouveau mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/nouveau
Acked-by on v2 1/1
#5 lore raw
Quoted 6 lines · Rhys Kidd

nouveau supports the Tegra K1 and higher after the SoC-based GPUs converged with the main GeForce GPU families.

v2:
- Qualify that support is Tegra K1+ (Martin Peres)

Quoted 6 lines
---
 drivers/gpu/drm/nouveau/nouveau_drv.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Ben's probably picked this up already, but FWIW:

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

#6 lore raw
Quoted 7 lines · Thierry Reding
Quoted 6 lines · Rhys Kidd

nouveau supports the Tegra K1 and higher after the SoC-based GPUs converged with the main GeForce GPU families.

v2:
- Qualify that support is Tegra K1+ (Martin Peres)

Quoted 8 lines
Quoted 6 lines
---
 drivers/gpu/drm/nouveau/nouveau_drv.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Ben's probably picked this up already, but FWIW:

It's queued in my tree, but, I've added your ack. Thanks!

Quoted 3 lines

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel