> > These are ROC type things that means their values are cleared when the > > SFRs are read. > > They are usually read in ISR when an UIC error occur. > > Thus, their values would be zero at many cases. And there might be a > > little bit risky when they are read to be cleared before the ISR reads > > them, e.g. the case that a command is timed-out, ufshcd_dump_regs is > > called in ufshcd_abort and an UIC error occurs at the nearly same > > time. In this case, ISR will be called but UFS error handler will not > > be scheduled. > > This patch is to make UFS driver not read those SFRs in the dump > > function, i.e. ufshcd_dump_regs. > > This is essentially a fix, so perhaps a fixes tag? > > Wouldn't hurt to wrap the commit description more nicely. Thank you for your opinion. Thanks. Kiwoong Kim
