Hello Ryan, On 8/28/20 9:32 AM, Ryan Chen wrote: > This patch series refactor the exiting ASPEED clock name define sync > with Linux kernel. And also add SPDX-License
All the patchset seems correct but the patch numbering is a bit confusing. I have received : [1/3] cosmetic: aspeed: ast2500: Rename clock header [1/1] Remove not used export function header. [2/3] aspeed:clock: Sync with Linux kernel clock header define [3/3] cosmetic: aspeed: Modify for SPDX-License Could you please merge the first two together maybe and resend ? Thanks, C. > Ryan Chen (3): > cosmetic: aspeed: ast2500: Rename clock header > aspeed:clock: Sync with Linux kernel clock header define > cosmetic: aspeed: Modify for SPDX-License > > arch/arm/dts/ast2500-u-boot.dtsi | 23 ++++++----- > arch/arm/mach-aspeed/ast2500/sdram_ast2500.c | 2 +- > drivers/clk/aspeed/clk_ast2500.c | 40 +++++++++---------- > include/dt-bindings/clock/aspeed-clock.h | 42 ++++++++++++++++++++ > include/dt-bindings/clock/ast2500-scu.h | 30 -------------- > 5 files changed, 74 insertions(+), 63 deletions(-) > create mode 100644 include/dt-bindings/clock/aspeed-clock.h > delete mode 100644 include/dt-bindings/clock/ast2500-scu.h >