Re: [PATCH] scsi: libsas: no need to join wide port again in sas_ex_discover_dev()

From: John Garry
Date: Mon May 20 2019 - 08:41:46 EST


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.