On 10/31/23 21:00, Can Guo wrote:
+ return sysfs_emit(buf, "%s\n", hba->pwr_info.hs_rate ?
+ ufshcd_ufs_hs_gear_to_string(hba->pwr_info.gear_rx) :
+ ufshcd_ufs_pwm_gear_to_string(hba->pwr_info.gear_rx));
+}
A nit: this is not how kernel code should be indented. I think that
"ufshcd_..." should be aligned with "buf,".
Anyway:
Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>