On 19/05/2023 16:56, Jagadeesh Kona wrote:
+MODULE_DESCRIPTION("QTI CAM_CC SM8550 Driver");
This pattern looks odd and inconsistent in our clock drivers. We mostly have CAMCC or VIDEOCC but occasionally have CAM_CC and VIDEO_CC.
I'd drop that underscore to, in addition to Konrad's comment on Ucase hex. Other than that, LGTM. Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>