Subject: Re: [PATCH 7/7] e1000e: remove unused read value in ASPM disable code.

From: Jeff Garzik <[email-masked]>
Date: Sun, 24 Feb 2008 00:11:57 -0500

Auke Kok wrote:
> Signed-off-by: Auke Kok <[email-masked]>
> ---
> 
>  drivers/net/e1000e/netdev.c |    2 --
>  1 files changed, 0 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/e1000e/netdev.c b/drivers/net/e1000e/netdev.c
> index 7824bc2..fc5c63f 100644
> --- a/drivers/net/e1000e/netdev.c
> +++ b/drivers/net/e1000e/netdev.c
> @@ -3490,7 +3490,6 @@ static int e1000_suspend(struct pci_dev *pdev, pm_message_t state)
>  static void e1000e_disable_l1aspm(struct pci_dev *pdev)
>  {
>  	int pos;
> -	u32 cap;
>  	u16 val;
>  
>  	/*
> @@ -3505,7 +3504,6 @@ static void e1000e_disable_l1aspm(struct pci_dev *pdev)
>  	 * active.
>  	 */
>  	pos = pci_find_capability(pdev, PCI_CAP_ID_EXP);
> -	pci_read_config_dword(pdev, pos + PCI_EXP_LNKCAP, &cap);
>  	pci_read_config_word(pdev, pos + PCI_EXP_LNKCTL, &val);
>  	if (val & 0x2) {

applied 1-7

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/