RE: [PATCH v1] dt-bindings: clock: exynosautov920: add hsi2 clock definitions
From: Alim Akhtar
Date: Fri May 09 2025 - 22:22:18 EST
> -----Original Message-----
> From: Raghav Sharma <raghav.s@xxxxxxxxxxx>
> Sent: Friday, May 9, 2025 6:54 PM
> To: krzk@xxxxxxxxxx; s.nawrocki@xxxxxxxxxxx; cw00.choi@xxxxxxxxxxx;
> alim.akhtar@xxxxxxxxxxx; mturquette@xxxxxxxxxxxx; sboyd@xxxxxxxxxx;
> robh@xxxxxxxxxx; conor+dt@xxxxxxxxxx; sunyeal.hong@xxxxxxxxxxx;
> shin.son@xxxxxxxxxxx
> Cc: linux-samsung-soc@xxxxxxxxxxxxxxx; linux-clk@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; Raghav Sharma <raghav.s@xxxxxxxxxxx>
> Subject: [PATCH v1] dt-bindings: clock: exynosautov920: add hsi2 clock
> definitions
>
> Add device tree clock binding definitions for CMU_HSI2
>
> Signed-off-by: Raghav Sharma <raghav.s@xxxxxxxxxxx>
> ---
> .../clock/samsung,exynosautov920-clock.yaml | 29 +++++++++++++++++--
> .../clock/samsung,exynosautov920.h | 9 ++++++
> 2 files changed, 36 insertions(+), 2 deletions(-)
>
Hmm, you sent three separate patches that are interdependent but did not mention their dependencies.
e.g.
The patch that adds the clock driver cannot be compiled without the one that adds the header file, which is part of this binding patch
Go through the below section of submitting-patches documentation
https://www.kernel.org/doc/html/latest/process/submitting-patches.html#separate-your-changes
Make sure that each patch is individually compilable (or dependencies are mentioned clearly) and bisectable
ask someone if you are unsure of what you are submitting.