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>