Messages in this thread | | | Date | Wed, 12 Jul 2023 12:00:05 +0530 | Subject | Re: [PATCH V4 3/4] dt-bindings: qcom-qce: add SoC compatible string for ipq9574 | From | Anusha Canchi <> |
| |
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@microchip.com> >>> Reviewed-by: Bhupesh Sharma <bhupesh.sharma@linaro.org> >>> Signed-off-by: Anusha Rao <quic_anusha@quicinc.com> >>> --- >>> 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 >
| |