From mboxrd@z Thu Jan 1 00:00:00 1970 From: Auke Kok Subject: Re: [PATCH]: e1000: Janitor: Use #defined values for literals Date: Fri, 23 Jun 2006 11:07:28 -0700 Message-ID: <449C2DE0.4000907@foo-projects.org> References: <20060623163624.GM8866@austin.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, john.ronciak@intel.com, jesse.brandeburg@intel.com, jeffrey.t.kirsher@intel.com, "Zhang, Yanmin" , Jeff Garzik , linux-kernel@vger.kernel.org Return-path: Received: from mga05.intel.com ([192.55.52.89]:43162 "EHLO fmsmga101.fm.intel.com") by vger.kernel.org with ESMTP id S1751848AbWFWSIG (ORCPT ); Fri, 23 Jun 2006 14:08:06 -0400 To: Linas Vepstas In-Reply-To: <20060623163624.GM8866@austin.ibm.com> Sender: netdev-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Linas Vepstas wrote: > Minor janitorial patch: use #defines for literal values. > > Signed-off-by: Linas Vepstas Ack! I thought we had gotten these out already. Cheers, Auke > > ---- > drivers/net/e1000/e1000_main.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > Index: linux-2.6.17-rc6-mm2/drivers/net/e1000/e1000_main.c > =================================================================== > --- linux-2.6.17-rc6-mm2.orig/drivers/net/e1000/e1000_main.c 2006-06-13 18:13:30.000000000 -0500 > +++ linux-2.6.17-rc6-mm2/drivers/net/e1000/e1000_main.c 2006-06-23 11:27:47.000000000 -0500 > @@ -4663,8 +4663,8 @@ static pci_ers_result_t e1000_io_slot_re > } > pci_set_master(pdev); > > - pci_enable_wake(pdev, 3, 0); > - pci_enable_wake(pdev, 4, 0); /* 4 == D3 cold */ > + pci_enable_wake(pdev, PCI_D3hot, 0); > + pci_enable_wake(pdev, PCI_D3cold, 0); > > /* Perform card reset only on one instance of the card */ > if (PCI_FUNC (pdev->devfn) != 0) > - > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html