[Karmic SRU] [PATCH] UBUNTU: drm/i915: Support for another B43 device id
Ike Panhc
ike.pan at canonical.com
Tue Sep 28 10:11:17 UTC 2010
On 09/28/2010 05:15 PM, Stefan Bader wrote:
>> + agp_bridge->dev->device == PCI_DEVICE_ID_INTEL_B43_1_HB || \
>> + agp_bridge->dev->device == PCI_DEVICE_ID_INTEL_B43_1_HB || \
>
> Why does that add the same line twice? I assume IG is the internal graphics part
> and HB the host bridge? The rest seems to add the ids to the correct places (as
> far as one can tell).
>
Sorry, this is my fault. The second line is redundant. Will resend.
>> (dev)->pci_device == 0x2E22 || \
>> (dev)->pci_device == 0x2E32 || \
>> (dev)->pci_device == 0x2E42 || \
>> + (dev)->pci_device == 0x2E92 || \
>> (dev)->pci_device == 0x0042 || \
>> (dev)->pci_device == 0x0046)
>>
> It would be simpler to read if upstream had decided to stick with the defines
> rather than falling back to raw numbers, but that is like it is upstream. Just a
> thought.
>
I have this patch remade against karmic not cherry-picked because the drm/i915 has
been modified a lot from .31 to .36. I choose to follow .31 way so that the patch
can be smaller as it can.
On 09/28/2010 05:17 PM, Andy Whitcroft wrote:
>
> If this commit is in mainline we should be just cherry-picking this as
> an upstream patch. Though I see this upstream version is different. Is
> this therefore effectivly a backport to karmic ?
>
Since cherry-pick from upstream brings lots of conflicts, I think this is a backport
patch.
More information about the kernel-team
mailing list