diff for duplicates of <20171128153051.GU10981@intel.com> diff --git a/a/1.txt b/N1/1.txt index 2f66db3..522585f 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,38 +1,45 @@ On Mon, Nov 27, 2017 at 11:57:46AM -0500, Sinan Kaya wrote: -> pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as +> pci_get_bus_and_slot() is restrictive such that it assumes domain=3D0 as > where a PCI device is present. This restricts the device drivers to be > reused for other domain numbers. -> +> = + > Getting ready to remove pci_get_bus_and_slot() function in favor of > pci_get_domain_bus_and_slot(). -> +> = + > Extract the domain number from drm_device and pass it into > pci_get_domain_bus_and_slot() function. -> +> = + > Signed-off-by: Sinan Kaya <okaya@codeaurora.org> > --- > drivers/gpu/drm/i915/i915_drv.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) -> -> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c +> = + +> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_= +drv.c > index 9f45cfe..5a8cb79 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c -> @@ -419,7 +419,10 @@ static int i915_getparam(struct drm_device *dev, void *data, -> +> @@ -419,7 +419,10 @@ static int i915_getparam(struct drm_device *dev, voi= +d *data, +> = + > static int i915_get_bridge_dev(struct drm_i915_private *dev_priv) > { -> - dev_priv->bridge_dev = pci_get_bus_and_slot(0, PCI_DEVFN(0, 0)); -> + int domain = pci_domain_nr(dev_priv->drm.pdev->bus); +> - dev_priv->bridge_dev =3D pci_get_bus_and_slot(0, PCI_DEVFN(0, 0)); +> + int domain =3D pci_domain_nr(dev_priv->drm.pdev->bus); > + -> + dev_priv->bridge_dev = +> + dev_priv->bridge_dev =3D > + pci_get_domain_bus_and_slot(domain, 0, PCI_DEVFN(0, 0)); Maybe just pci_get_slot(pdev->bus, PCI_DEVFN(0, 0)) ? I guess if we want to be pedantic we could go for: -bus = pci_find_host_bridge(pdev->bus)->bus; +bus =3D pci_find_host_bridge(pdev->bus)->bus; pci_get_slot(bus, PCI_DEVFN(0, 0)) but I think the GPU should always be on the root bus, so the simpler @@ -41,18 +48,22 @@ form should be fine. > if (!dev_priv->bridge_dev) { > DRM_ERROR("bridge device not found\n"); > return -1; -> -- +> -- = + > 1.9.1 -> +> = + > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx --- -Ville Syrjälä +-- = + +Ville Syrj=E4l=E4 Intel OTC + _______________________________________________ -dri-devel mailing list -dri-devel@lists.freedesktop.org -https://lists.freedesktop.org/mailman/listinfo/dri-devel +linux-arm-kernel mailing list +linux-arm-kernel@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-arm-kernel diff --git a/a/content_digest b/N1/content_digest index 7c2add1..eae1e5d 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -16,40 +16,47 @@ "\00:1\0" "b\0" "On Mon, Nov 27, 2017 at 11:57:46AM -0500, Sinan Kaya wrote:\n" - "> pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as\n" + "> pci_get_bus_and_slot() is restrictive such that it assumes domain=3D0 as\n" "> where a PCI device is present. This restricts the device drivers to be\n" "> reused for other domain numbers.\n" - "> \n" + "> =\n" + "\n" "> Getting ready to remove pci_get_bus_and_slot() function in favor of\n" "> pci_get_domain_bus_and_slot().\n" - "> \n" + "> =\n" + "\n" "> Extract the domain number from drm_device and pass it into\n" "> pci_get_domain_bus_and_slot() function.\n" - "> \n" + "> =\n" + "\n" "> Signed-off-by: Sinan Kaya <okaya@codeaurora.org>\n" "> ---\n" "> drivers/gpu/drm/i915/i915_drv.c | 5 ++++-\n" "> 1 file changed, 4 insertions(+), 1 deletion(-)\n" - "> \n" - "> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c\n" + "> =\n" + "\n" + "> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_=\n" + "drv.c\n" "> index 9f45cfe..5a8cb79 100644\n" "> --- a/drivers/gpu/drm/i915/i915_drv.c\n" "> +++ b/drivers/gpu/drm/i915/i915_drv.c\n" - "> @@ -419,7 +419,10 @@ static int i915_getparam(struct drm_device *dev, void *data,\n" - "> \n" + "> @@ -419,7 +419,10 @@ static int i915_getparam(struct drm_device *dev, voi=\n" + "d *data,\n" + "> =\n" + "\n" "> static int i915_get_bridge_dev(struct drm_i915_private *dev_priv)\n" "> {\n" - "> -\tdev_priv->bridge_dev = pci_get_bus_and_slot(0, PCI_DEVFN(0, 0));\n" - "> +\tint domain = pci_domain_nr(dev_priv->drm.pdev->bus);\n" + "> -\tdev_priv->bridge_dev =3D pci_get_bus_and_slot(0, PCI_DEVFN(0, 0));\n" + "> +\tint domain =3D pci_domain_nr(dev_priv->drm.pdev->bus);\n" "> +\n" - "> +\tdev_priv->bridge_dev =\n" + "> +\tdev_priv->bridge_dev =3D\n" "> +\t\tpci_get_domain_bus_and_slot(domain, 0, PCI_DEVFN(0, 0));\n" "\n" "Maybe just pci_get_slot(pdev->bus, PCI_DEVFN(0, 0)) ?\n" "\n" "I guess if we want to be pedantic we could go for:\n" "\n" - "bus = pci_find_host_bridge(pdev->bus)->bus;\n" + "bus =3D pci_find_host_bridge(pdev->bus)->bus;\n" "pci_get_slot(bus, PCI_DEVFN(0, 0))\n" "\n" "but I think the GPU should always be on the root bus, so the simpler\n" @@ -58,20 +65,24 @@ "> \tif (!dev_priv->bridge_dev) {\n" "> \t\tDRM_ERROR(\"bridge device not found\\n\");\n" "> \t\treturn -1;\n" - "> -- \n" + "> -- =\n" + "\n" "> 1.9.1\n" - "> \n" + "> =\n" + "\n" "> _______________________________________________\n" "> Intel-gfx mailing list\n" "> Intel-gfx@lists.freedesktop.org\n" "> https://lists.freedesktop.org/mailman/listinfo/intel-gfx\n" "\n" - "-- \n" - "Ville Syrj\303\244l\303\244\n" + "-- =\n" + "\n" + "Ville Syrj=E4l=E4\n" "Intel OTC\n" + "\n" "_______________________________________________\n" - "dri-devel mailing list\n" - "dri-devel@lists.freedesktop.org\n" - https://lists.freedesktop.org/mailman/listinfo/dri-devel + "linux-arm-kernel mailing list\n" + "linux-arm-kernel@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -17800e38155e5cdac4a76553c8737a0f6860725bd862a01d84606f4b8a48a487 +b26bf96ef7bc8997ba2d5ff6f22501333dce38994303f3bdac7edf2d90abb8e4
diff --git a/a/1.txt b/N2/1.txt index 2f66db3..e3fea5f 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -46,13 +46,9 @@ form should be fine. > > _______________________________________________ > Intel-gfx mailing list -> Intel-gfx@lists.freedesktop.org +> Intel-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- -Ville Syrjälä +Ville Syrj?l? Intel OTC -_______________________________________________ -dri-devel mailing list -dri-devel@lists.freedesktop.org -https://lists.freedesktop.org/mailman/listinfo/dri-devel diff --git a/a/content_digest b/N2/content_digest index 7c2add1..8d3ce86 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,18 +1,9 @@ "ref\01511801886-6753-1-git-send-email-okaya@codeaurora.org\0" "ref\01511801886-6753-10-git-send-email-okaya@codeaurora.org\0" - "From\0Ville Syrj\303\244l\303\244 <ville.syrjala@linux.intel.com>\0" - "Subject\0Re: [Intel-gfx] [PATCH V3 09/29] drm/i915: deprecate pci_get_bus_and_slot()\0" + "From\0ville.syrjala@linux.intel.com (Ville Syrj\303\244l\303\244)\0" + "Subject\0[Intel-gfx] [PATCH V3 09/29] drm/i915: deprecate pci_get_bus_and_slot()\0" "Date\0Tue, 28 Nov 2017 17:30:51 +0200\0" - "To\0Sinan Kaya <okaya@codeaurora.org>\0" - "Cc\0David Airlie <airlied@linux.ie>" - linux-pci@vger.kernel.org - timur@codeaurora.org - Rodrigo Vivi <rodrigo.vivi@intel.com> - open list <linux-kernel@vger.kernel.org> - open list:DRM DRIVERS <dri-devel@lists.freedesktop.org> - linux-arm-msm@vger.kernel.org - intel-gfx@lists.freedesktop.org - " linux-arm-kernel@lists.infradead.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Mon, Nov 27, 2017 at 11:57:46AM -0500, Sinan Kaya wrote:\n" @@ -63,15 +54,11 @@ "> \n" "> _______________________________________________\n" "> Intel-gfx mailing list\n" - "> Intel-gfx@lists.freedesktop.org\n" + "> Intel-gfx at lists.freedesktop.org\n" "> https://lists.freedesktop.org/mailman/listinfo/intel-gfx\n" "\n" "-- \n" - "Ville Syrj\303\244l\303\244\n" - "Intel OTC\n" - "_______________________________________________\n" - "dri-devel mailing list\n" - "dri-devel@lists.freedesktop.org\n" - https://lists.freedesktop.org/mailman/listinfo/dri-devel + "Ville Syrj?l?\n" + Intel OTC -17800e38155e5cdac4a76553c8737a0f6860725bd862a01d84606f4b8a48a487 +7fde0785640612528f2322cc25e4c1e82300c0a5e2be45724d3dbc5080893eaa
diff --git a/a/1.txt b/N3/1.txt index 2f66db3..7d60700 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -52,7 +52,3 @@ form should be fine. -- Ville Syrjälä Intel OTC -_______________________________________________ -dri-devel mailing list -dri-devel@lists.freedesktop.org -https://lists.freedesktop.org/mailman/listinfo/dri-devel diff --git a/a/content_digest b/N3/content_digest index 7c2add1..192addd 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -4,14 +4,14 @@ "Subject\0Re: [Intel-gfx] [PATCH V3 09/29] drm/i915: deprecate pci_get_bus_and_slot()\0" "Date\0Tue, 28 Nov 2017 17:30:51 +0200\0" "To\0Sinan Kaya <okaya@codeaurora.org>\0" - "Cc\0David Airlie <airlied@linux.ie>" - linux-pci@vger.kernel.org + "Cc\0linux-pci@vger.kernel.org" timur@codeaurora.org - Rodrigo Vivi <rodrigo.vivi@intel.com> - open list <linux-kernel@vger.kernel.org> - open list:DRM DRIVERS <dri-devel@lists.freedesktop.org> + David Airlie <airlied@linux.ie> linux-arm-msm@vger.kernel.org intel-gfx@lists.freedesktop.org + open list <linux-kernel@vger.kernel.org> + open list:DRM DRIVERS <dri-devel@lists.freedesktop.org> + Rodrigo Vivi <rodrigo.vivi@intel.com> " linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" @@ -68,10 +68,6 @@ "\n" "-- \n" "Ville Syrj\303\244l\303\244\n" - "Intel OTC\n" - "_______________________________________________\n" - "dri-devel mailing list\n" - "dri-devel@lists.freedesktop.org\n" - https://lists.freedesktop.org/mailman/listinfo/dri-devel + Intel OTC -17800e38155e5cdac4a76553c8737a0f6860725bd862a01d84606f4b8a48a487 +5e939bf8a44015578fe9f790736a038327be6c769ccc7c6b0d743c593e45103c
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.