Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Ran Wang <ran.wang_1@nxp.com>
To:     Peter Chen <peter.chen@nxp.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "balbi@kernel.org" <balbi@kernel.org>, Leo Li <leoyang.li@nxp.com>
CC:     "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 1/2] USB: phy: fsl-usb: remove sysfs abuse
Thread-Topic: [PATCH 1/2] USB: phy: fsl-usb: remove sysfs abuse
Thread-Index: AQHWUEKAQaIemoHQu0ecXIYPoFfzAKjz/9UAgAEXB7A=
Date:   Fri, 3 Jul 2020 01:49:19 +0000
Message-ID: <AM6PR04MB541321C805C5EE3DADE28D81F16A0@AM6PR04MB5413.eurprd04.prod.outlook.com>
References: <20200702072914.1072878-1-gregkh@linuxfoundation.org>
 <AM7PR04MB7157F5D247041AA604C3D1368B6D0@AM7PR04MB7157.eurprd04.prod.outlook.com>
In-Reply-To: <AM7PR04MB7157F5D247041AA604C3D1368B6D0@AM7PR04MB7157.eurprd04.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 2d2a7835-ac93-4ad0-9613-08d81ef34d88
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Jul 2020 01:49:19.8498
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: x3VGPVwp6njkGvEmZAoWSGC5ZlSNaqRC91eOVARZqHuyGopT8JHT3aMtrLDeERqz9H/qjENsTdthjXD4Ap09ew==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5PR04MB3153
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi,

On Thursday, July 2, 2020 5:07 PM, wrote:
> >
> > This file has a HUGE debugging sysfs file that spews out a lot of
> > information all at once, which violates the one-value-per-file rule
> > for sysfs.  If this is really needed, it should go into debugfs, but
> > given the age of this driver, I strongly doubt anyone is using it anymo=
re.
> >
> > So just remove the file entirely, it was never documented, so
> > obviously, no one actually needed it :)
> >
>=20
>=20
> Add related people from NXP.

Thanks for having me. Agree to clean this up for above reasons.

Regards,
Ran

> Peter
>=20
> > Cc: Felipe Balbi <balbi@kernel.org>
> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > ---
> >  drivers/usb/phy/phy-fsl-usb.c | 93
> > -----------------------------------
> >  1 file changed, 93 deletions(-)
> >
> > diff --git a/drivers/usb/phy/phy-fsl-usb.c
> > b/drivers/usb/phy/phy-fsl-usb.c index
> > b451f4695f3f..93d2257aeec8 100644
> > --- a/drivers/usb/phy/phy-fsl-usb.c
> > +++ b/drivers/usb/phy/phy-fsl-usb.c
> > @@ -957,98 +957,6 @@ int usb_otg_start(struct platform_device *pdev)
> >  	return 0;
> >  }
> >
> > -/*
> > - * state file in sysfs
> > - */
> > -static ssize_t show_fsl_usb2_otg_state(struct device *dev,
> > -				   struct device_attribute *attr, char *buf)
> > -{
> > -	struct otg_fsm *fsm =3D &fsl_otg_dev->fsm;
> > -	char *next =3D buf;
> > -	unsigned size =3D PAGE_SIZE;
> > -	int t;
> > -
> > -	mutex_lock(&fsm->lock);
> > -
> > -	/* basic driver infomation */
> > -	t =3D scnprintf(next, size,
> > -			DRIVER_DESC "\n" "fsl_usb2_otg version: %s\n\n",
> > -			DRIVER_VERSION);
> > -	size -=3D t;
> > -	next +=3D t;
> > -
> > -	/* Registers */
> > -	t =3D scnprintf(next, size,
> > -			"OTGSC:   0x%08x\n"
> > -			"PORTSC:  0x%08x\n"
> > -			"USBMODE: 0x%08x\n"
> > -			"USBCMD:  0x%08x\n"
> > -			"USBSTS:  0x%08x\n"
> > -			"USBINTR: 0x%08x\n",
> > -			fsl_readl(&usb_dr_regs->otgsc),
> > -			fsl_readl(&usb_dr_regs->portsc),
> > -			fsl_readl(&usb_dr_regs->usbmode),
> > -			fsl_readl(&usb_dr_regs->usbcmd),
> > -			fsl_readl(&usb_dr_regs->usbsts),
> > -			fsl_readl(&usb_dr_regs->usbintr));
> > -	size -=3D t;
> > -	next +=3D t;
> > -
> > -	/* State */
> > -	t =3D scnprintf(next, size,
> > -		      "OTG state: %s\n\n",
> > -		      usb_otg_state_string(fsl_otg_dev->phy.otg->state));
> > -	size -=3D t;
> > -	next +=3D t;
> > -
> > -	/* State Machine Variables */
> > -	t =3D scnprintf(next, size,
> > -			"a_bus_req: %d\n"
> > -			"b_bus_req: %d\n"
> > -			"a_bus_resume: %d\n"
> > -			"a_bus_suspend: %d\n"
> > -			"a_conn: %d\n"
> > -			"a_sess_vld: %d\n"
> > -			"a_srp_det: %d\n"
> > -			"a_vbus_vld: %d\n"
> > -			"b_bus_resume: %d\n"
> > -			"b_bus_suspend: %d\n"
> > -			"b_conn: %d\n"
> > -			"b_se0_srp: %d\n"
> > -			"b_ssend_srp: %d\n"
> > -			"b_sess_vld: %d\n"
> > -			"id: %d\n",
> > -			fsm->a_bus_req,
> > -			fsm->b_bus_req,
> > -			fsm->a_bus_resume,
> > -			fsm->a_bus_suspend,
> > -			fsm->a_conn,
> > -			fsm->a_sess_vld,
> > -			fsm->a_srp_det,
> > -			fsm->a_vbus_vld,
> > -			fsm->b_bus_resume,
> > -			fsm->b_bus_suspend,
> > -			fsm->b_conn,
> > -			fsm->b_se0_srp,
> > -			fsm->b_ssend_srp,
> > -			fsm->b_sess_vld,
> > -			fsm->id);
> > -	size -=3D t;
> > -	next +=3D t;
> > -
> > -	mutex_unlock(&fsm->lock);
> > -
> > -	return PAGE_SIZE - size;
> > -}
> > -
> > -static DEVICE_ATTR(fsl_usb2_otg_state, S_IRUGO,
> > show_fsl_usb2_otg_state, NULL);
> > -
> > -static struct attribute *fsl_otg_attrs[] =3D {
> > -	&dev_attr_fsl_usb2_otg_state.attr,
> > -	NULL,
> > -};
> > -ATTRIBUTE_GROUPS(fsl_otg);
> > -
> >  /* Char driver interface to control some OTG input */
> >
> >  /*
> > @@ -1167,7 +1075,6 @@ struct platform_driver fsl_otg_driver =3D {
> >  	.driver =3D {
> >  		.name =3D driver_name,
> >  		.owner =3D THIS_MODULE,
> > -		.dev_groups =3D fsl_otg_groups,
> >  	},
> >  };
> >
> > --
> > 2.27.0