linux-kernel - RE: [PATCH resend] cpqarray: use pci

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <0F5B06BAB751E047AB5C87D1F77A77887D5522D9A8@GVW0547EXC.americas.hpqcorp.net>
Date:	Thu, 21 Jul 2011 17:21:47 +0100
From:	"Miller, Mike (OS Dev)" <Mike.Miller@...com>
To:	Sergei Shtylyov <sshtylyov@...mvista.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"chirag.kantharia@...com" <chirag.kantharia@...com>,
	ISS StorageDev <iss_storagedev@...com>
Subject: RE: [PATCH resend] cpqarray: use pci_dev->revision



-- mikem

"If men are so wicked with religion, what would they be if without it?" --Benjamin Franklin 


> -----Original Message-----
> From: Sergei Shtylyov [mailto:sshtylyov@...mvista.com]
> Sent: Thursday, July 21, 2011 10:51 AM
> To: linux-kernel@...r.kernel.org; chirag.kantharia@...com; ISS
> StorageDev
> Subject: [PATCH resend] cpqarray: use pci_dev->revision
> 
> This driver uses PCI_CLASS_REVISION instead of PCI_REVISION_ID, so it
> wasn't
> converted by commit 44c10138fd4bbc4b6d6bff0873c24902f2a9da65 (PCI:
> Change all
> drivers to use pci_device->revision).
> 
> Signed-off-by: Sergei Shtylyov <sshtylyov@...mvista.com>
> 
> ---
> The patch is against the recent Linus' tree.
> It wasn't merged in either 2.6.39 or 3.0 time, hopefully it can be
> merged to
> 3.1...
> 
>  drivers/block/cpqarray.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Mike Miller <mike.miller@...com>

> Index: linux-2.6/drivers/block/cpqarray.c
> ===================================================================
> --- linux-2.6.orig/drivers/block/cpqarray.c
> +++ linux-2.6/drivers/block/cpqarray.c
> @@ -620,6 +620,7 @@ static int cpqarray_pci_init(ctlr_info_t
>  	}
>  	vendor_id = pdev->vendor;
>  	device_id = pdev->device;
> +	revision  = pdev->revision;
>  	irq = pdev->irq;
> 
>  	for(i=0; i<6; i++)
> @@ -632,7 +633,6 @@ static int cpqarray_pci_init(ctlr_info_t
>  	}
> 
>  	pci_read_config_word(pdev, PCI_COMMAND, &command);
> -	pci_read_config_byte(pdev, PCI_CLASS_REVISION, &revision);
>  	pci_read_config_byte(pdev, PCI_CACHE_LINE_SIZE, &cache_line_size);
>  	pci_read_config_byte(pdev, PCI_LATENCY_TIMER, &latency_timer);
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/