Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1235478 > unrolled thread

Re: [PATCH] mtd: nand: denali: max_banks calculation changed in revision 5.1

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH] mtd: nand: denali: max_banks calculation changed in  revision 5.1 Brian Norris <computersforpeace@gmail.com> - 2015-09-29 20:40 +0200
    Re: [PATCH] mtd: nand: denali: max_banks calculation changed in  revision 5.1 Brian Norris <computersforpeace@gmail.com> - 2015-09-29 22:00 +0200

#1235478 — Re: [PATCH] mtd: nand: denali: max_banks calculation changed in revision 5.1

On Tue, Sep 29, 2015 at 10:01:10AM -0500, Graham Moore wrote:
> Howdy folks,
> 
> Any comment on this patch?

A tiny comment below.

> Is anyone else using Denali NAND rev 5.1
> controller?

I have no devices to test, but CC'ing Enrico, who sent a denali patch
recently.

In general, MTD has a lot of drivers for various hardware, and I have
few of them to test. If anyone is interested in getting CC's (so they
can test, review, and otherwise help expedite the process), I'd be happy
to ack an addition to MAINTAINERS, perhaps as 'R:' (reviewer)...

> Thanks,
> Graham
> 
> On 07/21/2015 09:39 AM, Graham Moore wrote:
> >Read Denali hardware revision number and use it to
> >calculate max_banks,  The encoding of max_banks changed
> >in Denali revision 5.1.
> >
> >Signed-off-by: Graham Moore <grmoore@opensource.altera.com>
> >---
> >  drivers/mtd/nand/denali.c |   11 ++++++++++-
> >  drivers/mtd/nand/denali.h |    2 ++
> >  2 files changed, 12 insertions(+), 1 deletion(-)
> >
> >diff --git a/drivers/mtd/nand/denali.c b/drivers/mtd/nand/denali.c
> >index 870c7fc..a98b41e 100644
> >--- a/drivers/mtd/nand/denali.c
> >+++ b/drivers/mtd/nand/denali.c
> >@@ -458,8 +458,17 @@ static void find_valid_banks(struct denali_nand_info *denali)
> >  static void detect_max_banks(struct denali_nand_info *denali)
> >  {
> >  	uint32_t features = ioread32(denali->flash_reg + FEATURES);
> >+	/*
> >+	 * Read the revision register, so we can calculate the max_banks
> >+	 * properly: the encoding changed from rev 5.0 to 5.1
> >+	 */
> >+	u32 revision = MAKE_COMPARABLE_REVISION(
> >+				ioread32(denali->flash_reg + REVISION));
> >
> >-	denali->max_banks = 2 << (features & FEATURES__N_BANKS);
> >+	if (revision < REVISION_5_1)
> >+		denali->max_banks = 2 << (features & FEATURES__N_BANKS);
> >+	else
> >+		denali->max_banks = 1 << (features & FEATURES__N_BANKS);
> >  }
> >
> >  static void detect_partition_feature(struct denali_nand_info *denali)
> >diff --git a/drivers/mtd/nand/denali.h b/drivers/mtd/nand/denali.h
> >index 145bf88..4bf13bb 100644
> >--- a/drivers/mtd/nand/denali.h
> >+++ b/drivers/mtd/nand/denali.h
> >@@ -178,6 +178,8 @@
> >
> >  #define REVISION				0x370
> >  #define     REVISION__VALUE				0xffff
> >+#define MAKE_COMPARABLE_REVISION(x)		swab16(x & REVISION__VALUE)

It's probably best to add parentheses around the 'x'.

> >+#define REVISION_5_1				0x00000501
> >
> >  #define ONFI_DEVICE_FEATURES			0x380
> >  #define     ONFI_DEVICE_FEATURES__VALUE			0x003f
> >
> 

I can fix that up and apply, if no objections.

Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1235515

On Tue, Sep 29, 2015 at 11:31:21AM -0700, Brian Norris wrote:
> On Tue, Sep 29, 2015 at 10:01:10AM -0500, Graham Moore wrote:
> > On 07/21/2015 09:39 AM, Graham Moore wrote:
> > >diff --git a/drivers/mtd/nand/denali.c b/drivers/mtd/nand/denali.c
> > >index 870c7fc..a98b41e 100644
> > >--- a/drivers/mtd/nand/denali.h
> > >+++ b/drivers/mtd/nand/denali.h
> > >@@ -178,6 +178,8 @@
> > >
> > >  #define REVISION				0x370
> > >  #define     REVISION__VALUE				0xffff
> > >+#define MAKE_COMPARABLE_REVISION(x)		swab16(x & REVISION__VALUE)
> 
> It's probably best to add parentheses around the 'x'.
> 
> > >+#define REVISION_5_1				0x00000501
> > >
> > >  #define ONFI_DEVICE_FEATURES			0x380
> > >  #define     ONFI_DEVICE_FEATURES__VALUE			0x003f
> > >
> > 
> 
> I can fix that up and apply, if no objections.

Pushed to l2-mtd.git with that fixup.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web