On Mon, Jan 20, 2020 at 5:50 AM <Deepak.Ukey@xxxxxxxxxxxxx> wrote: > > > 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! How much effort to convert the HBA application to use general sas_phy information? IMHO converting HBA application to use general interface is the right way to go. Thanks
