Hi Nobuhiro, On Fri, 20 Sep 2013 09:21:16 +0900, Nobuhiro Iwamatsu <[email protected]> wrote:
> Hi, Albert. > > (2013/09/19 18:09), Albert ARIBAUD wrote: > > Hi Nobuhiro, > > > > On Thu, 22 Aug 2013 13:05:49 +0900, Nobuhiro Iwamatsu > > <[email protected]> wrote: > > > >> Signed-off-by: Nobuhiro Iwamatsu<[email protected]> > >> --- > >> 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? OK -- I'll mark these 2 patches as "changes requested". > Best regards, > Nobuhiro Amicalement, -- Albert. _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
