[U-Boot] [UBOOT][PATCH 0/2] misc fix for memory mapped support, dra config

[U-Boot] [UBOOT][PATCH 0/2] misc fix for memory mapped support, dra config

[Sourav Poddar]

Hi Jagan,
Here are the miscellaneous fix and config update for dra7 qspi flash.

Patch series adds:
Add BAR config in dra7 config file.
Set spi controller device control registers before
doing a memory mapped read.

Patches available here:
git://gitorious.org/u-boot-shared/u-boot-qspi.git debug_dra_qspi

Tested on dra7 evm with qspi boot.

Sourav Poddar (2):
  config: dra7_evm: Add Bank Address Register(BAR) config
  driver: mtd: sf_ops: claim bus while doing memcpy

 drivers/mtd/spi/sf_ops.c     |    6 ++++++
 include/configs/dra7xx_evm.h |    1 +
 2 files changed, 7 insertions(+), 0 deletions(-)

[U-Boot] [UBOOT][PATCH 1/2] config: dra7_evm: Add Bank Address Register(BAR) config

[Sourav Poddar]

Add config to support bank address register.

Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
Tested-by: Yebio Mesfin <ymesfin@ti.com>
---
 include/configs/dra7xx_evm.h |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/configs/dra7xx_evm.h b/include/configs/dra7xx_evm.h
index 1600131..46cc6db 100644
--- a/include/configs/dra7xx_evm.h
+++ b/include/configs/dra7xx_evm.h
@@ -54,6 +54,7 @@
 #define CONFIG_SPI_FLASH_SPANSION
 #define CONFIG_CMD_SF
 #define CONFIG_CMD_SPI
+#define CONFIG_SPI_FLASH_BAR
 #define CONFIG_TI_SPI_MMAP
 #define CONFIG_SF_DEFAULT_SPEED                48000000
 #define CONFIG_DEFAULT_SPI_MODE                SPI_MODE_3
-- 
1.7.1

[U-Boot] [UBOOT][PATCH 2/2] driver: mtd: sf_ops: claim bus while doing memcpy

[Sourav Poddar]

claim spi bus while doing memory copy, this will set up
the spi controller device control register before doing
a memory read.

Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
Tested-by: Yebio Mesfin <ymesfin@ti.com>
---
 drivers/mtd/spi/sf_ops.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/drivers/mtd/spi/sf_ops.c b/drivers/mtd/spi/sf_ops.c
