lkml.org 
[lkml]   [2004]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    SubjectRe: [PATCH] PCI fixes for 2.6.10-rc1
    Date
    From
    ChangeSet 1.2026.66.19, 2004/11/10 16:44:13-08:00, hannal@us.ibm.com

    [PATCH] zr36120.c: Convert pci_find_device to pci_dev_present

    --On Thursday, September 30, 2004 10:27:48 AM +0200 Gerd Knorr <kraxel@bytesex.org> wrote:
    >> The whole driver is CONFIG_BROKEN anyway... I only verified my changes
    >> didn't add new compiler errors. What part should I remove? Just this check?
    >
    > pci/quirks.c does these checks and sets the flags.
    > Replacing with a check for "pci_pci_problems & PCIPCI_TRITON" should do.
    >
    > bttv does simliar things in bttv_check_chipset() (bttv-cards.c),
    > you might want to have a look there ...
    >

    Thanks a lot for your help. Is this what you were thinking?

    Signed-off-by: Hanna Linder <hannal@us.ibm.com>
    Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>


    drivers/media/video/zr36120.c | 5 +----
    1 files changed, 1 insertion(+), 4 deletions(-)


    diff -Nru a/drivers/media/video/zr36120.c b/drivers/media/video/zr36120.c
    --- a/drivers/media/video/zr36120.c 2004-11-12 15:12:34 -08:00
    +++ b/drivers/media/video/zr36120.c 2004-11-12 15:12:34 -08:00
    @@ -145,14 +145,11 @@
    static
    void __init handle_chipset(void)
    {
    - struct pci_dev *dev = NULL;
    -
    /* Just in case some nut set this to something dangerous */
    if (triton1)
    triton1 = ZORAN_VDC_TRICOM;

    - while ((dev = pci_find_device(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_82437, dev)))
    - {
    + if (pci_pci_problems & PCIPCI_TRITON) {
    printk(KERN_INFO "zoran: Host bridge 82437FX Triton PIIX\n");
    triton1 = ZORAN_VDC_TRICOM;
    }
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2005-03-22 14:08    [W:0.021 / U:0.620 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site