From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [ PATCH 2/3 ] net:smc911x Modify driver to also work with omap24xx Date: Thu, 3 Jul 2008 13:14:56 +0300 Message-ID: <20080703101452.GF14276@atomide.com> References: <20080626122202.GE12992@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-01-bos.mailhop.org ([63.208.196.178]:49192 "EHLO mho-01-bos.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751997AbYGCMKE (ORCPT ); Thu, 3 Jul 2008 08:10:04 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Arun KS Cc: linux-omap@vger.kernel.org * Arun KS [080626 18:17]: > On Thu, Jun 26, 2008 at 5:52 PM, Tony Lindgren wrote: > > Hi, > > > > * Arun KS [080624 15:14]: > >> net:smc911x Modify driver to also work with omap24xx > > > > Few comments below. > > > > Also, can you please send this to the Linux netdev mailing list? > > Also Cc linux-omap list too. > > > >> Signed-off-by: Arun KS > >> --- > >> drivers/net/Kconfig | 2 +- > >> drivers/net/smc911x.h | 5 +++++ > >> 2 files changed, 6 insertions(+), 1 deletions(-) > >> > >> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig > >> index 61ecee7..17d0a9c 100644 > >> --- a/drivers/net/Kconfig > >> +++ b/drivers/net/Kconfig > >> @@ -955,7 +955,7 @@ config SMC911X > >> tristate "SMSC LAN911[5678] support" > >> select CRC32 > >> select MII > >> - depends on ARCH_PXA || SH_MAGIC_PANEL_R2 || ARCH_OMAP34XX > >> + depends on ARCH_PXA || SH_MAGIC_PANEL_R2 || ARCH_OMAP34XX || ARCH_OMAP24XX > >> help > >> This is a driver for SMSC's LAN911x series of Ethernet chipsets > >> including the new LAN9115, LAN9116, LAN9117, and LAN9118. > > > > Maybe have ARCH_OMAP24XX before ARCH_OMAP34XX to be consistent > > how it's done elsewhere. > > > > > >> diff --git a/drivers/net/smc911x.h b/drivers/net/smc911x.h > >> index dad93a8..0f406a0 100644 > >> --- a/drivers/net/smc911x.h > >> +++ b/drivers/net/smc911x.h > >> @@ -47,6 +47,11 @@ > >> #define SMC_USE_32BIT 1 > >> #define SMC_IRQ_SENSE IRQF_TRIGGER_LOW > >> #define SMC_MEM_RESERVED 1 > >> +#elif defined(CONFIG_ARCH_OMAP24XX) > >> + #define SMC_USE_16BIT 0 > >> + #define SMC_USE_32BIT 1 > >> + #define SMC_IRQ_SENSE IRQF_TRIGGER_LOW > >> + #define SMC_MEM_RESERVED 1 > Here i used the tab itself, when this patch is applied it comes pretty fine in > the code. I checked the patch with checkpatch.pl also. Can you tell me > what is going wrong? > > After code formating issues are resolved i can resend the patch again. OK, I was just looking the patch. I trust you on the formatting issue :) Tony > > Regards, > Arun > >> #endif > > > > You should probably use tabs here in the new code even if old code > > does not. > > > > Regards, > > > > Tony > > > >> > >> > >> -- > >> 1.5.3.4 > > > >> From 8b45bfd40aca7d5f89195f95d668bae78e2dbba9 Mon Sep 17 00:00:00 2001 > >> From: Arun KS > >> Date: Tue, 24 Jun 2008 17:33:08 +0530 > >> Subject: [PATCH] net:smc911x Modify driver to also work with omap24xx > >> > >> net:smc911x Modify driver to also work with omap24xx > >> > >> Signed-off-by: Arun KS > >> --- > >> drivers/net/Kconfig | 2 +- > >> drivers/net/smc911x.h | 5 +++++ > >> 2 files changed, 6 insertions(+), 1 deletions(-) > >> > >> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig > >> index 61ecee7..17d0a9c 100644 > >> --- a/drivers/net/Kconfig > >> +++ b/drivers/net/Kconfig > >> @@ -955,7 +955,7 @@ config SMC911X > >> tristate "SMSC LAN911[5678] support" > >> select CRC32 > >> select MII > >> - depends on ARCH_PXA || SH_MAGIC_PANEL_R2 || ARCH_OMAP34XX > >> + depends on ARCH_PXA || SH_MAGIC_PANEL_R2 || ARCH_OMAP34XX || ARCH_OMAP24XX > >> help > >> This is a driver for SMSC's LAN911x series of Ethernet chipsets > >> including the new LAN9115, LAN9116, LAN9117, and LAN9118. > >> diff --git a/drivers/net/smc911x.h b/drivers/net/smc911x.h > >> index dad93a8..0f406a0 100644 > >> --- a/drivers/net/smc911x.h > >> +++ b/drivers/net/smc911x.h > >> @@ -47,6 +47,11 @@ > >> #define SMC_USE_32BIT 1 > >> #define SMC_IRQ_SENSE IRQF_TRIGGER_LOW > >> #define SMC_MEM_RESERVED 1 > >> +#elif defined(CONFIG_ARCH_OMAP24XX) > >> + #define SMC_USE_16BIT 0 > >> + #define SMC_USE_32BIT 1 > >> + #define SMC_IRQ_SENSE IRQF_TRIGGER_LOW > >> + #define SMC_MEM_RESERVED 1 > >> #endif > >> > >> > >> -- > >> 1.5.3.4 > >> > > > >