Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-03-01 Thread Vignesh R
On Tuesday 28 February 2017 10:36 PM, Rush, Jason A. wrote: > R, Vignesh wrote: >> On 2/28/2017 8:38 PM, Rush, Jason A. wrote: >> [...] >>> >>> This also works. >>> >>> Marek - how do you feel about a patch series with the following: >>> >>> 1. revert commit

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-28 Thread Rush, Jason A.
R, Vignesh wrote: > On 2/28/2017 8:38 PM, Rush, Jason A. wrote: > [...] >> >> This also works. >> >> Marek - how do you feel about a patch series with the following: >> >> 1. revert commit 57897c13de03ac0136d64641a3eab526c6810387 >> spi: cadence_qspi_apb: Use 32 bit indirect write transaction

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-28 Thread R, Vignesh
On 2/28/2017 8:38 PM, Rush, Jason A. wrote: [...] > > This also works. > > Marek - how do you feel about a patch series with the following: > > 1. revert commit 57897c13de03ac0136d64641a3eab526c6810387 > spi: cadence_qspi_apb: Use 32 bit indirect write transaction when > possible > 2.

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-28 Thread Rush, Jason A.
I don't know if this message successfully sent last weekend. My mail server was undergoing maintenance, and I didn't see my original message on the u-boot mailing list archive. So I'm resending, my apologies if this is a repost. R, Vignesh wrote: > On 2/25/2017 1:25 AM, Rush, Jason A. wrote:

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-24 Thread R, Vignesh
On 2/25/2017 1:25 AM, Rush, Jason A. wrote: > R, Vignesh wrote: >> On 2/24/2017 12:55 AM, Marek Vasut wrote: >>> On 02/23/2017 08:22 PM, Rush, Jason A. wrote: Marek Vasut wrote: > On 02/22/2017 06:37 PM, Rush, Jason A. wrote: >> Marek Vasut wrote: >>> On 02/21/2017 05:50 PM, Rush,

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-24 Thread Rush, Jason A.
R, Vignesh wrote: > On 2/24/2017 12:55 AM, Marek Vasut wrote: >> On 02/23/2017 08:22 PM, Rush, Jason A. wrote: >>> Marek Vasut wrote: On 02/22/2017 06:37 PM, Rush, Jason A. wrote: > Marek Vasut wrote: >> On 02/21/2017 05:50 PM, Rush, Jason A. wrote: > [...] >>> >>> While I was

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-24 Thread R, Vignesh
On 2/24/2017 12:55 AM, Marek Vasut wrote: > On 02/23/2017 08:22 PM, Rush, Jason A. wrote: >> Marek Vasut wrote: >>> On 02/22/2017 06:37 PM, Rush, Jason A. wrote: Marek Vasut wrote: > On 02/21/2017 05:50 PM, Rush, Jason A. wrote: [...] >> >> While I was debugging some of my changes, I

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-23 Thread Marek Vasut
On 02/23/2017 08:22 PM, Rush, Jason A. wrote: > Marek Vasut wrote: >> On 02/22/2017 06:37 PM, Rush, Jason A. wrote: >>> Marek Vasut wrote: On 02/21/2017 05:50 PM, Rush, Jason A. wrote: > The socfpga arch uses a different value for the indaddrtrig reg than > the ahbbase address.

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-23 Thread Rush, Jason A.
Marek Vasut wrote: > On 02/22/2017 06:37 PM, Rush, Jason A. wrote: >> Marek Vasut wrote: >>> On 02/21/2017 05:50 PM, Rush, Jason A. wrote: The socfpga arch uses a different value for the indaddrtrig reg than the ahbbase address. Adopting the Linux DT bindings separates the ahbbase

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-22 Thread Marek Vasut
On 02/22/2017 06:37 PM, Rush, Jason A. wrote: > Marek Vasut wrote: >> On 02/21/2017 05:50 PM, Rush, Jason A. wrote: >>> The socfpga arch uses a different value for the indaddrtrig reg than >>> the ahbbase address. Adopting the Linux DT bindings separates the >>> ahbbase and trigger-base addresses,

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-22 Thread Rush, Jason A.
Marek Vasut wrote: > On 02/21/2017 05:50 PM, Rush, Jason A. wrote: >> The socfpga arch uses a different value for the indaddrtrig reg than >> the ahbbase address. Adopting the Linux DT bindings separates the >> ahbbase and trigger-base addresses, allowing the trigger-base to be+ >> set correctly

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-22 Thread Marek Vasut
On 02/21/2017 05:50 PM, Rush, Jason A. wrote: > The socfpga arch uses a different value for the indaddrtrig reg than > the ahbbase address. Adopting the Linux DT bindings separates the > ahbbase and trigger-base addresses, allowing the trigger-base to be+ > set correctly on the socfpga arch. > >

Re: [U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-21 Thread Vignesh R
On Tuesday 21 February 2017 10:20 PM, Rush, Jason A. wrote: > The socfpga arch uses a different value for the indaddrtrig reg than > the ahbbase address. Adopting the Linux DT bindings separates the > ahbbase and trigger-base addresses, allowing the trigger-base to be+ > set correctly on the

[U-Boot] [PATCH] spi: cadence_qspi_apb: Add trigger-base DT bindings from Linux

2017-02-21 Thread Rush, Jason A.
The socfpga arch uses a different value for the indaddrtrig reg than the ahbbase address. Adopting the Linux DT bindings separates the ahbbase and trigger-base addresses, allowing the trigger-base to be+ set correctly on the socfpga arch. Tested on Terasic SoCkit dev board (Altera Cyclone V)