From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Chou <thomas@wytron.com.tw>
Date: Tue, 29 Dec 2015 21:00:27 +0800
Subject: [U-Boot] [PATCH v5 4/5] mips: ath79: add spi driver
In-Reply-To: <BLU436-SMTP175B0C852CA31B66DC45984FFFC0@phx.gbl>
References: <1451387833-932-1-git-send-email-wills.wang@live.com>
 <BLU436-SMTP175B0C852CA31B66DC45984FFFC0@phx.gbl>
Message-ID: <568283EB.7000003@wytron.com.tw>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Wills,

On 2015?12?29? 19:17, Wills Wang wrote:
> diff --git a/arch/mips/mach-ath79/Kconfig b/arch/mips/mach-ath79/Kconfig
> index 1d92a5b..b5668e9 100644
> --- a/arch/mips/mach-ath79/Kconfig
> +++ b/arch/mips/mach-ath79/Kconfig
> @@ -15,6 +15,7 @@ config SOC_AR933X
>   	select SYS_MIPS_CACHE_INIT_RAM_LOAD
>   	select MIPS_TUNE_24KC
>   	select AR933X_UART
> +	select ATH79_SPI

Both AR933X_UART and ATH79_SPI should not be selected with mach Kconfig. 
They should go with board defconfig.

> diff --git a/drivers/spi/ath79_spi.c b/drivers/spi/ath79_spi.c

> +static inline void ath79_spi_delay(int max_hz)
> +{
> +	uint64_t tick = get_tbclk();
> +
> +	do_div(tick, max_hz);
> +	tick = get_ticks() + tick + 1;	/* get current timestamp */
> +	while (get_ticks() < tick)	/* loop till event */
> +		 /*NOP*/;
> +}

Use udelay() instead.

Best regards,
Thomas