* [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440
@ 2009-06-07 13:04 Peter Korsgaard
2009-06-07 13:04 ` [PATCH] mtd/nand: s3c6400 support for s3c2410 driver Peter Korsgaard
` (2 more replies)
0 siblings, 3 replies; 12+ messages in thread
From: Peter Korsgaard @ 2009-06-07 13:04 UTC (permalink / raw)
To: ben-linux, David.Woodhouse, linux-mtd, linux-arm-kernel; +Cc: wookey
Use the correct S3C2440_NFCONF_* macros for the mask for the 2412/2440
variants instead of the 2410 ones which use wrong bit positions.
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
drivers/mtd/nand/s3c2410.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c
index 89b7905..01a105e 100644
--- a/drivers/mtd/nand/s3c2410.c
+++ b/drivers/mtd/nand/s3c2410.c
@@ -253,9 +253,9 @@ static int s3c2410_nand_setrate(struct s3c2410_nand_info *info)
case TYPE_S3C2440:
case TYPE_S3C2412:
- mask = (S3C2410_NFCONF_TACLS(tacls_max - 1) |
- S3C2410_NFCONF_TWRPH0(7) |
- S3C2410_NFCONF_TWRPH1(7));
+ mask = (S3C2440_NFCONF_TACLS(tacls_max - 1) |
+ S3C2440_NFCONF_TWRPH0(7) |
+ S3C2440_NFCONF_TWRPH1(7));
set = S3C2440_NFCONF_TACLS(tacls - 1);
set |= S3C2440_NFCONF_TWRPH0(twrph0 - 1);
--
1.6.2
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH] mtd/nand: s3c6400 support for s3c2410 driver
2009-06-07 13:04 [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440 Peter Korsgaard
@ 2009-06-07 13:04 ` Peter Korsgaard
2009-06-08 11:11 ` Ben Dooks
2009-06-08 11:41 ` David Woodhouse
2009-06-08 5:28 ` [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440 Artem Bityutskiy
2009-06-08 11:20 ` Ben Dooks
2 siblings, 2 replies; 12+ messages in thread
From: Peter Korsgaard @ 2009-06-07 13:04 UTC (permalink / raw)
To: ben-linux, David.Woodhouse, linux-mtd, linux-arm-kernel; +Cc: wookey
Add s3c6400 support to the s3c2410 driver. The nand controller in
the s3c64xx devices is compatible with the one in the s3c2412, so
simply reuse that code.
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
drivers/mtd/nand/Kconfig | 18 +++++++++---------
drivers/mtd/nand/s3c2410.c | 3 +++
2 files changed, 12 insertions(+), 9 deletions(-)
diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
index 890936d..4d63d7b 100644
--- a/drivers/mtd/nand/Kconfig
+++ b/drivers/mtd/nand/Kconfig
@@ -139,27 +139,27 @@ config MTD_NAND_PPCHAMELEONEVB
This enables the NAND flash driver on the PPChameleon EVB Board.
config MTD_NAND_S3C2410
- tristate "NAND Flash support for S3C2410/S3C2440 SoC"
- depends on ARCH_S3C2410
+ tristate "NAND Flash support for Samsung S3C SoCs"
+ depends on ARCH_S3C2410 || ARCH_S3C64XX
help
- This enables the NAND flash controller on the S3C2410 and S3C2440
+ This enables the NAND flash controller on the S3C24xx and S3C64xx
SoCs
No board specific support is done by this driver, each board
must advertise a platform_device for the driver to attach.
config MTD_NAND_S3C2410_DEBUG
- bool "S3C2410 NAND driver debug"
+ bool "Samsung S3C NAND driver debug"
depends on MTD_NAND_S3C2410
help
- Enable debugging of the S3C2410 NAND driver
+ Enable debugging of the S3C NAND driver
config MTD_NAND_S3C2410_HWECC
- bool "S3C2410 NAND Hardware ECC"
+ bool "Samsung S3C NAND Hardware ECC"
depends on MTD_NAND_S3C2410
help
- Enable the use of the S3C2410's internal ECC generator when
- using NAND. Early versions of the chip have had problems with
+ Enable the use of the controller's internal ECC generator when
+ using NAND. Early versions of the chips have had problems with
incorrect ECC generation, and if using these, the default of
software ECC is preferable.
@@ -171,7 +171,7 @@ config MTD_NAND_NDFC
NDFC Nand Flash Controllers are integrated in IBM/AMCC's 4xx SoCs
config MTD_NAND_S3C2410_CLKSTOP
- bool "S3C2410 NAND IDLE clock stop"
+ bool "Samsung S3C NAND IDLE clock stop"
depends on MTD_NAND_S3C2410
default n
help
diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c
index 01a105e..11dc7e6 100644
--- a/drivers/mtd/nand/s3c2410.c
+++ b/drivers/mtd/nand/s3c2410.c
@@ -1111,6 +1111,9 @@ static struct platform_device_id s3c24xx_driver_ids[] = {
}, {
.name = "s3c2412-nand",
.driver_data = TYPE_S3C2412,
+ }, {
+ .name = "s3c6400-nand",
+ .driver_data = TYPE_S3C2412, /* compatible with 2412 */
},
{ }
};
--
1.6.2
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440
2009-06-07 13:04 [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440 Peter Korsgaard
2009-06-07 13:04 ` [PATCH] mtd/nand: s3c6400 support for s3c2410 driver Peter Korsgaard
@ 2009-06-08 5:28 ` Artem Bityutskiy
2009-06-08 5:38 ` Peter Korsgaard
2009-06-08 11:20 ` Ben Dooks
2 siblings, 1 reply; 12+ messages in thread
From: Artem Bityutskiy @ 2009-06-08 5:28 UTC (permalink / raw)
To: Peter Korsgaard
Cc: wookey, linux-mtd, David.Woodhouse, ben-linux, linux-arm-kernel
On Sun, 2009-06-07 at 15:04 +0200, Peter Korsgaard wrote:
> Use the correct S3C2440_NFCONF_* macros for the mask for the 2412/2440
> variants instead of the 2410 ones which use wrong bit positions.
>
> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Your patch is already in the mtd-2.6.git tree.
--
Best regards,
Artem Bityutskiy (Битюцкий Артём)
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440
2009-06-08 5:28 ` [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440 Artem Bityutskiy
@ 2009-06-08 5:38 ` Peter Korsgaard
2009-06-08 5:47 ` Artem Bityutskiy
0 siblings, 1 reply; 12+ messages in thread
From: Peter Korsgaard @ 2009-06-08 5:38 UTC (permalink / raw)
To: dedekind; +Cc: wookey, linux-mtd, David.Woodhouse, ben-linux, linux-arm-kernel
>>>>> "Artem" == Artem Bityutskiy <dedekind@infradead.org> writes:
Artem> On Sun, 2009-06-07 at 15:04 +0200, Peter Korsgaard wrote:
>> Use the correct S3C2440_NFCONF_* macros for the mask for the 2412/2440
>> variants instead of the 2410 ones which use wrong bit positions.
>>
>> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Artem> Your patch is already in the mtd-2.6.git tree.
Ahh, sorry - Missed that. I resent it as it is a prerequisite to the
s3c64xx support to the driver I sent yesterday.
That patch seems to have never made it to the list though, so I will
resend.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440
2009-06-08 5:38 ` Peter Korsgaard
@ 2009-06-08 5:47 ` Artem Bityutskiy
2009-06-08 6:32 ` Peter Korsgaard
0 siblings, 1 reply; 12+ messages in thread
From: Artem Bityutskiy @ 2009-06-08 5:47 UTC (permalink / raw)
To: Peter Korsgaard
Cc: wookey, linux-mtd, David.Woodhouse, ben-linux, linux-arm-kernel
On Mon, 2009-06-08 at 07:38 +0200, Peter Korsgaard wrote:
> >>>>> "Artem" == Artem Bityutskiy <dedekind@infradead.org> writes:
>
> Artem> On Sun, 2009-06-07 at 15:04 +0200, Peter Korsgaard wrote:
> >> Use the correct S3C2440_NFCONF_* macros for the mask for the 2412/2440
> >> variants instead of the 2410 ones which use wrong bit positions.
> >>
> >> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
>
> Artem> Your patch is already in the mtd-2.6.git tree.
>
> Ahh, sorry - Missed that. I resent it as it is a prerequisite to the
> s3c64xx support to the driver I sent yesterday.
>
> That patch seems to have never made it to the list though, so I will
> resend.
It was trapped, but now approved.
--
Best regards,
Artem Bityutskiy (Битюцкий Артём)
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440
2009-06-08 5:47 ` Artem Bityutskiy
@ 2009-06-08 6:32 ` Peter Korsgaard
2009-06-08 6:41 ` Artem Bityutskiy
0 siblings, 1 reply; 12+ messages in thread
From: Peter Korsgaard @ 2009-06-08 6:32 UTC (permalink / raw)
To: dedekind; +Cc: wookey, linux-mtd, David.Woodhouse, ben-linux, linux-arm-kernel
>>>>> "Artem" == Artem Bityutskiy <dedekind@infradead.org> writes:
Hi,
>> That patch seems to have never made it to the list though, so I
>> will resend.
Artem> It was trapped, but now approved.
Thanks. What did I do wrong to get it trapped?
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440
2009-06-08 6:32 ` Peter Korsgaard
@ 2009-06-08 6:41 ` Artem Bityutskiy
2009-06-08 7:51 ` David Woodhouse
0 siblings, 1 reply; 12+ messages in thread
From: Artem Bityutskiy @ 2009-06-08 6:41 UTC (permalink / raw)
To: Peter Korsgaard
Cc: wookey, linux-mtd, David.Woodhouse, ben-linux, linux-arm-kernel
On Mon, 2009-06-08 at 08:32 +0200, Peter Korsgaard wrote:
> >>>>> "Artem" == Artem Bityutskiy <dedekind@infradead.org> writes:
>
> Hi,
>
> >> That patch seems to have never made it to the list though, so I
> >> will resend.
>
> Artem> It was trapped, but now approved.
>
> Thanks. What did I do wrong to get it trapped?
"Re in subject but no in-reply-to". dwmw2 was going to fix this
few years ago :-)
--
Best regards,
Artem Bityutskiy (Битюцкий Артём)
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440
2009-06-08 6:41 ` Artem Bityutskiy
@ 2009-06-08 7:51 ` David Woodhouse
2009-06-08 7:54 ` Peter Korsgaard
0 siblings, 1 reply; 12+ messages in thread
From: David Woodhouse @ 2009-06-08 7:51 UTC (permalink / raw)
To: dedekind; +Cc: wookey, linux-mtd, linux-arm-kernel, ben-linux
On Sun, 2009-06-07 at 23:41 -0700, Artem Bityutskiy wrote:
>
> "Re in subject but no in-reply-to". dwmw2 was going to fix this
> few years ago :-)
Other way round, surely. It's the thread-hijack check, which checks for
messages with In-Reply-To: or References: headers indicating that
they're a reply to an existing thread, but without 'Re:' or 'Aw:' in the
Subject: header.
In the years since I implemented that basic netiquette check, it's
become acceptable to do exactly that for patch sequences. I thought I'd
fixed it to accept 'PATCH' as well as 'Re:' and 'Aw:', but on looking
closer I see I screwed it up -- it was trying to do a case-insensitive
compare by turning the subject header into lower case before the
comparison.... and then comparing against 'PATCH' in upper case... :)
Fixed now:
http://git.infradead.org/users/dwmw2/eximconf.git?a=commitdiff;h=47505756
--
David Woodhouse Open Source Technology Centre
David.Woodhouse@intel.com Intel Corporation
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440
2009-06-08 7:51 ` David Woodhouse
@ 2009-06-08 7:54 ` Peter Korsgaard
0 siblings, 0 replies; 12+ messages in thread
From: Peter Korsgaard @ 2009-06-08 7:54 UTC (permalink / raw)
To: David Woodhouse; +Cc: wookey, linux-mtd, linux-arm-kernel, ben-linux
>>>>> "David" == David Woodhouse <dwmw2@infradead.org> writes:
Hi,
David> In the years since I implemented that basic netiquette check,
David> it's become acceptable to do exactly that for patch
David> sequences. I thought I'd fixed it to accept 'PATCH' as well as
David> 'Re:' and 'Aw:', but on looking closer I see I screwed it up
David> -- it was trying to do a case-insensitive compare by turning
David> the subject header into lower case before the
David> comparison.... and then comparing against 'PATCH' in upper
David> case... :)
Thanks!
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] mtd/nand: s3c6400 support for s3c2410 driver
2009-06-07 13:04 ` [PATCH] mtd/nand: s3c6400 support for s3c2410 driver Peter Korsgaard
@ 2009-06-08 11:11 ` Ben Dooks
2009-06-08 11:41 ` David Woodhouse
1 sibling, 0 replies; 12+ messages in thread
From: Ben Dooks @ 2009-06-08 11:11 UTC (permalink / raw)
To: Peter Korsgaard
Cc: wookey, linux-mtd, David.Woodhouse, ben-linux, linux-arm-kernel
On Sun, Jun 07, 2009 at 03:04:23PM +0200, Peter Korsgaard wrote:
> Add s3c6400 support to the s3c2410 driver. The nand controller in
> the s3c64xx devices is compatible with the one in the s3c2412, so
> simply reuse that code.
>
> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Acked-by: Ben Dooks <ben-linux@fluff.org>
> ---
> drivers/mtd/nand/Kconfig | 18 +++++++++---------
> drivers/mtd/nand/s3c2410.c | 3 +++
> 2 files changed, 12 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
> index 890936d..4d63d7b 100644
> --- a/drivers/mtd/nand/Kconfig
> +++ b/drivers/mtd/nand/Kconfig
> @@ -139,27 +139,27 @@ config MTD_NAND_PPCHAMELEONEVB
> This enables the NAND flash driver on the PPChameleon EVB Board.
>
> config MTD_NAND_S3C2410
> - tristate "NAND Flash support for S3C2410/S3C2440 SoC"
> - depends on ARCH_S3C2410
> + tristate "NAND Flash support for Samsung S3C SoCs"
> + depends on ARCH_S3C2410 || ARCH_S3C64XX
> help
> - This enables the NAND flash controller on the S3C2410 and S3C2440
> + This enables the NAND flash controller on the S3C24xx and S3C64xx
> SoCs
>
> No board specific support is done by this driver, each board
> must advertise a platform_device for the driver to attach.
>
> config MTD_NAND_S3C2410_DEBUG
> - bool "S3C2410 NAND driver debug"
> + bool "Samsung S3C NAND driver debug"
> depends on MTD_NAND_S3C2410
> help
> - Enable debugging of the S3C2410 NAND driver
> + Enable debugging of the S3C NAND driver
>
> config MTD_NAND_S3C2410_HWECC
> - bool "S3C2410 NAND Hardware ECC"
> + bool "Samsung S3C NAND Hardware ECC"
> depends on MTD_NAND_S3C2410
> help
> - Enable the use of the S3C2410's internal ECC generator when
> - using NAND. Early versions of the chip have had problems with
> + Enable the use of the controller's internal ECC generator when
> + using NAND. Early versions of the chips have had problems with
> incorrect ECC generation, and if using these, the default of
> software ECC is preferable.
>
> @@ -171,7 +171,7 @@ config MTD_NAND_NDFC
> NDFC Nand Flash Controllers are integrated in IBM/AMCC's 4xx SoCs
>
> config MTD_NAND_S3C2410_CLKSTOP
> - bool "S3C2410 NAND IDLE clock stop"
> + bool "Samsung S3C NAND IDLE clock stop"
> depends on MTD_NAND_S3C2410
> default n
> help
> diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c
> index 01a105e..11dc7e6 100644
> --- a/drivers/mtd/nand/s3c2410.c
> +++ b/drivers/mtd/nand/s3c2410.c
> @@ -1111,6 +1111,9 @@ static struct platform_device_id s3c24xx_driver_ids[] = {
> }, {
> .name = "s3c2412-nand",
> .driver_data = TYPE_S3C2412,
> + }, {
> + .name = "s3c6400-nand",
> + .driver_data = TYPE_S3C2412, /* compatible with 2412 */
> },
> { }
> };
> --
> 1.6.2
>
>
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/
--
Ben (ben@fluff.org, http://www.fluff.org/)
'a smiley only costs 4 bytes'
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440
2009-06-07 13:04 [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440 Peter Korsgaard
2009-06-07 13:04 ` [PATCH] mtd/nand: s3c6400 support for s3c2410 driver Peter Korsgaard
2009-06-08 5:28 ` [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440 Artem Bityutskiy
@ 2009-06-08 11:20 ` Ben Dooks
2 siblings, 0 replies; 12+ messages in thread
From: Ben Dooks @ 2009-06-08 11:20 UTC (permalink / raw)
To: Peter Korsgaard
Cc: wookey, linux-mtd, David.Woodhouse, ben-linux, linux-arm-kernel
On Sun, Jun 07, 2009 at 03:04:22PM +0200, Peter Korsgaard wrote:
> Use the correct S3C2440_NFCONF_* macros for the mask for the 2412/2440
> variants instead of the 2410 ones which use wrong bit positions.
>
> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
Acked-by: Ben Dooks <ben-linux@fluff.org>
> ---
> drivers/mtd/nand/s3c2410.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c
> index 89b7905..01a105e 100644
> --- a/drivers/mtd/nand/s3c2410.c
> +++ b/drivers/mtd/nand/s3c2410.c
> @@ -253,9 +253,9 @@ static int s3c2410_nand_setrate(struct s3c2410_nand_info *info)
>
> case TYPE_S3C2440:
> case TYPE_S3C2412:
> - mask = (S3C2410_NFCONF_TACLS(tacls_max - 1) |
> - S3C2410_NFCONF_TWRPH0(7) |
> - S3C2410_NFCONF_TWRPH1(7));
> + mask = (S3C2440_NFCONF_TACLS(tacls_max - 1) |
> + S3C2440_NFCONF_TWRPH0(7) |
> + S3C2440_NFCONF_TWRPH1(7));
>
> set = S3C2440_NFCONF_TACLS(tacls - 1);
> set |= S3C2440_NFCONF_TWRPH0(twrph0 - 1);
> --
> 1.6.2
>
>
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/
--
Ben (ben@fluff.org, http://www.fluff.org/)
'a smiley only costs 4 bytes'
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] mtd/nand: s3c6400 support for s3c2410 driver
2009-06-07 13:04 ` [PATCH] mtd/nand: s3c6400 support for s3c2410 driver Peter Korsgaard
2009-06-08 11:11 ` Ben Dooks
@ 2009-06-08 11:41 ` David Woodhouse
1 sibling, 0 replies; 12+ messages in thread
From: David Woodhouse @ 2009-06-08 11:41 UTC (permalink / raw)
To: Peter Korsgaard; +Cc: wookey, linux-mtd, linux-arm-kernel, ben-linux
On Sun, 2009-06-07 at 06:04 -0700, Peter Korsgaard wrote:
> Add s3c6400 support to the s3c2410 driver. The nand controller in
> the s3c64xx devices is compatible with the one in the s3c2412, so
> simply reuse that code.
>
> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
> ---
> drivers/mtd/nand/Kconfig | 18 +++++++++---------
> drivers/mtd/nand/s3c2410.c | 3 +++
> 2 files changed, 12 insertions(+), 9 deletions(-)
Applied; thanks.
--
David Woodhouse Open Source Technology Centre
David.Woodhouse@intel.com Intel Corporation
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2009-06-08 11:41 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-06-07 13:04 [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440 Peter Korsgaard
2009-06-07 13:04 ` [PATCH] mtd/nand: s3c6400 support for s3c2410 driver Peter Korsgaard
2009-06-08 11:11 ` Ben Dooks
2009-06-08 11:41 ` David Woodhouse
2009-06-08 5:28 ` [PATCH] nand/s3c2410: s3c2410_nand_setrate(): use correct macros for 2412/2440 Artem Bityutskiy
2009-06-08 5:38 ` Peter Korsgaard
2009-06-08 5:47 ` Artem Bityutskiy
2009-06-08 6:32 ` Peter Korsgaard
2009-06-08 6:41 ` Artem Bityutskiy
2009-06-08 7:51 ` David Woodhouse
2009-06-08 7:54 ` Peter Korsgaard
2009-06-08 11:20 ` Ben Dooks
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.