Re: [PATCH v10 1/4] media: dt-bindings: update clocks for sc7280-camss

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 17/12/2024 14:10, Krzysztof Kozlowski wrote:
On 17/12/2024 15:06, Vikram Sharma wrote:
This patch change clock names to make it consistent with
existing platforms as gcc_cam_hf_axi -> gcc_axi_hf.
This also adds gcc_axi_sf and remove gcc_camera_ahb.

Don't combine ABI changes with some less important things.

You miss here explanation why doing the ABI change in the first place.
Without that explanation I find it rather churn. But anyway, reason for
ABI break and impact should be documented in commit msg.


Signed-off-by: Vikram Sharma <quic_vikramsa@xxxxxxxxxxx>
---

Best regards,
Krzysztof

This change should be fine since we haven't committed and upstream dtsi, so there's no ABI to break yet.

Agree the cover letter should have been explicit in explaining.

---
bod




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux