2018-10-25 06:45:20

by Mason Yang

[permalink] [raw]
Subject: [PATCH] mtd: rawnand: Add all 1.8V AC chips have a broken GET_FEATURES(TIMINGS)

From: Mason Yang <[email protected]>

Hi Boris,
I patched this for Macronix all 1.8V AC chips.
Thanks for your review.

best regards,
Mason

Mason Yang (1):
mtd: rawnand: Add All 1.8V AC chips have a broken
GET_FEATURES(TIMINGS)

drivers/mtd/nand/raw/nand_macronix.c | 7 +++++++
1 file changed, 7 insertions(+)

--
1.9.1



2018-10-25 06:46:49

by Mason Yang

[permalink] [raw]
Subject: [PATCH] mtd: rawnand: Add All 1.8V AC chips have a broken GET_FEATURES(TIMINGS)

From: Mason Yang <[email protected]>

Make sure we flag all 1.8V broken chips as not supporting this features.

Signed-off-by: Mason Yang <[email protected]>
---
drivers/mtd/nand/raw/nand_macronix.c | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/drivers/mtd/nand/raw/nand_macronix.c b/drivers/mtd/nand/raw/nand_macronix.c
index 49c546c..3836544 100644
--- a/drivers/mtd/nand/raw/nand_macronix.c
+++ b/drivers/mtd/nand/raw/nand_macronix.c
@@ -33,6 +33,13 @@ static void macronix_nand_fix_broken_get_timings(struct nand_chip *chip)
"MX30LF4G18AC",
"MX30LF4G28AC",
"MX60LF8G18AC",
+ "MX30UF1G18AC",
+ "MX30UF1G16AC",
+ "MX30UF2G18AC",
+ "MX30UF2G16AC",
+ "MX30UF4G18AC",
+ "MX30UF4G16AC",
+ "MX30UF4G28AC",
};

if (!chip->parameters.supports_set_get_features)
--
1.9.1


2018-10-25 07:17:35

by Boris Brezillon

[permalink] [raw]
Subject: Re: [PATCH] mtd: rawnand: Add all 1.8V AC chips have a broken GET_FEATURES(TIMINGS)

Hi Mason,

On Thu, 25 Oct 2018 14:44:30 +0800
[email protected] wrote:

> From: Mason Yang <[email protected]>
>
> Hi Boris,
> I patched this for Macronix all 1.8V AC chips.
> Thanks for your review.

No need to add a cover letter when you only have patch.

> best regards,
> Mason
>
> Mason Yang (1):
> mtd: rawnand: Add All 1.8V AC chips have a broken
> GET_FEATURES(TIMINGS)
>
> drivers/mtd/nand/raw/nand_macronix.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>


2018-10-25 07:18:56

by Boris Brezillon

[permalink] [raw]
Subject: Re: [PATCH] mtd: rawnand: Add all 1.8V AC chips have a broken GET_FEATURES(TIMINGS)

On Thu, 25 Oct 2018 09:16:53 +0200
Boris Brezillon <[email protected]> wrote:

> Hi Mason,
>
> On Thu, 25 Oct 2018 14:44:30 +0800
> [email protected] wrote:
>
> > From: Mason Yang <[email protected]>
> >
> > Hi Boris,
> > I patched this for Macronix all 1.8V AC chips.
> > Thanks for your review.
>
> No need to add a cover letter when you only have patch.

^ one

>
> > best regards,
> > Mason
> >
> > Mason Yang (1):
> > mtd: rawnand: Add All 1.8V AC chips have a broken
> > GET_FEATURES(TIMINGS)
> >
> > drivers/mtd/nand/raw/nand_macronix.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
>


2018-11-11 20:56:30

by Miquel Raynal

[permalink] [raw]
Subject: Re: [PATCH] mtd: rawnand: Add All 1.8V AC chips have a broken GET_FEATURES(TIMINGS)

Hi Mason,

[email protected] wrote on Thu, 25 Oct 2018 14:44:31 +0800:

> From: Mason Yang <[email protected]>
>
> Make sure we flag all 1.8V broken chips as not supporting this features.
>
> Signed-off-by: Mason Yang <[email protected]>
> ---

Applied to nand/next (with the title slightly modified).

Thanks,
Miquèl