On 26/12/2023 07:21, Aakarsh Jain wrote: >> -int s5p_mfc_clock_on(void); >> -void s5p_mfc_clock_off(void); >> -int s5p_mfc_power_on(void); >> -int s5p_mfc_power_off(void); >> +int s5p_mfc_clock_on(struct s5p_mfc_dev *dev); void >> +s5p_mfc_clock_off(struct s5p_mfc_dev *dev); int >> s5p_mfc_power_on(struct >> +s5p_mfc_dev *dev); int s5p_mfc_power_off(struct s5p_mfc_dev *dev); >> >> #endif /* S5P_MFC_PM_H_ */ >> >> -- >> 2.34.1 > > Reviewed-by: Aakarsh Jain <aakarsh.jain@xxxxxxxxxxx> Pleasy kindly trim the replies from unnecessary context. It makes it much easier to find new content. Best regards, Krzysztof