* [ PATCH 2/3 ] net:smc911x Modify driver to also work with omap24xx
@ 2008-06-24 12:14 Arun KS
2008-06-26 12:22 ` Tony Lindgren
0 siblings, 1 reply; 4+ messages in thread
From: Arun KS @ 2008-06-24 12:14 UTC (permalink / raw)
To: linux-omap
[-- Attachment #1: Type: text/plain, Size: 1242 bytes --]
net:smc911x Modify driver to also work with omap24xx
Signed-off-by: Arun KS <arunks@mistralsolutions.com>
---
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
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0002-net-smc911x-Modify-driver-to-also-work-with-omap24xx.patch --]
[-- Type: text/x-patch; name=0002-net-smc911x-Modify-driver-to-also-work-with-omap24xx.patch, Size: 1424 bytes --]
From 8b45bfd40aca7d5f89195f95d668bae78e2dbba9 Mon Sep 17 00:00:00 2001
From: Arun KS <arunks@mistralsolutions.com>
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@mistralsolutions.com>
---
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
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [ PATCH 2/3 ] net:smc911x Modify driver to also work with omap24xx
2008-06-24 12:14 [ PATCH 2/3 ] net:smc911x Modify driver to also work with omap24xx Arun KS
@ 2008-06-26 12:22 ` Tony Lindgren
2008-06-26 15:17 ` Arun KS
0 siblings, 1 reply; 4+ messages in thread
From: Tony Lindgren @ 2008-06-26 12:22 UTC (permalink / raw)
To: Arun KS; +Cc: linux-omap
Hi,
* Arun KS <getarunks@gmail.com> [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@mistralsolutions.com>
> ---
> 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
> #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@mistralsolutions.com>
> 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@mistralsolutions.com>
> ---
> 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
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [ PATCH 2/3 ] net:smc911x Modify driver to also work with omap24xx
2008-06-26 12:22 ` Tony Lindgren
@ 2008-06-26 15:17 ` Arun KS
2008-07-03 10:14 ` Tony Lindgren
0 siblings, 1 reply; 4+ messages in thread
From: Arun KS @ 2008-06-26 15:17 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap
On Thu, Jun 26, 2008 at 5:52 PM, Tony Lindgren <tony@atomide.com> wrote:
> Hi,
>
> * Arun KS <getarunks@gmail.com> [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@mistralsolutions.com>
>> ---
>> 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.
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@mistralsolutions.com>
>> 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@mistralsolutions.com>
>> ---
>> 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
>>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [ PATCH 2/3 ] net:smc911x Modify driver to also work with omap24xx
2008-06-26 15:17 ` Arun KS
@ 2008-07-03 10:14 ` Tony Lindgren
0 siblings, 0 replies; 4+ messages in thread
From: Tony Lindgren @ 2008-07-03 10:14 UTC (permalink / raw)
To: Arun KS; +Cc: linux-omap
* Arun KS <getarunks@gmail.com> [080626 18:17]:
> On Thu, Jun 26, 2008 at 5:52 PM, Tony Lindgren <tony@atomide.com> wrote:
> > Hi,
> >
> > * Arun KS <getarunks@gmail.com> [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@mistralsolutions.com>
> >> ---
> >> 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@mistralsolutions.com>
> >> 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@mistralsolutions.com>
> >> ---
> >> 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
> >>
> >
> >
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2008-07-03 12:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-06-24 12:14 [ PATCH 2/3 ] net:smc911x Modify driver to also work with omap24xx Arun KS
2008-06-26 12:22 ` Tony Lindgren
2008-06-26 15:17 ` Arun KS
2008-07-03 10:14 ` Tony Lindgren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox