Messages in this thread | | | Subject | Re: [PATCH] scsi: libsas: no need to join wide port again in sas_ex_discover_dev() | From | John Garry <> | Date | Mon, 20 May 2019 13:39:00 +0100 |
| |
On 20/05/2019 13:06, Jason Yan wrote: > OK. > >> >> In case of "second fanout expander...", before this, we don't attempt >> to discover, and just disable the PHY. In that case, is the log proper? >> > > In that case the log is not proper. I think we can directly return in > the case of "second fanout expander..."? Actually nothing to do after > the phy is disabled.
Yeah, that looks fine.
> >> And, if indeed proper, it would seem to merit a higher log level than >> debug, maybe notice is better.
| |