From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lokesh Vutla Date: Mon, 28 Sep 2015 16:25:01 +0530 Subject: [U-Boot] [PATCH 3/6] am43xx_evm.h: : switch to CONFIG_OMAP_SERIAL for non-SPL DM_SERIAL In-Reply-To: <1443437272-1939-4-git-send-email-mugunthanvnm@ti.com> References: <1443437272-1939-1-git-send-email-mugunthanvnm@ti.com> <1443437272-1939-4-git-send-email-mugunthanvnm@ti.com> Message-ID: <56091C85.5050306@ti.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 Monday 28 September 2015 04:17 PM, Mugunthan V N wrote: > Tested on am437x gp evm and am437x sk evm > > Signed-off-by: Mugunthan V N > --- > include/configs/am43xx_evm.h | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/include/configs/am43xx_evm.h b/include/configs/am43xx_evm.h > index d868442..03713e3 100644 > --- a/include/configs/am43xx_evm.h > +++ b/include/configs/am43xx_evm.h > @@ -23,9 +23,13 @@ > > /* NS16550 Configuration */ > #define CONFIG_SYS_NS16550 > +#if defined(CONFIG_SPL_BUILD) || !defined(CONFIG_DM_SERIAL) > #define CONFIG_SYS_NS16550_SERIAL > -#define CONFIG_SYS_NS16550_REG_SIZE (-4) > #define CONFIG_SYS_NS16550_CLK 48000000 > +#define CONFIG_SYS_NS16550_REG_SIZE (-4) You are deleting and adding the same line under if. Is this really necessary? Rest looks good to me. Thanks and regards, Lokesh