EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe On Fri, Jan 17, 2020 at 8:10 AM Deepak Ukey <deepak.ukey@xxxxxxxxxxxxx> wrote: > > From: Viswas G <Viswas.G@xxxxxxxxxxxxx> > > Added sysfs attribute to show number of phys. > > Signed-off-by: Deepak Ukey <deepak.ukey@xxxxxxxxxxxxx> > Signed-off-by: Viswas G <Viswas.G@xxxxxxxxxxxxx> > Signed-off-by: Radha Ramachandran <radha@xxxxxxxxxx> I agree with John Gary, the mgmt tool can get the info from /sys/class/sas_phy, no need to add an extra sysfs. I suggest dropping the patch. > We have HBA application already in use by customer which uses this sysfs entry. So is it fine to keep this patch? Thanks!
