* [PATCH v2 0/2] Add big endian and ONFI support to iProc NAND
@ 2016-07-20 21:53 Ray Jui
2016-07-20 21:53 ` [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support Ray Jui
2016-07-20 21:53 ` [PATCH v2 2/2] arm64: dts: Updated NAND DT properties for NS2 SVK Ray Jui
0 siblings, 2 replies; 10+ messages in thread
From: Ray Jui @ 2016-07-20 21:53 UTC (permalink / raw)
To: linux-arm-kernel
This patch series adds big endian and ONFI support to iProc based NAND wrapper
that utilizes the BRCMSTB NAND core driver. It also adds missing NAND DT
properties for NS2 SVK
This patch series is based off v4.7-rc1 and is available on GIHUB:
repo: https://github.com/Broadcom/arm64-linux.git
branch: iproc-nand-be-v2
Changes from v1:
- Fixed minor format by adding braces around the 'else'
Ray Jui (2):
mtd: brcmnand: iProc big endian and ONFI support
arm64: dts: Updated NAND DT properties for NS2 SVK
arch/arm64/boot/dts/broadcom/ns2-svk.dts | 2 ++
drivers/mtd/nand/brcmnand/brcmnand.c | 12 ++++++------
drivers/mtd/nand/brcmnand/brcmnand.h | 13 ++++++++-----
drivers/mtd/nand/brcmnand/iproc_nand.c | 18 ++++++++++++++----
4 files changed, 30 insertions(+), 15 deletions(-)
--
2.1.4
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support
2016-07-20 21:53 [PATCH v2 0/2] Add big endian and ONFI support to iProc NAND Ray Jui
@ 2016-07-20 21:53 ` Ray Jui
2016-08-10 14:24 ` Boris Brezillon
2016-08-26 7:47 ` Boris Brezillon
2016-07-20 21:53 ` [PATCH v2 2/2] arm64: dts: Updated NAND DT properties for NS2 SVK Ray Jui
1 sibling, 2 replies; 10+ messages in thread
From: Ray Jui @ 2016-07-20 21:53 UTC (permalink / raw)
To: linux-arm-kernel
This patch adds big endian and ONFI support for various iProc based
SoCs that use the core brcmstb NAND controller
This patch was originally implemented by Prafulla Kota
<prafulla.kota@broadcom.com> and fully tested on iProc based NS2 SVK
Signed-off-by: Prafulla Kota <prafulla.kota@broadcom.com>
Signed-off-by: Ray Jui <ray.jui@broadcom.com>
---
drivers/mtd/nand/brcmnand/brcmnand.c | 12 ++++++------
drivers/mtd/nand/brcmnand/brcmnand.h | 13 ++++++++-----
drivers/mtd/nand/brcmnand/iproc_nand.c | 18 ++++++++++++++----
3 files changed, 28 insertions(+), 15 deletions(-)
diff --git a/drivers/mtd/nand/brcmnand/brcmnand.c b/drivers/mtd/nand/brcmnand/brcmnand.c
index b76ad7c..12a1585 100644
--- a/drivers/mtd/nand/brcmnand/brcmnand.c
+++ b/drivers/mtd/nand/brcmnand/brcmnand.c
@@ -1279,7 +1279,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
u32 *flash_cache = (u32 *)ctrl->flash_cache;
int i;
- brcmnand_soc_data_bus_prepare(ctrl->soc);
+ brcmnand_soc_data_bus_prepare(ctrl->soc, true);
/*
* Must cache the FLASH_CACHE now, since changes in
@@ -1292,7 +1292,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
*/
flash_cache[i] = be32_to_cpu(brcmnand_read_fc(ctrl, i));
- brcmnand_soc_data_bus_unprepare(ctrl->soc);
+ brcmnand_soc_data_bus_unprepare(ctrl->soc, true);
/* Cleanup from HW quirk: restore SECTOR_SIZE_1K */
if (host->hwcfg.sector_size_1k)
@@ -1508,12 +1508,12 @@ static int brcmnand_read_by_pio(struct mtd_info *mtd, struct nand_chip *chip,
brcmnand_waitfunc(mtd, chip);
if (likely(buf)) {
- brcmnand_soc_data_bus_prepare(ctrl->soc);
+ brcmnand_soc_data_bus_prepare(ctrl->soc, false);
for (j = 0; j < FC_WORDS; j++, buf++)
*buf = brcmnand_read_fc(ctrl, j);
- brcmnand_soc_data_bus_unprepare(ctrl->soc);
+ brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
}
if (oob)
@@ -1678,12 +1678,12 @@ static int brcmnand_write(struct mtd_info *mtd, struct nand_chip *chip,
(void)brcmnand_read_reg(ctrl, BRCMNAND_CMD_ADDRESS);
if (buf) {
- brcmnand_soc_data_bus_prepare(ctrl->soc);
+ brcmnand_soc_data_bus_prepare(ctrl->soc, false);
for (j = 0; j < FC_WORDS; j++, buf++)
brcmnand_write_fc(ctrl, j, *buf);
- brcmnand_soc_data_bus_unprepare(ctrl->soc);
+ brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
} else if (oob) {
for (j = 0; j < FC_WORDS; j++)
brcmnand_write_fc(ctrl, j, 0xffffffff);
diff --git a/drivers/mtd/nand/brcmnand/brcmnand.h b/drivers/mtd/nand/brcmnand/brcmnand.h
index ef5eabb..5c44cd4 100644
--- a/drivers/mtd/nand/brcmnand/brcmnand.h
+++ b/drivers/mtd/nand/brcmnand/brcmnand.h
@@ -23,19 +23,22 @@ struct dev_pm_ops;
struct brcmnand_soc {
bool (*ctlrdy_ack)(struct brcmnand_soc *soc);
void (*ctlrdy_set_enabled)(struct brcmnand_soc *soc, bool en);
- void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare);
+ void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare,
+ bool is_param);
};
-static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc)
+static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc,
+ bool is_param)
{
if (soc && soc->prepare_data_bus)
- soc->prepare_data_bus(soc, true);
+ soc->prepare_data_bus(soc, true, is_param);
}
-static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc)
+static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc,
+ bool is_param)
{
if (soc && soc->prepare_data_bus)
- soc->prepare_data_bus(soc, false);
+ soc->prepare_data_bus(soc, false, is_param);
}
static inline u32 brcmnand_readl(void __iomem *addr)
diff --git a/drivers/mtd/nand/brcmnand/iproc_nand.c b/drivers/mtd/nand/brcmnand/iproc_nand.c
index 585596c..4c6ae11 100644
--- a/drivers/mtd/nand/brcmnand/iproc_nand.c
+++ b/drivers/mtd/nand/brcmnand/iproc_nand.c
@@ -74,7 +74,8 @@ static void iproc_nand_intc_set(struct brcmnand_soc *soc, bool en)
spin_unlock_irqrestore(&priv->idm_lock, flags);
}
-static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
+static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare,
+ bool is_param)
{
struct iproc_nand_soc *priv =
container_of(soc, struct iproc_nand_soc, soc);
@@ -86,10 +87,19 @@ static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
val = brcmnand_readl(mmio);
- if (prepare)
- val |= IPROC_NAND_APB_LE_MODE;
- else
+ /*
+ * In the case of BE or when dealing with NAND data, alway configure
+ * the APB bus to LE mode before accessing the FIFO and back to BE mode
+ * after the access is done
+ */
+ if (IS_ENABLED(CONFIG_CPU_BIG_ENDIAN) || !is_param) {
+ if (prepare)
+ val |= IPROC_NAND_APB_LE_MODE;
+ else
+ val &= ~IPROC_NAND_APB_LE_MODE;
+ } else { /* when in LE accessing the parameter page, keep APB in BE */
val &= ~IPROC_NAND_APB_LE_MODE;
+ }
brcmnand_writel(val, mmio);
--
2.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 2/2] arm64: dts: Updated NAND DT properties for NS2 SVK
2016-07-20 21:53 [PATCH v2 0/2] Add big endian and ONFI support to iProc NAND Ray Jui
2016-07-20 21:53 ` [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support Ray Jui
@ 2016-07-20 21:53 ` Ray Jui
2016-10-17 20:49 ` Ray Jui
1 sibling, 1 reply; 10+ messages in thread
From: Ray Jui @ 2016-07-20 21:53 UTC (permalink / raw)
To: linux-arm-kernel
This patch adds NAND DT properties for NS2 SVK to configure the bus
width width and OOB sector size
Signed-off-by: Prafulla Kota <prafulla.kota@broadcom.com>
Signed-off-by: Ray Jui <ray.jui@broadcom.com>
---
arch/arm64/boot/dts/broadcom/ns2-svk.dts | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/broadcom/ns2-svk.dts b/arch/arm64/boot/dts/broadcom/ns2-svk.dts
index 54ca40c..5ef0407 100644
--- a/arch/arm64/boot/dts/broadcom/ns2-svk.dts
+++ b/arch/arm64/boot/dts/broadcom/ns2-svk.dts
@@ -128,6 +128,8 @@
nand-ecc-mode = "hw";
nand-ecc-strength = <8>;
nand-ecc-step-size = <512>;
+ nand-bus-width = <16>;
+ brcm,nand-oob-sector-size = <16>;
#address-cells = <1>;
#size-cells = <1>;
};
--
2.1.4
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support
2016-07-20 21:53 ` [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support Ray Jui
@ 2016-08-10 14:24 ` Boris Brezillon
2016-08-19 15:14 ` Kamal Dasu
2016-08-22 15:42 ` Kamal Dasu
2016-08-26 7:47 ` Boris Brezillon
1 sibling, 2 replies; 10+ messages in thread
From: Boris Brezillon @ 2016-08-10 14:24 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, 20 Jul 2016 14:53:50 -0700
Ray Jui <ray.jui@broadcom.com> wrote:
> This patch adds big endian and ONFI support for various iProc based
> SoCs that use the core brcmstb NAND controller
Brian, Kamal, can you review this patch?
>
> This patch was originally implemented by Prafulla Kota
> <prafulla.kota@broadcom.com> and fully tested on iProc based NS2 SVK
>
> Signed-off-by: Prafulla Kota <prafulla.kota@broadcom.com>
> Signed-off-by: Ray Jui <ray.jui@broadcom.com>
> ---
> drivers/mtd/nand/brcmnand/brcmnand.c | 12 ++++++------
> drivers/mtd/nand/brcmnand/brcmnand.h | 13 ++++++++-----
> drivers/mtd/nand/brcmnand/iproc_nand.c | 18 ++++++++++++++----
> 3 files changed, 28 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.c b/drivers/mtd/nand/brcmnand/brcmnand.c
> index b76ad7c..12a1585 100644
> --- a/drivers/mtd/nand/brcmnand/brcmnand.c
> +++ b/drivers/mtd/nand/brcmnand/brcmnand.c
> @@ -1279,7 +1279,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
> u32 *flash_cache = (u32 *)ctrl->flash_cache;
> int i;
>
> - brcmnand_soc_data_bus_prepare(ctrl->soc);
> + brcmnand_soc_data_bus_prepare(ctrl->soc, true);
>
> /*
> * Must cache the FLASH_CACHE now, since changes in
> @@ -1292,7 +1292,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
> */
> flash_cache[i] = be32_to_cpu(brcmnand_read_fc(ctrl, i));
>
> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
> + brcmnand_soc_data_bus_unprepare(ctrl->soc, true);
>
> /* Cleanup from HW quirk: restore SECTOR_SIZE_1K */
> if (host->hwcfg.sector_size_1k)
> @@ -1508,12 +1508,12 @@ static int brcmnand_read_by_pio(struct mtd_info *mtd, struct nand_chip *chip,
> brcmnand_waitfunc(mtd, chip);
>
> if (likely(buf)) {
> - brcmnand_soc_data_bus_prepare(ctrl->soc);
> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
>
> for (j = 0; j < FC_WORDS; j++, buf++)
> *buf = brcmnand_read_fc(ctrl, j);
>
> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
> }
>
> if (oob)
> @@ -1678,12 +1678,12 @@ static int brcmnand_write(struct mtd_info *mtd, struct nand_chip *chip,
> (void)brcmnand_read_reg(ctrl, BRCMNAND_CMD_ADDRESS);
>
> if (buf) {
> - brcmnand_soc_data_bus_prepare(ctrl->soc);
> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
>
> for (j = 0; j < FC_WORDS; j++, buf++)
> brcmnand_write_fc(ctrl, j, *buf);
>
> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
> } else if (oob) {
> for (j = 0; j < FC_WORDS; j++)
> brcmnand_write_fc(ctrl, j, 0xffffffff);
> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.h b/drivers/mtd/nand/brcmnand/brcmnand.h
> index ef5eabb..5c44cd4 100644
> --- a/drivers/mtd/nand/brcmnand/brcmnand.h
> +++ b/drivers/mtd/nand/brcmnand/brcmnand.h
> @@ -23,19 +23,22 @@ struct dev_pm_ops;
> struct brcmnand_soc {
> bool (*ctlrdy_ack)(struct brcmnand_soc *soc);
> void (*ctlrdy_set_enabled)(struct brcmnand_soc *soc, bool en);
> - void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare);
> + void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare,
> + bool is_param);
> };
>
> -static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc)
> +static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc,
> + bool is_param)
> {
> if (soc && soc->prepare_data_bus)
> - soc->prepare_data_bus(soc, true);
> + soc->prepare_data_bus(soc, true, is_param);
> }
>
> -static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc)
> +static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc,
> + bool is_param)
> {
> if (soc && soc->prepare_data_bus)
> - soc->prepare_data_bus(soc, false);
> + soc->prepare_data_bus(soc, false, is_param);
> }
>
> static inline u32 brcmnand_readl(void __iomem *addr)
> diff --git a/drivers/mtd/nand/brcmnand/iproc_nand.c b/drivers/mtd/nand/brcmnand/iproc_nand.c
> index 585596c..4c6ae11 100644
> --- a/drivers/mtd/nand/brcmnand/iproc_nand.c
> +++ b/drivers/mtd/nand/brcmnand/iproc_nand.c
> @@ -74,7 +74,8 @@ static void iproc_nand_intc_set(struct brcmnand_soc *soc, bool en)
> spin_unlock_irqrestore(&priv->idm_lock, flags);
> }
>
> -static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
> +static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare,
> + bool is_param)
> {
> struct iproc_nand_soc *priv =
> container_of(soc, struct iproc_nand_soc, soc);
> @@ -86,10 +87,19 @@ static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
>
> val = brcmnand_readl(mmio);
>
> - if (prepare)
> - val |= IPROC_NAND_APB_LE_MODE;
> - else
> + /*
> + * In the case of BE or when dealing with NAND data, alway configure
> + * the APB bus to LE mode before accessing the FIFO and back to BE mode
> + * after the access is done
> + */
> + if (IS_ENABLED(CONFIG_CPU_BIG_ENDIAN) || !is_param) {
> + if (prepare)
> + val |= IPROC_NAND_APB_LE_MODE;
> + else
> + val &= ~IPROC_NAND_APB_LE_MODE;
> + } else { /* when in LE accessing the parameter page, keep APB in BE */
> val &= ~IPROC_NAND_APB_LE_MODE;
> + }
>
> brcmnand_writel(val, mmio);
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support
2016-08-10 14:24 ` Boris Brezillon
@ 2016-08-19 15:14 ` Kamal Dasu
2016-08-22 7:44 ` Boris Brezillon
2016-08-22 15:42 ` Kamal Dasu
1 sibling, 1 reply; 10+ messages in thread
From: Kamal Dasu @ 2016-08-19 15:14 UTC (permalink / raw)
To: linux-arm-kernel
Boris
This change looks good to me.
On Wed, Aug 10, 2016 at 10:24 AM, Boris Brezillon
<boris.brezillon@free-electrons.com> wrote:
> On Wed, 20 Jul 2016 14:53:50 -0700
> Ray Jui <ray.jui@broadcom.com> wrote:
>
>> This patch adds big endian and ONFI support for various iProc based
>> SoCs that use the core brcmstb NAND controller
>
> Brian, Kamal, can you review this patch?
>
>>
>> This patch was originally implemented by Prafulla Kota
>> <prafulla.kota@broadcom.com> and fully tested on iProc based NS2 SVK
>>
>> Signed-off-by: Prafulla Kota <prafulla.kota@broadcom.com>
>> Signed-off-by: Ray Jui <ray.jui@broadcom.com>
Reviewed-by: Kamal Dasu <kdasu.kdev@gmail.com>
>> ---
>> drivers/mtd/nand/brcmnand/brcmnand.c | 12 ++++++------
>> drivers/mtd/nand/brcmnand/brcmnand.h | 13 ++++++++-----
>> drivers/mtd/nand/brcmnand/iproc_nand.c | 18 ++++++++++++++----
>> 3 files changed, 28 insertions(+), 15 deletions(-)
>>
>> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.c b/drivers/mtd/nand/brcmnand/brcmnand.c
>> index b76ad7c..12a1585 100644
>> --- a/drivers/mtd/nand/brcmnand/brcmnand.c
>> +++ b/drivers/mtd/nand/brcmnand/brcmnand.c
>> @@ -1279,7 +1279,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
>> u32 *flash_cache = (u32 *)ctrl->flash_cache;
>> int i;
>>
>> - brcmnand_soc_data_bus_prepare(ctrl->soc);
>> + brcmnand_soc_data_bus_prepare(ctrl->soc, true);
>>
>> /*
>> * Must cache the FLASH_CACHE now, since changes in
>> @@ -1292,7 +1292,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
>> */
>> flash_cache[i] = be32_to_cpu(brcmnand_read_fc(ctrl, i));
>>
>> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
>> + brcmnand_soc_data_bus_unprepare(ctrl->soc, true);
>>
>> /* Cleanup from HW quirk: restore SECTOR_SIZE_1K */
>> if (host->hwcfg.sector_size_1k)
>> @@ -1508,12 +1508,12 @@ static int brcmnand_read_by_pio(struct mtd_info *mtd, struct nand_chip *chip,
>> brcmnand_waitfunc(mtd, chip);
>>
>> if (likely(buf)) {
>> - brcmnand_soc_data_bus_prepare(ctrl->soc);
>> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
>>
>> for (j = 0; j < FC_WORDS; j++, buf++)
>> *buf = brcmnand_read_fc(ctrl, j);
>>
>> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
>> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
>> }
>>
>> if (oob)
>> @@ -1678,12 +1678,12 @@ static int brcmnand_write(struct mtd_info *mtd, struct nand_chip *chip,
>> (void)brcmnand_read_reg(ctrl, BRCMNAND_CMD_ADDRESS);
>>
>> if (buf) {
>> - brcmnand_soc_data_bus_prepare(ctrl->soc);
>> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
>>
>> for (j = 0; j < FC_WORDS; j++, buf++)
>> brcmnand_write_fc(ctrl, j, *buf);
>>
>> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
>> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
>> } else if (oob) {
>> for (j = 0; j < FC_WORDS; j++)
>> brcmnand_write_fc(ctrl, j, 0xffffffff);
>> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.h b/drivers/mtd/nand/brcmnand/brcmnand.h
>> index ef5eabb..5c44cd4 100644
>> --- a/drivers/mtd/nand/brcmnand/brcmnand.h
>> +++ b/drivers/mtd/nand/brcmnand/brcmnand.h
>> @@ -23,19 +23,22 @@ struct dev_pm_ops;
>> struct brcmnand_soc {
>> bool (*ctlrdy_ack)(struct brcmnand_soc *soc);
>> void (*ctlrdy_set_enabled)(struct brcmnand_soc *soc, bool en);
>> - void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare);
>> + void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare,
>> + bool is_param);
>> };
>>
>> -static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc)
>> +static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc,
>> + bool is_param)
>> {
>> if (soc && soc->prepare_data_bus)
>> - soc->prepare_data_bus(soc, true);
>> + soc->prepare_data_bus(soc, true, is_param);
>> }
>>
>> -static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc)
>> +static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc,
>> + bool is_param)
>> {
>> if (soc && soc->prepare_data_bus)
>> - soc->prepare_data_bus(soc, false);
>> + soc->prepare_data_bus(soc, false, is_param);
>> }
>>
>> static inline u32 brcmnand_readl(void __iomem *addr)
>> diff --git a/drivers/mtd/nand/brcmnand/iproc_nand.c b/drivers/mtd/nand/brcmnand/iproc_nand.c
>> index 585596c..4c6ae11 100644
>> --- a/drivers/mtd/nand/brcmnand/iproc_nand.c
>> +++ b/drivers/mtd/nand/brcmnand/iproc_nand.c
>> @@ -74,7 +74,8 @@ static void iproc_nand_intc_set(struct brcmnand_soc *soc, bool en)
>> spin_unlock_irqrestore(&priv->idm_lock, flags);
>> }
>>
>> -static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
>> +static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare,
>> + bool is_param)
>> {
>> struct iproc_nand_soc *priv =
>> container_of(soc, struct iproc_nand_soc, soc);
>> @@ -86,10 +87,19 @@ static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
>>
>> val = brcmnand_readl(mmio);
>>
>> - if (prepare)
>> - val |= IPROC_NAND_APB_LE_MODE;
>> - else
>> + /*
>> + * In the case of BE or when dealing with NAND data, alway configure
>> + * the APB bus to LE mode before accessing the FIFO and back to BE mode
>> + * after the access is done
>> + */
>> + if (IS_ENABLED(CONFIG_CPU_BIG_ENDIAN) || !is_param) {
>> + if (prepare)
>> + val |= IPROC_NAND_APB_LE_MODE;
>> + else
>> + val &= ~IPROC_NAND_APB_LE_MODE;
>> + } else { /* when in LE accessing the parameter page, keep APB in BE */
>> val &= ~IPROC_NAND_APB_LE_MODE;
>> + }
>>
>> brcmnand_writel(val, mmio);
>>
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support
2016-08-19 15:14 ` Kamal Dasu
@ 2016-08-22 7:44 ` Boris Brezillon
0 siblings, 0 replies; 10+ messages in thread
From: Boris Brezillon @ 2016-08-22 7:44 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, 19 Aug 2016 11:14:17 -0400
Kamal Dasu <kdasu.kdev@gmail.com> wrote:
> Boris
>
> This change looks good to me.
Is this an Acked-by or a Reviewed-by? :)
>
>
> On Wed, Aug 10, 2016 at 10:24 AM, Boris Brezillon
> <boris.brezillon@free-electrons.com> wrote:
> > On Wed, 20 Jul 2016 14:53:50 -0700
> > Ray Jui <ray.jui@broadcom.com> wrote:
> >
> >> This patch adds big endian and ONFI support for various iProc based
> >> SoCs that use the core brcmstb NAND controller
> >
> > Brian, Kamal, can you review this patch?
> >
> >>
> >> This patch was originally implemented by Prafulla Kota
> >> <prafulla.kota@broadcom.com> and fully tested on iProc based NS2 SVK
> >>
> >> Signed-off-by: Prafulla Kota <prafulla.kota@broadcom.com>
> >> Signed-off-by: Ray Jui <ray.jui@broadcom.com>
>
> Reviewed-by: Kamal Dasu <kdasu.kdev@gmail.com>
>
> >> ---
> >> drivers/mtd/nand/brcmnand/brcmnand.c | 12 ++++++------
> >> drivers/mtd/nand/brcmnand/brcmnand.h | 13 ++++++++-----
> >> drivers/mtd/nand/brcmnand/iproc_nand.c | 18 ++++++++++++++----
> >> 3 files changed, 28 insertions(+), 15 deletions(-)
> >>
> >> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.c b/drivers/mtd/nand/brcmnand/brcmnand.c
> >> index b76ad7c..12a1585 100644
> >> --- a/drivers/mtd/nand/brcmnand/brcmnand.c
> >> +++ b/drivers/mtd/nand/brcmnand/brcmnand.c
> >> @@ -1279,7 +1279,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
> >> u32 *flash_cache = (u32 *)ctrl->flash_cache;
> >> int i;
> >>
> >> - brcmnand_soc_data_bus_prepare(ctrl->soc);
> >> + brcmnand_soc_data_bus_prepare(ctrl->soc, true);
> >>
> >> /*
> >> * Must cache the FLASH_CACHE now, since changes in
> >> @@ -1292,7 +1292,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
> >> */
> >> flash_cache[i] = be32_to_cpu(brcmnand_read_fc(ctrl, i));
> >>
> >> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
> >> + brcmnand_soc_data_bus_unprepare(ctrl->soc, true);
> >>
> >> /* Cleanup from HW quirk: restore SECTOR_SIZE_1K */
> >> if (host->hwcfg.sector_size_1k)
> >> @@ -1508,12 +1508,12 @@ static int brcmnand_read_by_pio(struct mtd_info *mtd, struct nand_chip *chip,
> >> brcmnand_waitfunc(mtd, chip);
> >>
> >> if (likely(buf)) {
> >> - brcmnand_soc_data_bus_prepare(ctrl->soc);
> >> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
> >>
> >> for (j = 0; j < FC_WORDS; j++, buf++)
> >> *buf = brcmnand_read_fc(ctrl, j);
> >>
> >> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
> >> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
> >> }
> >>
> >> if (oob)
> >> @@ -1678,12 +1678,12 @@ static int brcmnand_write(struct mtd_info *mtd, struct nand_chip *chip,
> >> (void)brcmnand_read_reg(ctrl, BRCMNAND_CMD_ADDRESS);
> >>
> >> if (buf) {
> >> - brcmnand_soc_data_bus_prepare(ctrl->soc);
> >> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
> >>
> >> for (j = 0; j < FC_WORDS; j++, buf++)
> >> brcmnand_write_fc(ctrl, j, *buf);
> >>
> >> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
> >> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
> >> } else if (oob) {
> >> for (j = 0; j < FC_WORDS; j++)
> >> brcmnand_write_fc(ctrl, j, 0xffffffff);
> >> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.h b/drivers/mtd/nand/brcmnand/brcmnand.h
> >> index ef5eabb..5c44cd4 100644
> >> --- a/drivers/mtd/nand/brcmnand/brcmnand.h
> >> +++ b/drivers/mtd/nand/brcmnand/brcmnand.h
> >> @@ -23,19 +23,22 @@ struct dev_pm_ops;
> >> struct brcmnand_soc {
> >> bool (*ctlrdy_ack)(struct brcmnand_soc *soc);
> >> void (*ctlrdy_set_enabled)(struct brcmnand_soc *soc, bool en);
> >> - void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare);
> >> + void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare,
> >> + bool is_param);
> >> };
> >>
> >> -static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc)
> >> +static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc,
> >> + bool is_param)
> >> {
> >> if (soc && soc->prepare_data_bus)
> >> - soc->prepare_data_bus(soc, true);
> >> + soc->prepare_data_bus(soc, true, is_param);
> >> }
> >>
> >> -static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc)
> >> +static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc,
> >> + bool is_param)
> >> {
> >> if (soc && soc->prepare_data_bus)
> >> - soc->prepare_data_bus(soc, false);
> >> + soc->prepare_data_bus(soc, false, is_param);
> >> }
> >>
> >> static inline u32 brcmnand_readl(void __iomem *addr)
> >> diff --git a/drivers/mtd/nand/brcmnand/iproc_nand.c b/drivers/mtd/nand/brcmnand/iproc_nand.c
> >> index 585596c..4c6ae11 100644
> >> --- a/drivers/mtd/nand/brcmnand/iproc_nand.c
> >> +++ b/drivers/mtd/nand/brcmnand/iproc_nand.c
> >> @@ -74,7 +74,8 @@ static void iproc_nand_intc_set(struct brcmnand_soc *soc, bool en)
> >> spin_unlock_irqrestore(&priv->idm_lock, flags);
> >> }
> >>
> >> -static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
> >> +static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare,
> >> + bool is_param)
> >> {
> >> struct iproc_nand_soc *priv =
> >> container_of(soc, struct iproc_nand_soc, soc);
> >> @@ -86,10 +87,19 @@ static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
> >>
> >> val = brcmnand_readl(mmio);
> >>
> >> - if (prepare)
> >> - val |= IPROC_NAND_APB_LE_MODE;
> >> - else
> >> + /*
> >> + * In the case of BE or when dealing with NAND data, alway configure
> >> + * the APB bus to LE mode before accessing the FIFO and back to BE mode
> >> + * after the access is done
> >> + */
> >> + if (IS_ENABLED(CONFIG_CPU_BIG_ENDIAN) || !is_param) {
> >> + if (prepare)
> >> + val |= IPROC_NAND_APB_LE_MODE;
> >> + else
> >> + val &= ~IPROC_NAND_APB_LE_MODE;
> >> + } else { /* when in LE accessing the parameter page, keep APB in BE */
> >> val &= ~IPROC_NAND_APB_LE_MODE;
> >> + }
> >>
> >> brcmnand_writel(val, mmio);
> >>
> >
>
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support
2016-08-10 14:24 ` Boris Brezillon
2016-08-19 15:14 ` Kamal Dasu
@ 2016-08-22 15:42 ` Kamal Dasu
1 sibling, 0 replies; 10+ messages in thread
From: Kamal Dasu @ 2016-08-22 15:42 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Aug 10, 2016 at 10:24 AM, Boris Brezillon
<boris.brezillon@free-electrons.com> wrote:
> On Wed, 20 Jul 2016 14:53:50 -0700
> Ray Jui <ray.jui@broadcom.com> wrote:
>
>> This patch adds big endian and ONFI support for various iProc based
>> SoCs that use the core brcmstb NAND controller
>
> Brian, Kamal, can you review this patch?
>
>>
>> This patch was originally implemented by Prafulla Kota
>> <prafulla.kota@broadcom.com> and fully tested on iProc based NS2 SVK
>>
>> Signed-off-by: Prafulla Kota <prafulla.kota@broadcom.com>
>> Signed-off-by: Ray Jui <ray.jui@broadcom.com>
Acked-by: Kamal Dasu <kdasu.kdev@gmail.com>
>> ---
>> drivers/mtd/nand/brcmnand/brcmnand.c | 12 ++++++------
>> drivers/mtd/nand/brcmnand/brcmnand.h | 13 ++++++++-----
>> drivers/mtd/nand/brcmnand/iproc_nand.c | 18 ++++++++++++++----
>> 3 files changed, 28 insertions(+), 15 deletions(-)
>>
>> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.c b/drivers/mtd/nand/brcmnand/brcmnand.c
>> index b76ad7c..12a1585 100644
>> --- a/drivers/mtd/nand/brcmnand/brcmnand.c
>> +++ b/drivers/mtd/nand/brcmnand/brcmnand.c
>> @@ -1279,7 +1279,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
>> u32 *flash_cache = (u32 *)ctrl->flash_cache;
>> int i;
>>
>> - brcmnand_soc_data_bus_prepare(ctrl->soc);
>> + brcmnand_soc_data_bus_prepare(ctrl->soc, true);
>>
>> /*
>> * Must cache the FLASH_CACHE now, since changes in
>> @@ -1292,7 +1292,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
>> */
>> flash_cache[i] = be32_to_cpu(brcmnand_read_fc(ctrl, i));
>>
>> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
>> + brcmnand_soc_data_bus_unprepare(ctrl->soc, true);
>>
>> /* Cleanup from HW quirk: restore SECTOR_SIZE_1K */
>> if (host->hwcfg.sector_size_1k)
>> @@ -1508,12 +1508,12 @@ static int brcmnand_read_by_pio(struct mtd_info *mtd, struct nand_chip *chip,
>> brcmnand_waitfunc(mtd, chip);
>>
>> if (likely(buf)) {
>> - brcmnand_soc_data_bus_prepare(ctrl->soc);
>> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
>>
>> for (j = 0; j < FC_WORDS; j++, buf++)
>> *buf = brcmnand_read_fc(ctrl, j);
>>
>> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
>> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
>> }
>>
>> if (oob)
>> @@ -1678,12 +1678,12 @@ static int brcmnand_write(struct mtd_info *mtd, struct nand_chip *chip,
>> (void)brcmnand_read_reg(ctrl, BRCMNAND_CMD_ADDRESS);
>>
>> if (buf) {
>> - brcmnand_soc_data_bus_prepare(ctrl->soc);
>> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
>>
>> for (j = 0; j < FC_WORDS; j++, buf++)
>> brcmnand_write_fc(ctrl, j, *buf);
>>
>> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
>> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
>> } else if (oob) {
>> for (j = 0; j < FC_WORDS; j++)
>> brcmnand_write_fc(ctrl, j, 0xffffffff);
>> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.h b/drivers/mtd/nand/brcmnand/brcmnand.h
>> index ef5eabb..5c44cd4 100644
>> --- a/drivers/mtd/nand/brcmnand/brcmnand.h
>> +++ b/drivers/mtd/nand/brcmnand/brcmnand.h
>> @@ -23,19 +23,22 @@ struct dev_pm_ops;
>> struct brcmnand_soc {
>> bool (*ctlrdy_ack)(struct brcmnand_soc *soc);
>> void (*ctlrdy_set_enabled)(struct brcmnand_soc *soc, bool en);
>> - void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare);
>> + void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare,
>> + bool is_param);
>> };
>>
>> -static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc)
>> +static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc,
>> + bool is_param)
>> {
>> if (soc && soc->prepare_data_bus)
>> - soc->prepare_data_bus(soc, true);
>> + soc->prepare_data_bus(soc, true, is_param);
>> }
>>
>> -static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc)
>> +static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc,
>> + bool is_param)
>> {
>> if (soc && soc->prepare_data_bus)
>> - soc->prepare_data_bus(soc, false);
>> + soc->prepare_data_bus(soc, false, is_param);
>> }
>>
>> static inline u32 brcmnand_readl(void __iomem *addr)
>> diff --git a/drivers/mtd/nand/brcmnand/iproc_nand.c b/drivers/mtd/nand/brcmnand/iproc_nand.c
>> index 585596c..4c6ae11 100644
>> --- a/drivers/mtd/nand/brcmnand/iproc_nand.c
>> +++ b/drivers/mtd/nand/brcmnand/iproc_nand.c
>> @@ -74,7 +74,8 @@ static void iproc_nand_intc_set(struct brcmnand_soc *soc, bool en)
>> spin_unlock_irqrestore(&priv->idm_lock, flags);
>> }
>>
>> -static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
>> +static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare,
>> + bool is_param)
>> {
>> struct iproc_nand_soc *priv =
>> container_of(soc, struct iproc_nand_soc, soc);
>> @@ -86,10 +87,19 @@ static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
>>
>> val = brcmnand_readl(mmio);
>>
>> - if (prepare)
>> - val |= IPROC_NAND_APB_LE_MODE;
>> - else
>> + /*
>> + * In the case of BE or when dealing with NAND data, alway configure
>> + * the APB bus to LE mode before accessing the FIFO and back to BE mode
>> + * after the access is done
>> + */
>> + if (IS_ENABLED(CONFIG_CPU_BIG_ENDIAN) || !is_param) {
>> + if (prepare)
>> + val |= IPROC_NAND_APB_LE_MODE;
>> + else
>> + val &= ~IPROC_NAND_APB_LE_MODE;
>> + } else { /* when in LE accessing the parameter page, keep APB in BE */
>> val &= ~IPROC_NAND_APB_LE_MODE;
>> + }
>>
>> brcmnand_writel(val, mmio);
>>
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support
2016-07-20 21:53 ` [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support Ray Jui
2016-08-10 14:24 ` Boris Brezillon
@ 2016-08-26 7:47 ` Boris Brezillon
1 sibling, 0 replies; 10+ messages in thread
From: Boris Brezillon @ 2016-08-26 7:47 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, 20 Jul 2016 14:53:50 -0700
Ray Jui <ray.jui@broadcom.com> wrote:
> This patch adds big endian and ONFI support for various iProc based
> SoCs that use the core brcmstb NAND controller
>
> This patch was originally implemented by Prafulla Kota
> <prafulla.kota@broadcom.com> and fully tested on iProc based NS2 SVK
>
> Signed-off-by: Prafulla Kota <prafulla.kota@broadcom.com>
> Signed-off-by: Ray Jui <ray.jui@broadcom.com>
Applied.
Thanks,
Boris
> ---
> drivers/mtd/nand/brcmnand/brcmnand.c | 12 ++++++------
> drivers/mtd/nand/brcmnand/brcmnand.h | 13 ++++++++-----
> drivers/mtd/nand/brcmnand/iproc_nand.c | 18 ++++++++++++++----
> 3 files changed, 28 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.c b/drivers/mtd/nand/brcmnand/brcmnand.c
> index b76ad7c..12a1585 100644
> --- a/drivers/mtd/nand/brcmnand/brcmnand.c
> +++ b/drivers/mtd/nand/brcmnand/brcmnand.c
> @@ -1279,7 +1279,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
> u32 *flash_cache = (u32 *)ctrl->flash_cache;
> int i;
>
> - brcmnand_soc_data_bus_prepare(ctrl->soc);
> + brcmnand_soc_data_bus_prepare(ctrl->soc, true);
>
> /*
> * Must cache the FLASH_CACHE now, since changes in
> @@ -1292,7 +1292,7 @@ static void brcmnand_cmdfunc(struct mtd_info *mtd, unsigned command,
> */
> flash_cache[i] = be32_to_cpu(brcmnand_read_fc(ctrl, i));
>
> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
> + brcmnand_soc_data_bus_unprepare(ctrl->soc, true);
>
> /* Cleanup from HW quirk: restore SECTOR_SIZE_1K */
> if (host->hwcfg.sector_size_1k)
> @@ -1508,12 +1508,12 @@ static int brcmnand_read_by_pio(struct mtd_info *mtd, struct nand_chip *chip,
> brcmnand_waitfunc(mtd, chip);
>
> if (likely(buf)) {
> - brcmnand_soc_data_bus_prepare(ctrl->soc);
> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
>
> for (j = 0; j < FC_WORDS; j++, buf++)
> *buf = brcmnand_read_fc(ctrl, j);
>
> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
> }
>
> if (oob)
> @@ -1678,12 +1678,12 @@ static int brcmnand_write(struct mtd_info *mtd, struct nand_chip *chip,
> (void)brcmnand_read_reg(ctrl, BRCMNAND_CMD_ADDRESS);
>
> if (buf) {
> - brcmnand_soc_data_bus_prepare(ctrl->soc);
> + brcmnand_soc_data_bus_prepare(ctrl->soc, false);
>
> for (j = 0; j < FC_WORDS; j++, buf++)
> brcmnand_write_fc(ctrl, j, *buf);
>
> - brcmnand_soc_data_bus_unprepare(ctrl->soc);
> + brcmnand_soc_data_bus_unprepare(ctrl->soc, false);
> } else if (oob) {
> for (j = 0; j < FC_WORDS; j++)
> brcmnand_write_fc(ctrl, j, 0xffffffff);
> diff --git a/drivers/mtd/nand/brcmnand/brcmnand.h b/drivers/mtd/nand/brcmnand/brcmnand.h
> index ef5eabb..5c44cd4 100644
> --- a/drivers/mtd/nand/brcmnand/brcmnand.h
> +++ b/drivers/mtd/nand/brcmnand/brcmnand.h
> @@ -23,19 +23,22 @@ struct dev_pm_ops;
> struct brcmnand_soc {
> bool (*ctlrdy_ack)(struct brcmnand_soc *soc);
> void (*ctlrdy_set_enabled)(struct brcmnand_soc *soc, bool en);
> - void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare);
> + void (*prepare_data_bus)(struct brcmnand_soc *soc, bool prepare,
> + bool is_param);
> };
>
> -static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc)
> +static inline void brcmnand_soc_data_bus_prepare(struct brcmnand_soc *soc,
> + bool is_param)
> {
> if (soc && soc->prepare_data_bus)
> - soc->prepare_data_bus(soc, true);
> + soc->prepare_data_bus(soc, true, is_param);
> }
>
> -static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc)
> +static inline void brcmnand_soc_data_bus_unprepare(struct brcmnand_soc *soc,
> + bool is_param)
> {
> if (soc && soc->prepare_data_bus)
> - soc->prepare_data_bus(soc, false);
> + soc->prepare_data_bus(soc, false, is_param);
> }
>
> static inline u32 brcmnand_readl(void __iomem *addr)
> diff --git a/drivers/mtd/nand/brcmnand/iproc_nand.c b/drivers/mtd/nand/brcmnand/iproc_nand.c
> index 585596c..4c6ae11 100644
> --- a/drivers/mtd/nand/brcmnand/iproc_nand.c
> +++ b/drivers/mtd/nand/brcmnand/iproc_nand.c
> @@ -74,7 +74,8 @@ static void iproc_nand_intc_set(struct brcmnand_soc *soc, bool en)
> spin_unlock_irqrestore(&priv->idm_lock, flags);
> }
>
> -static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
> +static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare,
> + bool is_param)
> {
> struct iproc_nand_soc *priv =
> container_of(soc, struct iproc_nand_soc, soc);
> @@ -86,10 +87,19 @@ static void iproc_nand_apb_access(struct brcmnand_soc *soc, bool prepare)
>
> val = brcmnand_readl(mmio);
>
> - if (prepare)
> - val |= IPROC_NAND_APB_LE_MODE;
> - else
> + /*
> + * In the case of BE or when dealing with NAND data, alway configure
> + * the APB bus to LE mode before accessing the FIFO and back to BE mode
> + * after the access is done
> + */
> + if (IS_ENABLED(CONFIG_CPU_BIG_ENDIAN) || !is_param) {
> + if (prepare)
> + val |= IPROC_NAND_APB_LE_MODE;
> + else
> + val &= ~IPROC_NAND_APB_LE_MODE;
> + } else { /* when in LE accessing the parameter page, keep APB in BE */
> val &= ~IPROC_NAND_APB_LE_MODE;
> + }
>
> brcmnand_writel(val, mmio);
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 2/2] arm64: dts: Updated NAND DT properties for NS2 SVK
2016-07-20 21:53 ` [PATCH v2 2/2] arm64: dts: Updated NAND DT properties for NS2 SVK Ray Jui
@ 2016-10-17 20:49 ` Ray Jui
2016-10-18 0:54 ` Florian Fainelli
0 siblings, 1 reply; 10+ messages in thread
From: Ray Jui @ 2016-10-17 20:49 UTC (permalink / raw)
To: linux-arm-kernel
Hi Florian,
Would you be able to help to queue the following patch for v4.10? Sorry
I did not include you in the "To" field of the original email.
Thanks,
Ray
On 7/20/2016 2:53 PM, Ray Jui wrote:
> This patch adds NAND DT properties for NS2 SVK to configure the bus
> width width and OOB sector size
>
> Signed-off-by: Prafulla Kota <prafulla.kota@broadcom.com>
> Signed-off-by: Ray Jui <ray.jui@broadcom.com>
> ---
> arch/arm64/boot/dts/broadcom/ns2-svk.dts | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/broadcom/ns2-svk.dts b/arch/arm64/boot/dts/broadcom/ns2-svk.dts
> index 54ca40c..5ef0407 100644
> --- a/arch/arm64/boot/dts/broadcom/ns2-svk.dts
> +++ b/arch/arm64/boot/dts/broadcom/ns2-svk.dts
> @@ -128,6 +128,8 @@
> nand-ecc-mode = "hw";
> nand-ecc-strength = <8>;
> nand-ecc-step-size = <512>;
> + nand-bus-width = <16>;
> + brcm,nand-oob-sector-size = <16>;
> #address-cells = <1>;
> #size-cells = <1>;
> };
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 2/2] arm64: dts: Updated NAND DT properties for NS2 SVK
2016-10-17 20:49 ` Ray Jui
@ 2016-10-18 0:54 ` Florian Fainelli
0 siblings, 0 replies; 10+ messages in thread
From: Florian Fainelli @ 2016-10-18 0:54 UTC (permalink / raw)
To: linux-arm-kernel
On 10/17/2016 01:49 PM, Ray Jui wrote:
> Hi Florian,
>
> Would you be able to help to queue the following patch for v4.10? Sorry
> I did not include you in the "To" field of the original email.
Applied, thanks!
--
Florian
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-10-18 0:54 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-20 21:53 [PATCH v2 0/2] Add big endian and ONFI support to iProc NAND Ray Jui
2016-07-20 21:53 ` [PATCH v2 1/2] mtd: brcmnand: iProc big endian and ONFI support Ray Jui
2016-08-10 14:24 ` Boris Brezillon
2016-08-19 15:14 ` Kamal Dasu
2016-08-22 7:44 ` Boris Brezillon
2016-08-22 15:42 ` Kamal Dasu
2016-08-26 7:47 ` Boris Brezillon
2016-07-20 21:53 ` [PATCH v2 2/2] arm64: dts: Updated NAND DT properties for NS2 SVK Ray Jui
2016-10-17 20:49 ` Ray Jui
2016-10-18 0:54 ` Florian Fainelli
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).