Re: [PATCH V4 3/4] dt-bindings: qcom-qce: add SoC compatible string for ipq9574

From: Anusha Canchi
Date: Wed Jul 12 2023 - 02:34:02 EST




On 6/23/2023 6:23 PM, Krzysztof Kozlowski wrote:
On 23/06/2023 13:55, Anusha Rao wrote:
Document the compatible string for ipq9574.

Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
Reviewed-by: Bhupesh Sharma <bhupesh.sharma@xxxxxxxxxx>
Signed-off-by: Anusha Rao <quic_anusha@xxxxxxxxxxx>
---
Changes in V4:
- Picked up Reviewed-by tag.
A gentle reminder to pick the dt-binding patch.
As the dts change is picked, this patch is required to resolve dt-bindings check issues.
One patchset with four patches targeting three different subsystems, so
no wonder it gets missed. You will usually receive better results with
splitting such patchsets per subsystems.

One more thing is lack of proper subject prefix which indicates the
subsystem. Without it why anyone would pick it up? For example me, I
would just ignore it for my subsystem...
Ok..Referred previous commit titles of this file and added subject prefix. Do you suggest to repost it with
"dt-bindings: crypto: qcom-qce: add SoC compatible string for ipq9574" ?

Thanks,
Anusha

Best regards,
Krzysztof