Hi Simon, On Wed, Nov 14, 2018 at 4:02 AM Simon Glass <s...@chromium.org> wrote: > > On 13 November 2018 at 00:21, Bin Meng <bmeng...@gmail.com> wrote: > > This adds a timebase_freq member to the 'struct cpu_platdata', to > > hold the "timebase-frequency" value in the cpu or /cpus node. > > > > Signed-off-by: Bin Meng <bmeng...@gmail.com> > > --- > > > > include/cpu.h | 3 +++ > > 1 file changed, 3 insertions(+) > > Reviewed-by: Simon Glass <s...@chromium.org> > > Could we have comments for the struct members?
There are already comments added for the struct members. I am not sure what additional comments do you want to add? Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot