diff mbox series

[2/2] vgaarb: Keep adding VGA device in queue

Message ID 1535739600-8842-2-git-send-email-aaron.ma@canonical.com (mailing list archive)
State New, archived
Headers show
Series [1/2] vgaarb: Add support for 64-bit frame buffer address | expand

Commit Message

Aaron Ma Aug. 31, 2018, 6:20 p.m. UTC
If failed to find the deivice owning the boot framebuffer,
try to use the first VGA device instead of the last one.

Usually the 1st device is integrated GPU who owns the boot framebuffer.

Signed-off-by: Aaron Ma <aaron.ma@canonical.com>
---
 drivers/gpu/vga/vgaarb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alex Deucher Sept. 12, 2018, 8:21 p.m. UTC | #1
On Fri, Aug 31, 2018 at 5:39 PM Aaron Ma <aaron.ma@canonical.com> wrote:
>
> If failed to find the deivice owning the boot framebuffer,
> try to use the first VGA device instead of the last one.
>
> Usually the 1st device is integrated GPU who owns the boot framebuffer.
>
> Signed-off-by: Aaron Ma <aaron.ma@canonical.com>

Series is:
Acked-by: Alex Deucher <alexander.deucher@amd.com>

> ---
>  drivers/gpu/vga/vgaarb.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/vga/vgaarb.c b/drivers/gpu/vga/vgaarb.c
> index 8d90e66994b0..dc8e039bfab5 100644
> --- a/drivers/gpu/vga/vgaarb.c
> +++ b/drivers/gpu/vga/vgaarb.c
> @@ -676,7 +676,7 @@ static bool vga_arbiter_add_pci_device(struct pci_dev *pdev)
>         vga_arbiter_check_bridge_sharing(vgadev);
>
>         /* Add to the list */
> -       list_add(&vgadev->list, &vga_list);
> +       list_add_tail(&vgadev->list, &vga_list);
>         vga_count++;
>         vgaarb_info(&pdev->dev, "VGA device added: decodes=%s,owns=%s,locks=%s\n",
>                 vga_iostate_to_str(vgadev->decodes),
> --
> 2.17.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Aaron Ma Oct. 13, 2018, 4:09 p.m. UTC | #2
Hi David:

Could you review and apply these 2 patches?

Regards,
Aaron
Aaron Ma Oct. 20, 2018, 6:56 a.m. UTC | #3
Hi David:

get_maintainer.pl still got your old e-mail.

Add airlied@redhat.com.

Loop Sean too.

Please review and apply these 2 patches.

Thanks,
Aaron
Daniel Vetter Oct. 23, 2018, 5:23 a.m. UTC | #4
On Sat, Oct 20, 2018 at 02:56:36PM +0800, Aaron Ma wrote:
> Hi David:
> 
> get_maintainer.pl still got your old e-mail.
> 
> Add airlied@redhat.com.
> 
> Loop Sean too.
> 
> Please review and apply these 2 patches.

Both pushed to drm-misc-next.
-Daniel

> 
> Thanks,
> Aaron
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
diff mbox series

Patch

diff --git a/drivers/gpu/vga/vgaarb.c b/drivers/gpu/vga/vgaarb.c
index 8d90e66994b0..dc8e039bfab5 100644
--- a/drivers/gpu/vga/vgaarb.c
+++ b/drivers/gpu/vga/vgaarb.c
@@ -676,7 +676,7 @@  static bool vga_arbiter_add_pci_device(struct pci_dev *pdev)
 	vga_arbiter_check_bridge_sharing(vgadev);
 
 	/* Add to the list */
-	list_add(&vgadev->list, &vga_list);
+	list_add_tail(&vgadev->list, &vga_list);
 	vga_count++;
 	vgaarb_info(&pdev->dev, "VGA device added: decodes=%s,owns=%s,locks=%s\n",
 		vga_iostate_to_str(vgadev->decodes),