* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
@ 2018-01-10 15:33 Goldschmidt Simon
2018-01-10 15:39 ` Marek Vasut
0 siblings, 1 reply; 16+ messages in thread
From: Goldschmidt Simon @ 2018-01-10 15:33 UTC (permalink / raw)
To: u-boot
On Tue 09/01/18 14:19, Vignesh R wrote:
> This series reverts use of bounce_buf.c for non-DMA related alignment restriction
> and replaces it with local bounce buffer to handle problems with non 32 bit aligned
> writes on TI platforms.
> Based on top of Jason's series:
> https://patchwork.ozlabs.org/cover/856431/
>
> Tested on K2G EVM.
For this whole series:
Tested on a socfpga-cyclonev board (with a Micron N25QL256A):
Tested-by: Simon Goldschmidt <sgoldschmidt@de.pepperl-fuchs.com>
After applying this series on top of Jason's v5, qspi on the socfpga is finally
working without local fixes!
Regards,
Simon
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-10 15:33 [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses Goldschmidt Simon
@ 2018-01-10 15:39 ` Marek Vasut
0 siblings, 0 replies; 16+ messages in thread
From: Marek Vasut @ 2018-01-10 15:39 UTC (permalink / raw)
To: u-boot
On 01/10/2018 04:33 PM, Goldschmidt Simon wrote:
> On Tue 09/01/18 14:19, Vignesh R wrote:
>> This series reverts use of bounce_buf.c for non-DMA related alignment restriction
>> and replaces it with local bounce buffer to handle problems with non 32 bit aligned
>> writes on TI platforms.
>> Based on top of Jason's series:
>> https://patchwork.ozlabs.org/cover/856431/
>>
>> Tested on K2G EVM.
>
> For this whole series:
> Tested on a socfpga-cyclonev board (with a Micron N25QL256A):
> Tested-by: Simon Goldschmidt <sgoldschmidt@de.pepperl-fuchs.com>
>
> After applying this series on top of Jason's v5, qspi on the socfpga is finally
> working without local fixes!
Super, thanks for testing.
Also, please fix your mailer so it doesn't break threading.
> Regards,
> Simon
>
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-23 9:31 ` Jagan Teki
@ 2018-01-23 10:05 ` Vignesh R
0 siblings, 0 replies; 16+ messages in thread
From: Vignesh R @ 2018-01-23 10:05 UTC (permalink / raw)
To: u-boot
On Tuesday 23 January 2018 03:01 PM, Jagan Teki wrote:
[...]
>>>>> u-boot-spi/master throws few warnings when I build for my platform:
>>>>> ~/workspace/u-boot:0f520af57c60:~ make distclean;make
>>>>> k2g_evm_defconfig;make -j4 -s;
>>>>>
>>>>> In file included from drivers/spi/cadence_qspi_apb.c:31:0:
>>>>> include/wait_bit.h: In function 'wait_for_bit_be16':
>>>>> include/wait_bit.h:76:31: warning: implicit declaration of function
>>>>> 'readw_be' [-Wimplicit-function-declaration]
>>>>> BUILD_WAIT_FOR_BIT(be16, u16, readw_be)
>>>>> ^
>>>>> include/wait_bit.h:48:9: note: in definition of macro
>>>>> 'BUILD_WAIT_FOR_BIT'
>>>>> val = read(reg); \
>>>>> ^~~~
>>>>> include/wait_bit.h: In function 'wait_for_bit_be32':
>>>>> include/wait_bit.h:78:31: warning: implicit declaration of function
>>>>> 'readl_be' [-Wimplicit-function-declaration]
>>>>> BUILD_WAIT_FOR_BIT(be32, u32, readl_be)
>>>>> ^
>>>>> include/wait_bit.h:48:9: note: in definition of macro
>>>>> 'BUILD_WAIT_FOR_BIT'
>>>>> val = read(reg); \
>>>>> ^~~~
>>>>
>>>> Did you change wait_for_bit function with _le32? [1]
>>>>
>>>> [1]
>>>> http://git.denx.de/?p=u-boot-spi.git;a=commitdiff;h=9a5ff2669ef185d3a4bf73415c531e8d013993d8
>>>>
>>> Yes, I am building plain u-boot-spi/master(HEAD: 0f520af57c60f3baba8
>>> ("spi: kirkwood_spi: implement workaround for FE-9144572")). So, above
>>> commit is part of the tree.
>>>
>>> Problem is above commit adds
>>> BUILD_WAIT_FOR_BIT(be32, u32, read*_be) variants.
>>> But, readl_be and readw_be seems to be undefined for ARM?
>>>
>>>
>>
>> Vignesh, I think there's an ongoing discussion about this on the u-boot list
>> to fix this?
>
> didn't get what issue Vignesh, pointing at but with new wait_for_bit
> need a fix[2] to resolve warnings.
>
> [2] https://gist.github.com/Noltari/3e6ed4648b87484c73ca22e2f533f9b0
>
Yes, above patch should help to get rid of warnings. I will rebase, test
and resend.
--
Regards
Vignesh
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-23 9:26 ` Simon Goldschmidt
@ 2018-01-23 9:31 ` Jagan Teki
2018-01-23 10:05 ` Vignesh R
0 siblings, 1 reply; 16+ messages in thread
From: Jagan Teki @ 2018-01-23 9:31 UTC (permalink / raw)
To: u-boot
On Tue, Jan 23, 2018 at 2:56 PM, Simon Goldschmidt
<sgoldschmidt@de.pepperl-fuchs.com> wrote:
> On 23.01.2018 10:16, Vignesh R wrote:
>>
>>
>> On Tuesday 23 January 2018 02:37 PM, Jagan Teki wrote:
>>>
>>> On Tue, Jan 23, 2018 at 2:32 PM, Vignesh R <vigneshr@ti.com> wrote:
>>>>
>>>>
>>>> On Tuesday 23 January 2018 02:15 PM, Jagan Teki wrote:
>>>>>
>>>>> On Tue, Jan 23, 2018 at 1:42 PM, Simon Goldschmidt
>>>>> <sgoldschmidt@de.pepperl-fuchs.com> wrote:
>>>>>>
>>>>>> Jagan,
>>>>>>
>>>>>>
>>>>>> the merge window is now less than 1 week for 2018.03 if I'm not
>>>>>> mistaken.
>>>>>> Can you apply this, please?
>>>>>
>>>>> Can you resend these on-top of u-boot-spi/master, have few changes on
>>>>> wait_for_bit good to test those as well.
>>>>
>>>> u-boot-spi/master throws few warnings when I build for my platform:
>>>> ~/workspace/u-boot:0f520af57c60:~ make distclean;make
>>>> k2g_evm_defconfig;make -j4 -s;
>>>>
>>>> In file included from drivers/spi/cadence_qspi_apb.c:31:0:
>>>> include/wait_bit.h: In function 'wait_for_bit_be16':
>>>> include/wait_bit.h:76:31: warning: implicit declaration of function
>>>> 'readw_be' [-Wimplicit-function-declaration]
>>>> BUILD_WAIT_FOR_BIT(be16, u16, readw_be)
>>>> ^
>>>> include/wait_bit.h:48:9: note: in definition of macro
>>>> 'BUILD_WAIT_FOR_BIT'
>>>> val = read(reg); \
>>>> ^~~~
>>>> include/wait_bit.h: In function 'wait_for_bit_be32':
>>>> include/wait_bit.h:78:31: warning: implicit declaration of function
>>>> 'readl_be' [-Wimplicit-function-declaration]
>>>> BUILD_WAIT_FOR_BIT(be32, u32, readl_be)
>>>> ^
>>>> include/wait_bit.h:48:9: note: in definition of macro
>>>> 'BUILD_WAIT_FOR_BIT'
>>>> val = read(reg); \
>>>> ^~~~
>>>
>>> Did you change wait_for_bit function with _le32? [1]
>>>
>>> [1]
>>> http://git.denx.de/?p=u-boot-spi.git;a=commitdiff;h=9a5ff2669ef185d3a4bf73415c531e8d013993d8
>>>
>> Yes, I am building plain u-boot-spi/master(HEAD: 0f520af57c60f3baba8
>> ("spi: kirkwood_spi: implement workaround for FE-9144572")). So, above
>> commit is part of the tree.
>>
>> Problem is above commit adds
>> BUILD_WAIT_FOR_BIT(be32, u32, read*_be) variants.
>> But, readl_be and readw_be seems to be undefined for ARM?
>>
>>
>
> Vignesh, I think there's an ongoing discussion about this on the u-boot list
> to fix this?
didn't get what issue Vignesh, pointing at but with new wait_for_bit
need a fix[2] to resolve warnings.
[2] https://gist.github.com/Noltari/3e6ed4648b87484c73ca22e2f533f9b0
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-23 9:16 ` Vignesh R
2018-01-23 9:24 ` Jagan Teki
@ 2018-01-23 9:26 ` Simon Goldschmidt
2018-01-23 9:31 ` Jagan Teki
1 sibling, 1 reply; 16+ messages in thread
From: Simon Goldschmidt @ 2018-01-23 9:26 UTC (permalink / raw)
To: u-boot
On 23.01.2018 10:16, Vignesh R wrote:
>
> On Tuesday 23 January 2018 02:37 PM, Jagan Teki wrote:
>> On Tue, Jan 23, 2018 at 2:32 PM, Vignesh R <vigneshr@ti.com> wrote:
>>>
>>> On Tuesday 23 January 2018 02:15 PM, Jagan Teki wrote:
>>>> On Tue, Jan 23, 2018 at 1:42 PM, Simon Goldschmidt
>>>> <sgoldschmidt@de.pepperl-fuchs.com> wrote:
>>>>> Jagan,
>>>>>
>>>>>
>>>>> the merge window is now less than 1 week for 2018.03 if I'm not mistaken.
>>>>> Can you apply this, please?
>>>> Can you resend these on-top of u-boot-spi/master, have few changes on
>>>> wait_for_bit good to test those as well.
>>> u-boot-spi/master throws few warnings when I build for my platform:
>>> ~/workspace/u-boot:0f520af57c60:~ make distclean;make k2g_evm_defconfig;make -j4 -s;
>>>
>>> In file included from drivers/spi/cadence_qspi_apb.c:31:0:
>>> include/wait_bit.h: In function 'wait_for_bit_be16':
>>> include/wait_bit.h:76:31: warning: implicit declaration of function 'readw_be' [-Wimplicit-function-declaration]
>>> BUILD_WAIT_FOR_BIT(be16, u16, readw_be)
>>> ^
>>> include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
>>> val = read(reg); \
>>> ^~~~
>>> include/wait_bit.h: In function 'wait_for_bit_be32':
>>> include/wait_bit.h:78:31: warning: implicit declaration of function 'readl_be' [-Wimplicit-function-declaration]
>>> BUILD_WAIT_FOR_BIT(be32, u32, readl_be)
>>> ^
>>> include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
>>> val = read(reg); \
>>> ^~~~
>> Did you change wait_for_bit function with _le32? [1]
>>
>> [1] http://git.denx.de/?p=u-boot-spi.git;a=commitdiff;h=9a5ff2669ef185d3a4bf73415c531e8d013993d8
>>
> Yes, I am building plain u-boot-spi/master(HEAD: 0f520af57c60f3baba8
> ("spi: kirkwood_spi: implement workaround for FE-9144572")). So, above
> commit is part of the tree.
>
> Problem is above commit adds
> BUILD_WAIT_FOR_BIT(be32, u32, read*_be) variants.
> But, readl_be and readw_be seems to be undefined for ARM?
>
>
Vignesh, I think there's an ongoing discussion about this on the u-boot
list to fix this?
Regards,
Simon
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-23 9:16 ` Vignesh R
@ 2018-01-23 9:24 ` Jagan Teki
2018-01-23 9:26 ` Simon Goldschmidt
1 sibling, 0 replies; 16+ messages in thread
From: Jagan Teki @ 2018-01-23 9:24 UTC (permalink / raw)
To: u-boot
On Tue, Jan 23, 2018 at 2:46 PM, Vignesh R <vigneshr@ti.com> wrote:
>
>
> On Tuesday 23 January 2018 02:37 PM, Jagan Teki wrote:
>> On Tue, Jan 23, 2018 at 2:32 PM, Vignesh R <vigneshr@ti.com> wrote:
>>>
>>>
>>> On Tuesday 23 January 2018 02:15 PM, Jagan Teki wrote:
>>>> On Tue, Jan 23, 2018 at 1:42 PM, Simon Goldschmidt
>>>> <sgoldschmidt@de.pepperl-fuchs.com> wrote:
>>>>> Jagan,
>>>>>
>>>>>
>>>>> the merge window is now less than 1 week for 2018.03 if I'm not mistaken.
>>>>> Can you apply this, please?
>>>>
>>>> Can you resend these on-top of u-boot-spi/master, have few changes on
>>>> wait_for_bit good to test those as well.
>>>
>>> u-boot-spi/master throws few warnings when I build for my platform:
>>> ~/workspace/u-boot:0f520af57c60:~ make distclean;make k2g_evm_defconfig;make -j4 -s;
>>>
>>> In file included from drivers/spi/cadence_qspi_apb.c:31:0:
>>> include/wait_bit.h: In function 'wait_for_bit_be16':
>>> include/wait_bit.h:76:31: warning: implicit declaration of function 'readw_be' [-Wimplicit-function-declaration]
>>> BUILD_WAIT_FOR_BIT(be16, u16, readw_be)
>>> ^
>>> include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
>>> val = read(reg); \
>>> ^~~~
>>> include/wait_bit.h: In function 'wait_for_bit_be32':
>>> include/wait_bit.h:78:31: warning: implicit declaration of function 'readl_be' [-Wimplicit-function-declaration]
>>> BUILD_WAIT_FOR_BIT(be32, u32, readl_be)
>>> ^
>>> include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
>>> val = read(reg); \
>>> ^~~~
>>
>> Did you change wait_for_bit function with _le32? [1]
>>
>> [1] http://git.denx.de/?p=u-boot-spi.git;a=commitdiff;h=9a5ff2669ef185d3a4bf73415c531e8d013993d8
>>
>
> Yes, I am building plain u-boot-spi/master(HEAD: 0f520af57c60f3baba8
> ("spi: kirkwood_spi: implement workaround for FE-9144572")). So, above
> commit is part of the tree.
>
> Problem is above commit adds
> BUILD_WAIT_FOR_BIT(be32, u32, read*_be) variants.
> But, readl_be and readw_be seems to be undefined for ARM?
yes, but wait_for_bit_le32 work for arm?
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-23 9:07 ` Jagan Teki
@ 2018-01-23 9:16 ` Vignesh R
2018-01-23 9:24 ` Jagan Teki
2018-01-23 9:26 ` Simon Goldschmidt
0 siblings, 2 replies; 16+ messages in thread
From: Vignesh R @ 2018-01-23 9:16 UTC (permalink / raw)
To: u-boot
On Tuesday 23 January 2018 02:37 PM, Jagan Teki wrote:
> On Tue, Jan 23, 2018 at 2:32 PM, Vignesh R <vigneshr@ti.com> wrote:
>>
>>
>> On Tuesday 23 January 2018 02:15 PM, Jagan Teki wrote:
>>> On Tue, Jan 23, 2018 at 1:42 PM, Simon Goldschmidt
>>> <sgoldschmidt@de.pepperl-fuchs.com> wrote:
>>>> Jagan,
>>>>
>>>>
>>>> the merge window is now less than 1 week for 2018.03 if I'm not mistaken.
>>>> Can you apply this, please?
>>>
>>> Can you resend these on-top of u-boot-spi/master, have few changes on
>>> wait_for_bit good to test those as well.
>>
>> u-boot-spi/master throws few warnings when I build for my platform:
>> ~/workspace/u-boot:0f520af57c60:~ make distclean;make k2g_evm_defconfig;make -j4 -s;
>>
>> In file included from drivers/spi/cadence_qspi_apb.c:31:0:
>> include/wait_bit.h: In function 'wait_for_bit_be16':
>> include/wait_bit.h:76:31: warning: implicit declaration of function 'readw_be' [-Wimplicit-function-declaration]
>> BUILD_WAIT_FOR_BIT(be16, u16, readw_be)
>> ^
>> include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
>> val = read(reg); \
>> ^~~~
>> include/wait_bit.h: In function 'wait_for_bit_be32':
>> include/wait_bit.h:78:31: warning: implicit declaration of function 'readl_be' [-Wimplicit-function-declaration]
>> BUILD_WAIT_FOR_BIT(be32, u32, readl_be)
>> ^
>> include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
>> val = read(reg); \
>> ^~~~
>
> Did you change wait_for_bit function with _le32? [1]
>
> [1] http://git.denx.de/?p=u-boot-spi.git;a=commitdiff;h=9a5ff2669ef185d3a4bf73415c531e8d013993d8
>
Yes, I am building plain u-boot-spi/master(HEAD: 0f520af57c60f3baba8
("spi: kirkwood_spi: implement workaround for FE-9144572")). So, above
commit is part of the tree.
Problem is above commit adds
BUILD_WAIT_FOR_BIT(be32, u32, read*_be) variants.
But, readl_be and readw_be seems to be undefined for ARM?
--
Regards
Vignesh
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-23 9:02 ` Vignesh R
@ 2018-01-23 9:07 ` Jagan Teki
2018-01-23 9:16 ` Vignesh R
0 siblings, 1 reply; 16+ messages in thread
From: Jagan Teki @ 2018-01-23 9:07 UTC (permalink / raw)
To: u-boot
On Tue, Jan 23, 2018 at 2:32 PM, Vignesh R <vigneshr@ti.com> wrote:
>
>
> On Tuesday 23 January 2018 02:15 PM, Jagan Teki wrote:
>> On Tue, Jan 23, 2018 at 1:42 PM, Simon Goldschmidt
>> <sgoldschmidt@de.pepperl-fuchs.com> wrote:
>>> Jagan,
>>>
>>>
>>> the merge window is now less than 1 week for 2018.03 if I'm not mistaken.
>>> Can you apply this, please?
>>
>> Can you resend these on-top of u-boot-spi/master, have few changes on
>> wait_for_bit good to test those as well.
>
> u-boot-spi/master throws few warnings when I build for my platform:
> ~/workspace/u-boot:0f520af57c60:~ make distclean;make k2g_evm_defconfig;make -j4 -s;
>
> In file included from drivers/spi/cadence_qspi_apb.c:31:0:
> include/wait_bit.h: In function 'wait_for_bit_be16':
> include/wait_bit.h:76:31: warning: implicit declaration of function 'readw_be' [-Wimplicit-function-declaration]
> BUILD_WAIT_FOR_BIT(be16, u16, readw_be)
> ^
> include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
> val = read(reg); \
> ^~~~
> include/wait_bit.h: In function 'wait_for_bit_be32':
> include/wait_bit.h:78:31: warning: implicit declaration of function 'readl_be' [-Wimplicit-function-declaration]
> BUILD_WAIT_FOR_BIT(be32, u32, readl_be)
> ^
> include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
> val = read(reg); \
> ^~~~
Did you change wait_for_bit function with _le32? [1]
[1] http://git.denx.de/?p=u-boot-spi.git;a=commitdiff;h=9a5ff2669ef185d3a4bf73415c531e8d013993d8
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-23 8:45 ` Jagan Teki
@ 2018-01-23 9:02 ` Vignesh R
2018-01-23 9:07 ` Jagan Teki
0 siblings, 1 reply; 16+ messages in thread
From: Vignesh R @ 2018-01-23 9:02 UTC (permalink / raw)
To: u-boot
On Tuesday 23 January 2018 02:15 PM, Jagan Teki wrote:
> On Tue, Jan 23, 2018 at 1:42 PM, Simon Goldschmidt
> <sgoldschmidt@de.pepperl-fuchs.com> wrote:
>> Jagan,
>>
>>
>> the merge window is now less than 1 week for 2018.03 if I'm not mistaken.
>> Can you apply this, please?
>
> Can you resend these on-top of u-boot-spi/master, have few changes on
> wait_for_bit good to test those as well.
u-boot-spi/master throws few warnings when I build for my platform:
~/workspace/u-boot:0f520af57c60:~ make distclean;make k2g_evm_defconfig;make -j4 -s;
In file included from drivers/spi/cadence_qspi_apb.c:31:0:
include/wait_bit.h: In function 'wait_for_bit_be16':
include/wait_bit.h:76:31: warning: implicit declaration of function 'readw_be' [-Wimplicit-function-declaration]
BUILD_WAIT_FOR_BIT(be16, u16, readw_be)
^
include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
val = read(reg); \
^~~~
include/wait_bit.h: In function 'wait_for_bit_be32':
include/wait_bit.h:78:31: warning: implicit declaration of function 'readl_be' [-Wimplicit-function-declaration]
BUILD_WAIT_FOR_BIT(be32, u32, readl_be)
^
include/wait_bit.h:48:9: note: in definition of macro 'BUILD_WAIT_FOR_BIT'
val = read(reg); \
^~~~
--
Regards
Vignesh
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-23 8:12 ` Simon Goldschmidt
@ 2018-01-23 8:45 ` Jagan Teki
2018-01-23 9:02 ` Vignesh R
0 siblings, 1 reply; 16+ messages in thread
From: Jagan Teki @ 2018-01-23 8:45 UTC (permalink / raw)
To: u-boot
On Tue, Jan 23, 2018 at 1:42 PM, Simon Goldschmidt
<sgoldschmidt@de.pepperl-fuchs.com> wrote:
> Jagan,
>
>
> the merge window is now less than 1 week for 2018.03 if I'm not mistaken.
> Can you apply this, please?
Can you resend these on-top of u-boot-spi/master, have few changes on
wait_for_bit good to test those as well.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-15 13:47 ` Jason Rush
@ 2018-01-23 8:12 ` Simon Goldschmidt
2018-01-23 8:45 ` Jagan Teki
0 siblings, 1 reply; 16+ messages in thread
From: Simon Goldschmidt @ 2018-01-23 8:12 UTC (permalink / raw)
To: u-boot
Jagan,
the merge window is now less than 1 week for 2018.03 if I'm not
mistaken. Can you apply this, please?
Thanks,
Simon
On 15.01.2018 14:47, Jason Rush wrote:
> On 1/15/2018 6:01 AM, Simon Goldschmidt wrote:
>> Pepperl+Fuchs GmbH, Mannheim
>> Geschaeftsfuehrer/Managing Directors: Dr.-Ing. Gunther Kegel (Vors./CEO), Werner Guthier, Mehmet Hatiboglu
>> Vorsitzender des Aufsichtsrats/Chairman of the supervisory board: Claus Michael
>> Registergericht/Register Court: AG Mannheim HRB 4713
>> On 15.01.2018 12:45, Marek Vasut wrote:
>>> On 01/15/2018 12:36 PM, Vignesh R wrote:
>>>> Marek,
>>>>
>>>> On 09-Jan-18 6:49 PM, Vignesh R wrote:
>>>>> This series reverts use of bounce_buf.c for non-DMA related alignment
>>>>> restriction and replaces it with local bounce buffer to handle problems
>>>>> with non 32 bit aligned writes on TI platforms.
>>>>> Based on top of Jason's series:
>>>>> https://patchwork.ozlabs.org/cover/856431/
>>>>>
>>>>> Tested on K2G EVM.
>>>>>
>>>> Could you ack this series, if you are okay with the changes?
>>>>
>>>> Jagan,
>>>> Could you pick this up(along with the above dependent patches) for
>>>> 2018.03 once Marek's Ack is in place?
>>>>
>>> Acked-by: Marek Vasut <marex@denx.de>
>>>
>>> although you should get ack from Jason and Simon instead, those matter.
>>>
>> Acked-by: Simon Goldschmidt <sgoldschmidt@de.pepperl-fuchs.com>
>>
>> If that "Simon" above was me :-) I thought ack was sent by maintainers and TB by me, sorry.
>>
>> Simon
>>
>> Wichtiger Hinweis:
>> Diese E-Mail einschliesslich ihrer Anhaenge enthaelt vertrauliche und rechtlich geschuetzte Informationen, die nur fuer den Adressaten bestimmt sind. Sollten Sie nicht der bezeichnete Adressat sein, so teilen Sie dies bitte dem Absender umgehend mit und loeschen Sie diese Nachricht und ihre Anhaenge. Die unbefugte Weitergabe, das Anfertigen von Kopien und jede Veraenderung der E-Mail ist untersagt. Der Absender haftet nicht fuer Inhalte von veraenderten E-Mails.
>>
>>
>> Important Information:
>> This e-mail message including its attachments contains confidential and legally protected information solely intended for the addressee. If you are not the intended addressee of this message, please contact the addresser immediately and delete this message including its attachments. The unauthorized dissemination, copying and change of this e-mail are strictly forbidden. The addresser shall not be liable for the content of such changed e-mails.
> Looks good to me. Thanks Vignesh.
>
> Reviewed-by: Jason Rush <jarush@gmail.com>
> Acked-by: Jason Rush <jarush@gmail.com>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-15 12:01 ` Simon Goldschmidt
@ 2018-01-15 13:47 ` Jason Rush
2018-01-23 8:12 ` Simon Goldschmidt
0 siblings, 1 reply; 16+ messages in thread
From: Jason Rush @ 2018-01-15 13:47 UTC (permalink / raw)
To: u-boot
On 1/15/2018 6:01 AM, Simon Goldschmidt wrote:
>
> Pepperl+Fuchs GmbH, Mannheim
> Geschaeftsfuehrer/Managing Directors: Dr.-Ing. Gunther Kegel (Vors./CEO), Werner Guthier, Mehmet Hatiboglu
> Vorsitzender des Aufsichtsrats/Chairman of the supervisory board: Claus Michael
> Registergericht/Register Court: AG Mannheim HRB 4713
> On 15.01.2018 12:45, Marek Vasut wrote:
>> On 01/15/2018 12:36 PM, Vignesh R wrote:
>>> Marek,
>>>
>>> On 09-Jan-18 6:49 PM, Vignesh R wrote:
>>>> This series reverts use of bounce_buf.c for non-DMA related alignment
>>>> restriction and replaces it with local bounce buffer to handle problems
>>>> with non 32 bit aligned writes on TI platforms.
>>>> Based on top of Jason's series:
>>>> https://patchwork.ozlabs.org/cover/856431/
>>>>
>>>> Tested on K2G EVM.
>>>>
>>> Could you ack this series, if you are okay with the changes?
>>>
>>> Jagan,
>>> Could you pick this up(along with the above dependent patches) for
>>> 2018.03 once Marek's Ack is in place?
>>>
>> Acked-by: Marek Vasut <marex@denx.de>
>>
>> although you should get ack from Jason and Simon instead, those matter.
>>
>
> Acked-by: Simon Goldschmidt <sgoldschmidt@de.pepperl-fuchs.com>
>
> If that "Simon" above was me :-) I thought ack was sent by maintainers and TB by me, sorry.
>
> Simon
>
> Wichtiger Hinweis:
> Diese E-Mail einschliesslich ihrer Anhaenge enthaelt vertrauliche und rechtlich geschuetzte Informationen, die nur fuer den Adressaten bestimmt sind. Sollten Sie nicht der bezeichnete Adressat sein, so teilen Sie dies bitte dem Absender umgehend mit und loeschen Sie diese Nachricht und ihre Anhaenge. Die unbefugte Weitergabe, das Anfertigen von Kopien und jede Veraenderung der E-Mail ist untersagt. Der Absender haftet nicht fuer Inhalte von veraenderten E-Mails.
>
>
> Important Information:
> This e-mail message including its attachments contains confidential and legally protected information solely intended for the addressee. If you are not the intended addressee of this message, please contact the addresser immediately and delete this message including its attachments. The unauthorized dissemination, copying and change of this e-mail are strictly forbidden. The addresser shall not be liable for the content of such changed e-mails.
Looks good to me. Thanks Vignesh.
Reviewed-by: Jason Rush <jarush@gmail.com>
Acked-by: Jason Rush <jarush@gmail.com>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-15 11:45 ` Marek Vasut
@ 2018-01-15 12:01 ` Simon Goldschmidt
2018-01-15 13:47 ` Jason Rush
0 siblings, 1 reply; 16+ messages in thread
From: Simon Goldschmidt @ 2018-01-15 12:01 UTC (permalink / raw)
To: u-boot
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8-sig", Size: 2164 bytes --]
Pepperl+Fuchs GmbH, Mannheim
Geschaeftsfuehrer/Managing Directors: Dr.-Ing. Gunther Kegel (Vors./CEO), Werner Guthier, Mehmet Hatiboglu
Vorsitzender des Aufsichtsrats/Chairman of the supervisory board: Claus Michael
Registergericht/Register Court: AG Mannheim HRB 4713
On 15.01.2018 12:45, Marek Vasut wrote:
> On 01/15/2018 12:36 PM, Vignesh R wrote:
>> Marek,
>>
>> On 09-Jan-18 6:49 PM, Vignesh R wrote:
>>> This series reverts use of bounce_buf.c for non-DMA related alignment
>>> restriction and replaces it with local bounce buffer to handle problems
>>> with non 32 bit aligned writes on TI platforms.
>>> Based on top of Jason's series:
>>> https://patchwork.ozlabs.org/cover/856431/
>>>
>>> Tested on K2G EVM.
>>>
>> Could you ack this series, if you are okay with the changes?
>>
>> Jagan,
>> Could you pick this up(along with the above dependent patches) for
>> 2018.03 once Marek's Ack is in place?
>>
> Acked-by: Marek Vasut <marex@denx.de>
>
> although you should get ack from Jason and Simon instead, those matter.
>
Acked-by: Simon Goldschmidt <sgoldschmidt@de.pepperl-fuchs.com>
If that "Simon" above was me :-) I thought ack was sent by maintainers
and TB by me, sorry.
Simon
Wichtiger Hinweis:
Diese E-Mail einschliesslich ihrer Anhaenge enthaelt vertrauliche und rechtlich geschuetzte Informationen, die nur fuer den Adressaten bestimmt sind.
Sollten Sie nicht der bezeichnete Adressat sein, so teilen Sie dies bitte dem Absender umgehend mit und loeschen Sie diese Nachricht und ihre Anhaenge. Die unbefugte Weitergabe, das Anfertigen von Kopien und jede Veraenderung der E-Mail ist untersagt. Der Absender haftet nicht fuer Inhalte von veraenderten E-Mails.
Important Information:
This e-mail message including its attachments contains confidential and legally protected information solely intended for the addressee. If you are not the intended addressee of this message, please contact the addresser immediately and delete this message including its attachments. The unauthorized dissemination, copying and change of this e-mail are strictly forbidden. The addresser shall not be liable for the content of such changed e-mails.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-15 11:36 ` Vignesh R
@ 2018-01-15 11:45 ` Marek Vasut
2018-01-15 12:01 ` Simon Goldschmidt
0 siblings, 1 reply; 16+ messages in thread
From: Marek Vasut @ 2018-01-15 11:45 UTC (permalink / raw)
To: u-boot
On 01/15/2018 12:36 PM, Vignesh R wrote:
> Marek,
>
> On 09-Jan-18 6:49 PM, Vignesh R wrote:
>> This series reverts use of bounce_buf.c for non-DMA related alignment
>> restriction and replaces it with local bounce buffer to handle problems
>> with non 32 bit aligned writes on TI platforms.
>> Based on top of Jason's series:
>> https://patchwork.ozlabs.org/cover/856431/
>>
>> Tested on K2G EVM.
>>
>
> Could you ack this series, if you are okay with the changes?
>
> Jagan,
> Could you pick this up(along with the above dependent patches) for
> 2018.03 once Marek's Ack is in place?
>
Acked-by: Marek Vasut <marex@denx.de>
although you should get ack from Jason and Simon instead, those matter.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
2018-01-09 13:19 Vignesh R
@ 2018-01-15 11:36 ` Vignesh R
2018-01-15 11:45 ` Marek Vasut
0 siblings, 1 reply; 16+ messages in thread
From: Vignesh R @ 2018-01-15 11:36 UTC (permalink / raw)
To: u-boot
Marek,
On 09-Jan-18 6:49 PM, Vignesh R wrote:
> This series reverts use of bounce_buf.c for non-DMA related alignment
> restriction and replaces it with local bounce buffer to handle problems
> with non 32 bit aligned writes on TI platforms.
> Based on top of Jason's series:
> https://patchwork.ozlabs.org/cover/856431/
>
> Tested on K2G EVM.
>
Could you ack this series, if you are okay with the changes?
Jagan,
Could you pick this up(along with the above dependent patches) for
2018.03 once Marek's Ack is in place?
> Goldschmidt Simon (1):
> Revert "spi: cadence_qspi_apb: Use 32 bit indirect read transaction
> when possible"
>
> Vignesh R (2):
> Revert "spi: cadence_qspi_apb: Use 32 bit indirect write transaction
> when possible"
> spi: cadence_qspi_apb: Make flash writes 32 bit aligned
>
> drivers/spi/cadence_qspi_apb.c | 49 ++++++++++++++++++----------------------
> include/configs/k2g_evm.h | 1 -
> include/configs/socfpga_common.h | 1 -
> include/configs/stv0991.h | 1 -
> 4 files changed, 22 insertions(+), 30 deletions(-)
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses
@ 2018-01-09 13:19 Vignesh R
2018-01-15 11:36 ` Vignesh R
0 siblings, 1 reply; 16+ messages in thread
From: Vignesh R @ 2018-01-09 13:19 UTC (permalink / raw)
To: u-boot
This series reverts use of bounce_buf.c for non-DMA related alignment
restriction and replaces it with local bounce buffer to handle problems
with non 32 bit aligned writes on TI platforms.
Based on top of Jason's series:
https://patchwork.ozlabs.org/cover/856431/
Tested on K2G EVM.
Goldschmidt Simon (1):
Revert "spi: cadence_qspi_apb: Use 32 bit indirect read transaction
when possible"
Vignesh R (2):
Revert "spi: cadence_qspi_apb: Use 32 bit indirect write transaction
when possible"
spi: cadence_qspi_apb: Make flash writes 32 bit aligned
drivers/spi/cadence_qspi_apb.c | 49 ++++++++++++++++++----------------------
include/configs/k2g_evm.h | 1 -
include/configs/socfpga_common.h | 1 -
include/configs/stv0991.h | 1 -
4 files changed, 22 insertions(+), 30 deletions(-)
--
2.15.1
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2018-01-23 10:05 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-10 15:33 [U-Boot] [PATCH v2 0/3] cadence-quadspi: Fix issues with non 32bit aligned accesses Goldschmidt Simon
2018-01-10 15:39 ` Marek Vasut
-- strict thread matches above, loose matches on Subject: below --
2018-01-09 13:19 Vignesh R
2018-01-15 11:36 ` Vignesh R
2018-01-15 11:45 ` Marek Vasut
2018-01-15 12:01 ` Simon Goldschmidt
2018-01-15 13:47 ` Jason Rush
2018-01-23 8:12 ` Simon Goldschmidt
2018-01-23 8:45 ` Jagan Teki
2018-01-23 9:02 ` Vignesh R
2018-01-23 9:07 ` Jagan Teki
2018-01-23 9:16 ` Vignesh R
2018-01-23 9:24 ` Jagan Teki
2018-01-23 9:26 ` Simon Goldschmidt
2018-01-23 9:31 ` Jagan Teki
2018-01-23 10:05 ` Vignesh R
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.