From mboxrd@z Thu Jan 1 00:00:00 1970 From: Minkyu Kang Date: Thu, 05 Dec 2013 17:07:04 +0900 Subject: [U-Boot] [PATCH 2/5] ARM: Exynos5250: remove redundant defines in exynos5250-dt.h In-Reply-To: <1380105759-9687-3-git-send-email-andre.przywara@linaro.org> References: <1380105759-9687-1-git-send-email-andre.przywara@linaro.org> <1380105759-9687-3-git-send-email-andre.przywara@linaro.org> Message-ID: <52A03428.1030502@samsung.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 25/09/13 19:42, Andre Przywara wrote: > For some reasons the very same defines appear twice in > exynos5250-dt.h. Remove the second version. > > Signed-off-by: Andre Przywara > --- > include/configs/exynos5250-dt.h | 21 --------------------- > 1 file changed, 21 deletions(-) > > diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h > index 14d62e4..c8c4b35 100644 > --- a/include/configs/exynos5250-dt.h > +++ b/include/configs/exynos5250-dt.h > @@ -284,27 +284,6 @@ > #define CONFIG_POWER_I2C > #define CONFIG_POWER_MAX77686 > > -/* SPI */ > -#define CONFIG_ENV_IS_IN_SPI_FLASH > -#define CONFIG_SPI_FLASH > - > -#ifdef CONFIG_SPI_FLASH > -#define CONFIG_EXYNOS_SPI > -#define CONFIG_CMD_SF > -#define CONFIG_CMD_SPI > -#define CONFIG_SPI_FLASH_WINBOND > -#define CONFIG_SF_DEFAULT_MODE SPI_MODE_0 > -#define CONFIG_SF_DEFAULT_SPEED 50000000 > -#define EXYNOS5_SPI_NUM_CONTROLLERS 5 > -#endif > - > -#ifdef CONFIG_ENV_IS_IN_SPI_FLASH > -#define CONFIG_ENV_SPI_MODE SPI_MODE_0 > -#define CONFIG_ENV_SECT_SIZE CONFIG_ENV_SIZE > -#define CONFIG_ENV_SPI_BUS 1 > -#define CONFIG_ENV_SPI_MAX_HZ 50000000 > -#endif > - > /* Ethernet Controllor Driver */ > #ifdef CONFIG_CMD_NET > #define CONFIG_SMC911X > Same patch is applied to u-boot-samsung. commit 31e0bd6974cbaca15e15f400acf1d7499e86fe17 Author: Luka Perkov Date: Mon Nov 4 02:12:00 2013 +0100 config: arm: exynos5250: remove duplicate defines The SPI section is already defined in this file (lines 268-288) so we can remove the duplicate definitions. While at it, also fix one tiny whitespace typo. Signed-off-by: Luka Perkov Signed-off-by: Minkyu Kang Thanks, Minkyu Kang.