On 9/1/21 5:37 AM, Avri Altman wrote:
It is essentially up to the platform to decide what further actions need to be taken. So add a designated vop for that. Each chipset vendor can decide if it wants to use the thermal subsystem, hw monitor, or some Privet implementation.
Why to make chipset vendors define what to do in case of extreme temperatures? I'd prefer a single implementation in ufshcd.c instead of making each vendor come up with a different implementation.
+ void (*temp_notify)(struct ufs_hba *hba, u16 status);
Please do not add new vops without adding at least one implementation of that vop.
Thanks, Bart.
