* Arun KS <getarunks@xxxxxxxxx> [080626 18:17]: > On Thu, Jun 26, 2008 at 5:52 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote: > > Hi, > > > > * Arun KS <getarunks@xxxxxxxxx> [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 <arunks@xxxxxxxxxxxxxxxxxxxx> > >> --- > >> 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 <arunks@xxxxxxxxxxxxxxxxxxxx> > >> 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 <arunks@xxxxxxxxxxxxxxxxxxxx> > >> --- > >> 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 > >> > > > > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html