[PATCH] video: fbdev: offb: Call pci_enable_device() before using the PCI VGA device

Subsystems: framebuffer layer, the rest

STALE3597d

3 messages, 2 authors, 2016-09-27 · open the first message on its own page

[PATCH] video: fbdev: offb: Call pci_enable_device() before using the PCI VGA device

From: Yongji Xie <hidden>
Date: 2016-09-13 05:53:46

Currently the offb module will use the PCI VGA device as frame
buffer device without calling something like pci_enable_device().
However, this would cause some problem if we disable memory
decoding of the upstream bridge before. When the console driver
issued memory access to the VGA device, the access cannot be
supported by the bridge which will cause EEH error on Power machine.

Signed-off-by: Yongji Xie <redacted>
---
 drivers/video/fbdev/offb.c | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
diff --git a/drivers/video/fbdev/offb.c b/drivers/video/fbdev/offb.c
index fb60a8f..906c6e7 100644
--- a/drivers/video/fbdev/offb.c
+++ b/drivers/video/fbdev/offb.c
@@ -625,6 +625,21 @@ static void __init offb_init_nodriver(struct device_node *dp, int no_real_node)
 	if (address = OF_BAD_ADDR && addr_prop)
 		address = (u64)addr_prop;
 	if (address != OF_BAD_ADDR) {
+#ifdef CONFIG_PCI
+		const __be32 *vidp, *didp;
+		u32 vid, did;
+		struct pci_dev *pdev;
+
+		vidp = of_get_property(dp, "vendor-id", NULL);
+		didp = of_get_property(dp, "device-id", NULL);
+		if (vidp && didp) {
+			vid = be32_to_cpup(vidp);
+			did = be32_to_cpup(didp);
+			pdev = pci_get_device(vid, did, NULL);
+			if (!pdev || pci_enable_device(pdev))
+				return;
+		}
+#endif
 		/* kludge for valkyrie */
 		if (strcmp(dp->name, "valkyrie") = 0)
 			address += 0x1000;
-- 
1.8.3.1

Re: [PATCH] video: fbdev: offb: Call pci_enable_device() before using the PCI VGA device

From: Yongji Xie <hidden>
Date: 2016-09-27 03:02:19

Kindly ping...

Thanks,

Yongji

On 2016/9/13 13:53, Yongji Xie wrote:
quoted hunk
Currently the offb module will use the PCI VGA device as frame
buffer device without calling something like pci_enable_device().
However, this would cause some problem if we disable memory
decoding of the upstream bridge before. When the console driver
issued memory access to the VGA device, the access cannot be
supported by the bridge which will cause EEH error on Power machine.

Signed-off-by: Yongji Xie <redacted>
---
  drivers/video/fbdev/offb.c | 15 +++++++++++++++
  1 file changed, 15 insertions(+)
diff --git a/drivers/video/fbdev/offb.c b/drivers/video/fbdev/offb.c
index fb60a8f..906c6e7 100644
--- a/drivers/video/fbdev/offb.c
+++ b/drivers/video/fbdev/offb.c
@@ -625,6 +625,21 @@ static void __init offb_init_nodriver(struct device_node *dp, int no_real_node)
  	if (address = OF_BAD_ADDR && addr_prop)
  		address = (u64)addr_prop;
  	if (address != OF_BAD_ADDR) {
+#ifdef CONFIG_PCI
+		const __be32 *vidp, *didp;
+		u32 vid, did;
+		struct pci_dev *pdev;
+
+		vidp = of_get_property(dp, "vendor-id", NULL);
+		didp = of_get_property(dp, "device-id", NULL);
+		if (vidp && didp) {
+			vid = be32_to_cpup(vidp);
+			did = be32_to_cpup(didp);
+			pdev = pci_get_device(vid, did, NULL);
+			if (!pdev || pci_enable_device(pdev))
+				return;
+		}
+#endif
  		/* kludge for valkyrie */
  		if (strcmp(dp->name, "valkyrie") = 0)
  			address += 0x1000;

Re: [PATCH] video: fbdev: offb: Call pci_enable_device() before using the PCI VGA device

From: Tomi Valkeinen <hidden>
Date: 2016-09-27 07:55:33

On 13/09/16 08:53, Yongji Xie wrote:
Currently the offb module will use the PCI VGA device as frame
buffer device without calling something like pci_enable_device().
However, this would cause some problem if we disable memory
decoding of the upstream bridge before. When the console driver
issued memory access to the VGA device, the access cannot be
supported by the bridge which will cause EEH error on Power machine.

Signed-off-by: Yongji Xie <redacted>
---
 drivers/video/fbdev/offb.c | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
Thanks, queued for 4.9.

 Tomi
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help