Re: [PATCH 2/3] scsi: ufs: Add temperature notification exception handling

From: Bart Van Assche
Date: Wed Sep 01 2021 - 12:39:38 EST


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.