On 28/08/2024 09:36, Ryan Chen wrote: >> Subject: Re: [PATCH v2 1/3] dt-bindings: reset Add AST2700 reset bindings >> >> On 28/08/2024 08:27, Ryan Chen wrote: >>> Add dt bindings for AST2700 reset driver. >>> >>> Signed-off-by: Ryan Chen <ryan_chen@xxxxxxxxxxxxxx> >>> --- >>> .../dt-bindings/reset/aspeed,ast2700-reset.h | 125 >>> ++++++++++++++++++ >>> 1 file changed, 125 insertions(+) >>> create mode 100644 include/dt-bindings/reset/aspeed,ast2700-reset.h >> >> NAK. >> >> You still did not respond to my previous comments and did not implement >> them. >> >> Month ago I told you this is not a separate patch. >> >> https://lore.kernel.org/all/e13c16db-e1a7-4ee0-867b-b184d421de7f@kernel >> .org/ >> > Sorry, I think your point is still two files, but it should be at 1 patch. am I right? > If yes, I will put aspeed,ast2700-reset.h and aspeed,ast2700-clk.h to be 1 patch. Binding headers are part of bindings patch. Best regards, Krzysztof