diff mbox

[4/4] mach-u300: set apropriate FIFO trigger levels

Message ID 1308212094-22592-1-git-send-email-linus.walleij@stericsson.com (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Linus Walleij June 16, 2011, 8:14 a.m. UTC
From: Linus Walleij <linus.walleij@linaro.org>

The U300 just defined the fill level limits for the FIFOs to
1 item out of habit. It can easily handle four.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
 arch/arm/mach-u300/spi.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Comments

Grant Likely June 16, 2011, 12:22 p.m. UTC | #1
On Thu, Jun 16, 2011 at 10:14:54AM +0200, Linus Walleij wrote:
> From: Linus Walleij <linus.walleij@linaro.org>
> 
> The U300 just defined the fill level limits for the FIFOs to
> 1 item out of habit. It can easily handle four.
> 
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

I assume this one will go via your tree?

g.

> ---
>  arch/arm/mach-u300/spi.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-u300/spi.c b/arch/arm/mach-u300/spi.c
> index 5767208..7b597e2 100644
> --- a/arch/arm/mach-u300/spi.c
> +++ b/arch/arm/mach-u300/spi.c
> @@ -40,8 +40,8 @@ struct pl022_config_chip dummy_chip_info = {
>  	.hierarchy = SSP_MASTER,
>  	/* 0 = drive TX even as slave, 1 = do not drive TX as slave */
>  	.slave_tx_disable = 0,
> -	.rx_lev_trig = SSP_RX_1_OR_MORE_ELEM,
> -	.tx_lev_trig = SSP_TX_1_OR_MORE_EMPTY_LOC,
> +	.rx_lev_trig = SSP_RX_4_OR_MORE_ELEM,
> +	.tx_lev_trig = SSP_TX_4_OR_MORE_EMPTY_LOC,
>  	.ctrl_len = SSP_BITS_12,
>  	.wait_state = SSP_MWIRE_WAIT_ZERO,
>  	.duplex = SSP_MICROWIRE_CHANNEL_FULL_DUPLEX,
> -- 
> 1.7.3.2
> 

------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev
Linus Walleij June 16, 2011, 3:08 p.m. UTC | #2
On Thu, Jun 16, 2011 at 2:22 PM, Grant Likely <grant.likely@secretlab.ca> wrote:
> On Thu, Jun 16, 2011 at 10:14:54AM +0200, Linus Walleij wrote:
>> From: Linus Walleij <linus.walleij@linaro.org>
>>
>> The U300 just defined the fill level limits for the FIFOs to
>> 1 item out of habit. It can easily handle four.
>>
>> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
>
> I assume this one will go via your tree?

Yeah they can be merged out-of-order so I'll take it.

Thanks for fast action!
Linus Walleij

------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev
diff mbox

Patch

diff --git a/arch/arm/mach-u300/spi.c b/arch/arm/mach-u300/spi.c
index 5767208..7b597e2 100644
--- a/arch/arm/mach-u300/spi.c
+++ b/arch/arm/mach-u300/spi.c
@@ -40,8 +40,8 @@  struct pl022_config_chip dummy_chip_info = {
 	.hierarchy = SSP_MASTER,
 	/* 0 = drive TX even as slave, 1 = do not drive TX as slave */
 	.slave_tx_disable = 0,
-	.rx_lev_trig = SSP_RX_1_OR_MORE_ELEM,
-	.tx_lev_trig = SSP_TX_1_OR_MORE_EMPTY_LOC,
+	.rx_lev_trig = SSP_RX_4_OR_MORE_ELEM,
+	.tx_lev_trig = SSP_TX_4_OR_MORE_EMPTY_LOC,
 	.ctrl_len = SSP_BITS_12,
 	.wait_state = SSP_MWIRE_WAIT_ZERO,
 	.duplex = SSP_MICROWIRE_CHANNEL_FULL_DUPLEX,