Re: [PATCH] nvidiafb: detect the hardware support before removing console.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear Dave

Thank you for your patch.

On Sun, Feb 5, 2023 at 10:07 PM Dave Airlie <airlied@xxxxxxxxx> wrote:
>
> From: Dave Airlie <airlied@xxxxxxxxxx>
>
> This driver removed the console, but hasn't yet decided if it could
> take over the console yet. Instead of doing that, probe the hw for
> support and then remove the console afterwards.
>
> Signed-off-by: Dave Airlie <airlied@xxxxxxxxxx>
> Fixes: https://bugzilla.kernel.org/show_bug.cgi?id=216859
> Reported-by: Zeno Davatz <zdavatz@xxxxxxxxx>
> ---
>  drivers/video/fbdev/nvidia/nvidia.c | 81 +++++++++++++++--------------
>  1 file changed, 42 insertions(+), 39 deletions(-)
>
> diff --git a/drivers/video/fbdev/nvidia/nvidia.c b/drivers/video/fbdev/nvidia/nvidia.c
> index 1960916098d4..e60a276b4855 100644
> --- a/drivers/video/fbdev/nvidia/nvidia.c
> +++ b/drivers/video/fbdev/nvidia/nvidia.c
> @@ -1197,17 +1197,17 @@ static int nvidia_set_fbinfo(struct fb_info *info)
>         return nvidiafb_check_var(&info->var, info);
>  }
>
> -static u32 nvidia_get_chipset(struct fb_info *info)
> +static u32 nvidia_get_chipset(struct pci_dev *pci_dev,
> +                             volatile u32 __iomem *REGS)
>  {
> -       struct nvidia_par *par = info->par;
> -       u32 id = (par->pci_dev->vendor << 16) | par->pci_dev->device;
> +       u32 id = (pci_dev->vendor << 16) | pci_dev->device;
>
>         printk(KERN_INFO PFX "Device ID: %x \n", id);
>
>         if ((id & 0xfff0) == 0x00f0 ||
>             (id & 0xfff0) == 0x02e0) {
>                 /* pci-e */
> -               id = NV_RD32(par->REGS, 0x1800);
> +               id = NV_RD32(REGS, 0x1800);
>
>                 if ((id & 0x0000ffff) == 0x000010DE)
>                         id = 0x10DE0000 | (id >> 16);
> @@ -1220,12 +1220,11 @@ static u32 nvidia_get_chipset(struct fb_info *info)
>         return id;
>  }
>
> -static u32 nvidia_get_arch(struct fb_info *info)
> +static u32 nvidia_get_arch(u32 Chipset)
>  {
> -       struct nvidia_par *par = info->par;
>         u32 arch = 0;
>
> -       switch (par->Chipset & 0x0ff0) {
> +       switch (Chipset & 0x0ff0) {
>         case 0x0100:            /* GeForce 256 */
>         case 0x0110:            /* GeForce2 MX */
>         case 0x0150:            /* GeForce2 */
> @@ -1278,16 +1277,44 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
>         struct fb_info *info;
>         unsigned short cmd;
>         int ret;
> +       volatile u32 __iomem *REGS;
> +       int Chipset;
> +       u32 Architecture;
>
>         NVTRACE_ENTER();
>         assert(pd != NULL);
>
> +       if (pci_enable_device(pd)) {
> +               printk(KERN_ERR PFX "cannot enable PCI device\n");
> +               return -ENODEV;
> +       }
> +
> +       /* enable IO and mem if not already done */
> +       pci_read_config_word(pd, PCI_COMMAND, &cmd);
> +       cmd |= (PCI_COMMAND_IO | PCI_COMMAND_MEMORY);
> +       pci_write_config_word(pd, PCI_COMMAND, cmd);
> +
> +       nvidiafb_fix.mmio_start = pci_resource_start(pd, 0);
> +       nvidiafb_fix.mmio_len = pci_resource_len(pd, 0);
> +
> +       REGS = ioremap(nvidiafb_fix.mmio_start, nvidiafb_fix.mmio_len);
> +       if (!REGS) {
> +               printk(KERN_ERR PFX "cannot ioremap MMIO base\n");
> +               return -ENODEV;
> +       }
> +
> +       Chipset = nvidia_get_chipset(pd, REGS);
> +       Architecture = nvidia_get_arch(Chipset);
> +       if (Architecture == 0) {
> +               printk(KERN_ERR PFX "unknown NV_ARCH\n");
> +               goto err_out;
> +       }
> +
>         ret = aperture_remove_conflicting_pci_devices(pd, "nvidiafb");
>         if (ret)
> -               return ret;
> +               goto err_out;
>
>         info = framebuffer_alloc(sizeof(struct nvidia_par), &pd->dev);
> -
>         if (!info)
>                 goto err_out;
>
> @@ -1298,11 +1325,6 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
>         if (info->pixmap.addr == NULL)
>                 goto err_out_kfree;
>
> -       if (pci_enable_device(pd)) {
> -               printk(KERN_ERR PFX "cannot enable PCI device\n");
> -               goto err_out_enable;
> -       }
> -
>         if (pci_request_regions(pd, "nvidiafb")) {
>                 printk(KERN_ERR PFX "cannot request PCI regions\n");
>                 goto err_out_enable;
> @@ -1318,34 +1340,17 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
>         par->paneltweak = paneltweak;
>         par->reverse_i2c = reverse_i2c;
>
> -       /* enable IO and mem if not already done */
> -       pci_read_config_word(pd, PCI_COMMAND, &cmd);
> -       cmd |= (PCI_COMMAND_IO | PCI_COMMAND_MEMORY);
> -       pci_write_config_word(pd, PCI_COMMAND, cmd);
> -
> -       nvidiafb_fix.mmio_start = pci_resource_start(pd, 0);
>         nvidiafb_fix.smem_start = pci_resource_start(pd, 1);
> -       nvidiafb_fix.mmio_len = pci_resource_len(pd, 0);
> -
> -       par->REGS = ioremap(nvidiafb_fix.mmio_start, nvidiafb_fix.mmio_len);
>
> -       if (!par->REGS) {
> -               printk(KERN_ERR PFX "cannot ioremap MMIO base\n");
> -               goto err_out_free_base0;
> -       }
> +       par->REGS = REGS;
>
> -       par->Chipset = nvidia_get_chipset(info);
> -       par->Architecture = nvidia_get_arch(info);
> -
> -       if (par->Architecture == 0) {
> -               printk(KERN_ERR PFX "unknown NV_ARCH\n");
> -               goto err_out_arch;
> -       }
> +       par->Chipset = Chipset;
> +       par->Architecture = Architecture;
>
>         sprintf(nvidiafb_fix.id, "NV%x", (pd->device & 0x0ff0) >> 4);
>
>         if (NVCommonSetup(info))
> -               goto err_out_arch;
> +               goto err_out_free_base0;
>
>         par->FbAddress = nvidiafb_fix.smem_start;
>         par->FbMapSize = par->RamAmountKBytes * 1024;
> @@ -1401,7 +1406,6 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
>                 goto err_out_iounmap_fb;
>         }
>
> -
>         printk(KERN_INFO PFX
>                "PCI nVidia %s framebuffer (%dMB @ 0x%lX)\n",
>                info->fix.id,
> @@ -1415,15 +1419,14 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
>  err_out_free_base1:
>         fb_destroy_modedb(info->monspecs.modedb);
>         nvidia_delete_i2c_busses(par);
> -err_out_arch:
> -       iounmap(par->REGS);
> - err_out_free_base0:
> +err_out_free_base0:
>         pci_release_regions(pd);
>  err_out_enable:
>         kfree(info->pixmap.addr);
>  err_out_kfree:
>         framebuffer_release(info);
>  err_out:
> +       iounmap(REGS);
>         return -ENODEV;
>  }
>
> --
> 2.38.1

This patch fails for me.

sudo patch -p1 < /tmp/patch
Passwort:
patching file drivers/video/fbdev/nvidia/nvidia.c
Hunk #1 FAILED at 1197.
Hunk #2 FAILED at 1220.
Hunk #3 FAILED at 1278.
Hunk #4 FAILED at 1298.
Hunk #5 FAILED at 1318.
Hunk #6 FAILED at 1401.
Hunk #7 FAILED at 1415.
7 out of 7 hunks FAILED -- saving rejects to file
drivers/video/fbdev/nvidia/nvidia.c.rej

See attachments.

Best
Zeno

Attachment: patch
Description: Binary data

--- drivers/video/fbdev/nvidia/nvidia.c
+++ drivers/video/fbdev/nvidia/nvidia.c
@@ -1197,17 +1197,17 @@ static int nvidia_set_fbinfo(struct fb_info *info)
        return nvidiafb_check_var(&info->var, info);
 }
 
-static u32 nvidia_get_chipset(struct fb_info *info)
+static u32 nvidia_get_chipset(struct pci_dev *pci_dev,
+                             volatile u32 __iomem *REGS)
 {
-       struct nvidia_par *par = info->par;
-       u32 id = (par->pci_dev->vendor << 16) | par->pci_dev->device;
+       u32 id = (pci_dev->vendor << 16) | pci_dev->device;
 
        printk(KERN_INFO PFX "Device ID: %x \n", id);
 
        if ((id & 0xfff0) == 0x00f0 ||
            (id & 0xfff0) == 0x02e0) {
                /* pci-e */
-               id = NV_RD32(par->REGS, 0x1800);
+               id = NV_RD32(REGS, 0x1800);
 
                if ((id & 0x0000ffff) == 0x000010DE)
                        id = 0x10DE0000 | (id >> 16);
@@ -1220,12 +1220,11 @@ static u32 nvidia_get_chipset(struct fb_info *info)
        return id;
 }
 
-static u32 nvidia_get_arch(struct fb_info *info)
+static u32 nvidia_get_arch(u32 Chipset)
 {
-       struct nvidia_par *par = info->par;
        u32 arch = 0;
 
-       switch (par->Chipset & 0x0ff0) {
+       switch (Chipset & 0x0ff0) {
        case 0x0100:            /* GeForce 256 */
        case 0x0110:            /* GeForce2 MX */
        case 0x0150:            /* GeForce2 */
@@ -1278,16 +1277,44 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
        struct fb_info *info;
        unsigned short cmd;
        int ret;
+       volatile u32 __iomem *REGS;
+       int Chipset;
+       u32 Architecture;
 
        NVTRACE_ENTER();
        assert(pd != NULL);
 
+       if (pci_enable_device(pd)) {
+               printk(KERN_ERR PFX "cannot enable PCI device\n");
+               return -ENODEV;
+       }
+
+       /* enable IO and mem if not already done */
+       pci_read_config_word(pd, PCI_COMMAND, &cmd);
+       cmd |= (PCI_COMMAND_IO | PCI_COMMAND_MEMORY);
+       pci_write_config_word(pd, PCI_COMMAND, cmd);
+
+       nvidiafb_fix.mmio_start = pci_resource_start(pd, 0);
+       nvidiafb_fix.mmio_len = pci_resource_len(pd, 0);
+
+       REGS = ioremap(nvidiafb_fix.mmio_start, nvidiafb_fix.mmio_len);
+       if (!REGS) {
+               printk(KERN_ERR PFX "cannot ioremap MMIO base\n");
+               return -ENODEV;
+       }
+
+       Chipset = nvidia_get_chipset(pd, REGS);
+       Architecture = nvidia_get_arch(Chipset);
+       if (Architecture == 0) {
+               printk(KERN_ERR PFX "unknown NV_ARCH\n");
+               goto err_out;
+       }
+
        ret = aperture_remove_conflicting_pci_devices(pd, "nvidiafb");
        if (ret)
-               return ret;
+               goto err_out;
 
        info = framebuffer_alloc(sizeof(struct nvidia_par), &pd->dev);
-
        if (!info)
                goto err_out;
 
@@ -1298,11 +1325,6 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
        if (info->pixmap.addr == NULL)
                goto err_out_kfree;
 
-       if (pci_enable_device(pd)) {
-               printk(KERN_ERR PFX "cannot enable PCI device\n");
-               goto err_out_enable;
-       }
-
        if (pci_request_regions(pd, "nvidiafb")) {
                printk(KERN_ERR PFX "cannot request PCI regions\n");
                goto err_out_enable;
@@ -1318,34 +1340,17 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
        par->paneltweak = paneltweak;
        par->reverse_i2c = reverse_i2c;
 
-       /* enable IO and mem if not already done */
-       pci_read_config_word(pd, PCI_COMMAND, &cmd);
-       cmd |= (PCI_COMMAND_IO | PCI_COMMAND_MEMORY);
-       pci_write_config_word(pd, PCI_COMMAND, cmd);
-
-       nvidiafb_fix.mmio_start = pci_resource_start(pd, 0);
        nvidiafb_fix.smem_start = pci_resource_start(pd, 1);
-       nvidiafb_fix.mmio_len = pci_resource_len(pd, 0);
-
-       par->REGS = ioremap(nvidiafb_fix.mmio_start, nvidiafb_fix.mmio_len);
 
-       if (!par->REGS) {
-               printk(KERN_ERR PFX "cannot ioremap MMIO base\n");
-               goto err_out_free_base0;
-       }
+       par->REGS = REGS;
 
-       par->Chipset = nvidia_get_chipset(info);
-       par->Architecture = nvidia_get_arch(info);
-
-       if (par->Architecture == 0) {
-               printk(KERN_ERR PFX "unknown NV_ARCH\n");
-               goto err_out_arch;
-       }
+       par->Chipset = Chipset;
+       par->Architecture = Architecture;
 
        sprintf(nvidiafb_fix.id, "NV%x", (pd->device & 0x0ff0) >> 4);
 
        if (NVCommonSetup(info))
-               goto err_out_arch;
+               goto err_out_free_base0;
 
        par->FbAddress = nvidiafb_fix.smem_start;
        par->FbMapSize = par->RamAmountKBytes * 1024;
@@ -1401,7 +1406,6 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
                goto err_out_iounmap_fb;
        }
 
-
        printk(KERN_INFO PFX
               "PCI nVidia %s framebuffer (%dMB @ 0x%lX)\n",
               info->fix.id,
@@ -1415,15 +1419,14 @@ static int nvidiafb_probe(struct pci_dev *pd, const struct pci_device_id *ent)
 err_out_free_base1:
        fb_destroy_modedb(info->monspecs.modedb);
        nvidia_delete_i2c_busses(par);
-err_out_arch:
-       iounmap(par->REGS);
- err_out_free_base0:
+err_out_free_base0:
        pci_release_regions(pd);
 err_out_enable:
        kfree(info->pixmap.addr);
 err_out_kfree:
        framebuffer_release(info);
 err_out:
+       iounmap(REGS);
        return -ENODEV;
 }
 

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux