RE: [PATCH V2 06/13] pm80xx : sysfs attribute for number of phys.

Deepak.Ukey
2020-01-20T04:50:38Z

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@microchip.com> wrote:
>
> From: Viswas G <Viswas.G@microchip.com>
>
> Added sysfs attribute to show number of phys.
>
> Signed-off-by: Deepak Ukey <deepak.ukey@microchip.com>
> Signed-off-by: Viswas G <Viswas.G@microchip.com>
> Signed-off-by: Radha Ramachandran <radha@google.com>
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!