| Message ID | 20240602114439.1611-2-quic_jkona@quicinc.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | Add support for videocc and camcc on SM8650 | expand |
On 02/06/2024 13:44, Jagadeesh Kona wrote: > Correct the videocc header file name in SM8450 videocc bindings. > > Fixes: 1e910b2ba0ed ("dt-bindings: clock: qcom: Add SM8450 video clock controller") > Signed-off-by: Jagadeesh Kona <quic_jkona@quicinc.com> > Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > --- > .../devicetree/bindings/clock/qcom,sm8450-videocc.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml b/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml > index bad8f019a8d3..b135aa2e9f06 100644 > --- a/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml > +++ b/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml > @@ -13,7 +13,7 @@ description: | > Qualcomm video clock control module provides the clocks, resets and power > domains on SM8450. > > - See also:: include/dt-bindings/clock/qcom,videocc-sm8450.h > + See also: include/dt-bindings/clock/qcom,sm8450-videocc.h One patch like this for all files. Best regards, Krzysztof
On 02/06/2024 13:44, Jagadeesh Kona wrote: > Correct the videocc header file name in SM8450 videocc bindings. > > Fixes: 1e910b2ba0ed ("dt-bindings: clock: qcom: Add SM8450 video clock controller") > Signed-off-by: Jagadeesh Kona <quic_jkona@quicinc.com> > Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> Where did this review happen? Can you provide lore link? Best regards, Krzysztof
On 6/2/2024 8:54 PM, Krzysztof Kozlowski wrote: > On 02/06/2024 13:44, Jagadeesh Kona wrote: >> Correct the videocc header file name in SM8450 videocc bindings. >> >> Fixes: 1e910b2ba0ed ("dt-bindings: clock: qcom: Add SM8450 video clock controller") >> Signed-off-by: Jagadeesh Kona <quic_jkona@quicinc.com> >> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > > Where did this review happen? Can you provide lore link? > > Best regards, > Krzysztof > Please find the lore link for this review:- https://lore.kernel.org/all/e9109e46-4e72-4b3b-a995-4b2af3c31e45@linaro.org/#t Thanks, Jagadeesh
On 6/2/2024 8:52 PM, Krzysztof Kozlowski wrote: > On 02/06/2024 13:44, Jagadeesh Kona wrote: >> Correct the videocc header file name in SM8450 videocc bindings. >> >> Fixes: 1e910b2ba0ed ("dt-bindings: clock: qcom: Add SM8450 video clock controller") >> Signed-off-by: Jagadeesh Kona <quic_jkona@quicinc.com> >> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> >> --- >> .../devicetree/bindings/clock/qcom,sm8450-videocc.yaml | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml b/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml >> index bad8f019a8d3..b135aa2e9f06 100644 >> --- a/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml >> +++ b/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml >> @@ -13,7 +13,7 @@ description: | >> Qualcomm video clock control module provides the clocks, resets and power >> domains on SM8450. >> >> - See also:: include/dt-bindings/clock/qcom,videocc-sm8450.h >> + See also: include/dt-bindings/clock/qcom,sm8450-videocc.h > > One patch like this for all files. > > Best regards, > Krzysztof > Yes will check and post it as a separate patch outside of this series. Thanks, Jagadeesh
On 03/06/2024 15:27, Jagadeesh Kona wrote: > > > On 6/2/2024 8:54 PM, Krzysztof Kozlowski wrote: >> On 02/06/2024 13:44, Jagadeesh Kona wrote: >>> Correct the videocc header file name in SM8450 videocc bindings. >>> >>> Fixes: 1e910b2ba0ed ("dt-bindings: clock: qcom: Add SM8450 video clock controller") >>> Signed-off-by: Jagadeesh Kona <quic_jkona@quicinc.com> >>> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> >> >> Where did this review happen? Can you provide lore link? >> >> Best regards, >> Krzysztof >> > > Please find the lore link for this review:- > https://lore.kernel.org/all/e9109e46-4e72-4b3b-a995-4b2af3c31e45@linaro.org/#t Indeed, thanks. I could not find it. Best regards, Krzysztof
On 03/06/2024 15:27, Jagadeesh Kona wrote: >>> >>> - See also:: include/dt-bindings/clock/qcom,videocc-sm8450.h >>> + See also: include/dt-bindings/clock/qcom,sm8450-videocc.h >> >> One patch like this for all files. >> >> Best regards, >> Krzysztof >> > > Yes will check and post it as a separate patch outside of this series. Eh, I mixed up patchsets. It's fine, ignore my comments. Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml b/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml index bad8f019a8d3..b135aa2e9f06 100644 --- a/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml +++ b/Documentation/devicetree/bindings/clock/qcom,sm8450-videocc.yaml @@ -13,7 +13,7 @@ description: | Qualcomm video clock control module provides the clocks, resets and power domains on SM8450. - See also:: include/dt-bindings/clock/qcom,videocc-sm8450.h + See also: include/dt-bindings/clock/qcom,sm8450-videocc.h properties: compatible:
Correct the videocc header file name in SM8450 videocc bindings. Fixes: 1e910b2ba0ed ("dt-bindings: clock: qcom: Add SM8450 video clock controller") Signed-off-by: Jagadeesh Kona <quic_jkona@quicinc.com> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> --- .../devicetree/bindings/clock/qcom,sm8450-videocc.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)