index 108665f..e316a69 100644
--- a/drivers/mtd/spi/sf_ops.c
+++ b/drivers/mtd/spi/sf_ops.c
@@ -273,9 +273,15 @@ int spi_flash_cmd_read_ops(struct spi_flash *flash, u32 offset,
 
 	/* Handle memory-mapped SPI */
 	if (flash->memory_map) {
+		ret = spi_claim_bus(flash->spi);
+		if (ret) {
+			debug("SF: unable to claim SPI bus\n");
+			return ret;
+		}
 		spi_xfer(flash->spi, 0, NULL, NULL, SPI_XFER_MMAP);
 		memcpy(data, flash->memory_map + offset, len);
 		spi_xfer(flash->spi, 0, NULL, NULL, SPI_XFER_MMAP_END);
+		spi_release_bus(flash->spi);
 		return 0;
 	}
 
-- 
1.7.1

[U-Boot] [UBOOT][PATCH 0/2] misc fix for memory mapped support, dra config

[Sourav Poddar]

Hi Jagan,
On Thursday 14 November 2013 09:01 PM, Sourav Poddar wrote:
> Hi Jagan,
> Here are the miscellaneous fix and config update for dra7 qspi flash.
>
> Patch series adds:
> Add BAR config in dra7 config file.
> Set spi controller device control registers before
> doing a memory mapped read.
>
> Patches available here:
> git://gitorious.org/u-boot-shared/u-boot-qspi.git debug_dra_qspi
>
> Tested on dra7 evm with qspi boot.
Can these patches be picked?
> Sourav Poddar (2):
>    config: dra7_evm: Add Bank Address Register(BAR) config
>    driver: mtd: sf_ops: claim bus while doing memcpy
>
>   drivers/mtd/spi/sf_ops.c     |    6 ++++++
>   include/configs/dra7xx_evm.h |    1 +
>   2 files changed, 7 insertions(+), 0 deletions(-)
>

[U-Boot] [UBOOT][PATCH 0/2] misc fix for memory mapped support, dra config

[Sourav Poddar]

Jagan,
On Thursday 14 November 2013 09:01 PM, Sourav Poddar wrote:
> Hi Jagan,
> Here are the miscellaneous fix and config update for dra7 qspi flash.
>
> Patch series adds:
> Add BAR config in dra7 config file.
> Set spi controller device control registers before
> doing a memory mapped read.
>
> Patches available here:
> git://gitorious.org/u-boot-shared/u-boot-qspi.git debug_dra_qspi
>
> Tested on dra7 evm with qspi boot.
>
> Sourav Poddar (2):
>    config: dra7_evm: Add Bank Address Register(BAR) config
>    driver: mtd: sf_ops: claim bus while doing memcpy
>
>   drivers/mtd/spi/sf_ops.c     |    6 ++++++
>   include/configs/dra7xx_evm.h |    1 +
>   2 files changed, 7 insertions(+), 0 deletions(-)
>
Ping on this.
This are miscellaneous fixes for dra7 qspi uboot.
Can you pick this up ?

[U-Boot] [UBOOT][PATCH 0/2] misc fix for memory mapped support, dra config

[Jagan Teki]

On Thu, Dec 19, 2013 at 10:36 AM, Sourav Poddar <sourav.poddar@ti.com> wrote:
> Jagan,
> On Thursday 14 November 2013 09:01 PM, Sourav Poddar wrote:
>>
>> Hi Jagan,
>> Here are the miscellaneous fix and config update for dra7 qspi flash.
>>
>> Patch series adds:
>> Add BAR config in dra7 config file.
>> Set spi controller device control registers before
>> doing a memory mapped read.
>>
>> Patches available here:
>> git://gitorious.org/u-boot-shared/u-boot-qspi.git debug_dra_qspi
>>
>> Tested on dra7 evm with qspi boot.
>>
>> Sourav Poddar (2):
>>    config: dra7_evm: Add Bank Address Register(BAR) config
>>    driver: mtd: sf_ops: claim bus while doing memcpy
>>
>>   drivers/mtd/spi/sf_ops.c     |    6 ++++++
>>   include/configs/dra7xx_evm.h |    1 +
>>   2 files changed, 7 insertions(+), 0 deletions(-)
>>
> Ping on this.
> This are miscellaneous fixes for dra7 qspi uboot.
> Can you pick this up ?

What is this series - I couldn't find it on patchwork.
You sent a series with 0/5 and these two are part of that is it?

-- 
Thanks,
Jagan.

[U-Boot] [UBOOT][PATCH 0/2] misc fix for memory mapped support, dra config

[Sourav Poddar]

On Thursday 19 December 2013 11:41 AM, Jagan Teki wrote:
> On Thu, Dec 19, 2013 at 10:36 AM, Sourav Poddar<sourav.poddar@ti.com>  wrote:
>> Jagan,
>> On Thursday 14 November 2013 09:01 PM, Sourav Poddar wrote:
>>> Hi Jagan,
>>> Here are the miscellaneous fix and config update for dra7 qspi flash.
>>>
>>> Patch series adds:
>>> Add BAR config in dra7 config file.
>>> Set spi controller device control registers before
>>> doing a memory mapped read.
>>>
>>> Patches available here:
>>> git://gitorious.org/u-boot-shared/u-boot-qspi.git debug_dra_qspi
>>>
>>> Tested on dra7 evm with qspi boot.
>>>
>>> Sourav Poddar (2):
>>>     config: dra7_evm: Add Bank Address Register(BAR) config
>>>     driver: mtd: sf_ops: claim bus while doing memcpy
>>>
>>>    drivers/mtd/spi/sf_ops.c     |    6 ++++++
>>>    include/configs/dra7xx_evm.h |    1 +
>>>    2 files changed, 7 insertions(+), 0 deletions(-)
>>>
>> Ping on this.
>> This are miscellaneous fixes for dra7 qspi uboot.
>> Can you pick this up ?
> What is this series - I couldn't find it on patchwork.
> You sent a series with 0/5 and these two are part of that is it?
>
This is a differnet series for dra7 soc, it has miscellaneous cleanup
and fixes.

The other series (0/5) is for qspi on am43x soc

[U-Boot] [UBOOT][PATCH 0/2] misc fix for memory mapped support, dra config

[Jagan Teki]

On Thu, Dec 19, 2013 at 11:48 AM, Sourav Poddar <sourav.poddar@ti.com> wrote:
> On Thursday 19 December 2013 11:41 AM, Jagan Teki wrote:
>>
>> On Thu, Dec 19, 2013 at 10:36 AM, Sourav Poddar<sourav.poddar@ti.com>
>> wrote:
>>>
>>> Jagan,
>>> On Thursday 14 November 2013 09:01 PM, Sourav Poddar wrote:
>>>>
>>>> Hi Jagan,
>>>> Here are the miscellaneous fix and config update for dra7 qspi flash.
>>>>
>>>> Patch series adds:
>>>> Add BAR config in dra7 config file.
>>>> Set spi controller device control registers before
>>>> doing a memory mapped read.
>>>>
>>>> Patches available here:
>>>> git://gitorious.org/u-boot-shared/u-boot-qspi.git debug_dra_qspi
>>>>
>>>> Tested on dra7 evm with qspi boot.
>>>>
>>>> Sourav Poddar (2):
>>>>     config: dra7_evm: Add Bank Address Register(BAR) config
>>>>     driver: mtd: sf_ops: claim bus while doing memcpy
>>>>
>>>>    drivers/mtd/spi/sf_ops.c     |    6 ++++++
>>>>    include/configs/dra7xx_evm.h |    1 +
>>>>    2 files changed, 7 insertions(+), 0 deletions(-)
>>>>
>>> Ping on this.
>>> This are miscellaneous fixes for dra7 qspi uboot.
>>> Can you pick this up ?
>>
>> What is this series - I couldn't find it on patchwork.
>> You sent a series with 0/5 and these two are part of that is it?
>>
> This is a differnet series for dra7 soc, it has miscellaneous cleanup
> and fixes.
>
> The other series (0/5) is for qspi on am43x soc

Applied to u-boot-spi/master

-- 
Thanks,
Jagan.
--------
Jagannadha Sutradharudu Teki,
E: jagannadh.teki at gmail.com, P: +91-9676773388
Engineer - System Software Hacker
U-boot - SPI Custodian and Zynq APSOC
Ln: http://www.linkedin.com/in/jaganteki

[U-Boot] [UBOOT][PATCH 0/2] misc fix for memory mapped support, dra config

[Sourav Poddar]

On Thursday 19 December 2013 12:25 PM, Jagan Teki wrote:
> On Thu, Dec 19, 2013 at 11:48 AM, Sourav Poddar<sourav.poddar@ti.com>  wrote:
>> On Thursday 19 December 2013 11:41 AM, Jagan Teki wrote:
>>> On Thu, Dec 19, 2013 at 10:36 AM, Sourav Poddar<sourav.poddar@ti.com>
>>> wrote:
>>>> Jagan,
>>>> On Thursday 14 November 2013 09:01 PM, Sourav Poddar wrote:
>>>>> Hi Jagan,
>>>>> Here are the miscellaneous fix and config update for dra7 qspi flash.
>>>>>
>>>>> Patch series adds:
>>>>> Add BAR config in dra7 config file.
>>>>> Set spi controller device control registers before
>>>>> doing a memory mapped read.
>>>>>
>>>>> Patches available here:
>>>>> git://gitorious.org/u-boot-shared/u-boot-qspi.git debug_dra_qspi
>>>>>
>>>>> Tested on dra7 evm with qspi boot.
>>>>>
>>>>> Sourav Poddar (2):
>>>>>      config: dra7_evm: Add Bank Address Register(BAR) config
>>>>>      driver: mtd: sf_ops: claim bus while doing memcpy
>>>>>
>>>>>     drivers/mtd/spi/sf_ops.c     |    6 ++++++
>>>>>     include/configs/dra7xx_evm.h |    1 +
>>>>>     2 files changed, 7 insertions(+), 0 deletions(-)
>>>>>
>>>> Ping on this.
>>>> This are miscellaneous fixes for dra7 qspi uboot.
>>>> Can you pick this up ?
>>> What is this series - I couldn't find it on patchwork.
>>> You sent a series with 0/5 and these two are part of that is it?
>>>
>> This is a differnet series for dra7 soc, it has miscellaneous cleanup
>> and fixes.
>>
>> The other series (0/5) is for qspi on am43x soc
> Applied to u-boot-spi/master
>
Thanks!