From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nobuhiro Iwamatsu Date: Fri, 20 Sep 2013 09:21:16 +0900 Subject: [U-Boot] [PATCH 1/2] arm: rmobile: kzm9g: Change clock definition of SCIF from CONFIG_SYS_CLK_FREQ to CONFIG_SH_SCIF_CLK_FREQ In-Reply-To: <20130919110959.79602ca1@lilith> References: <1377144350-28324-1-git-send-email-nobuhiro.iwamatsu.yj@renesas.com> <20130919110959.79602ca1@lilith> Message-ID: <523B94FC.8000001@renesas.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, Albert. (2013/09/19 18:09), Albert ARIBAUD wrote: > Hi Nobuhiro, > > On Thu, 22 Aug 2013 13:05:49 +0900, Nobuhiro Iwamatsu > wrote: > >> Signed-off-by: Nobuhiro Iwamatsu >> --- >> include/configs/kzm9g.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/include/configs/kzm9g.h b/include/configs/kzm9g.h >> index 222725c..6323050 100644 >> --- a/include/configs/kzm9g.h >> +++ b/include/configs/kzm9g.h >> @@ -125,6 +125,7 @@ >> #define CONFIG_GLOBAL_TIMER >> #define CONFIG_SYS_CLK_FREQ (48000000) >> #define CONFIG_SYS_CPU_CLK (1196000000) >> +#define CONFIG_SH_SCIF_CLK_FREQ CONFIG_SYS_CLK_FREQ >> #define TMU_CLK_DIVIDER (4) /* 4 (default), 16, 64, 256 or 1024 */ >> #define CFG_HZ (1000) >> #define CONFIG_SYS_HZ CFG_HZ > > What's the point of this series? It only adds a define, but the > current codebase does not use it anywhere, so obviously it's "dead > code". This is used by drivers/serial/serial_sh.c. http://lists.denx.de/pipermail/u-boot/2013-August/161276.html I already applied this to my u-boot-sh repository. But I dont pull-request yet. I want to include in the next version of this change. > > If there is a further series which needs this define, then > include these patches in that further series. I see. I will be from next time, Is it OK? > > Amicalement, Best regards, Nobuhiro