On 19/05/2023 16:56, Jagadeesh Kona wrote:Will take care of this in next series.
+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>