Hi Bin, On 13 November 2018 at 17:14, Bin Meng <bmeng...@gmail.com> wrote: > 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?
Yes. I'm not sure what I was thinking there. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot