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:   Avri Altman <Avri.Altman@wdc.com>
To:     Mauro Carvalho Chehab <mchehab+huawei@kernel.org>,
        Linux Doc Mailing List <linux-doc@vger.kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>
CC:     Jonathan Corbet <corbet@lwn.net>,
        "Martin K. Petersen" <martin.petersen@oracle.com>,
        Adrian Hunter <adrian.hunter@intel.com>,
        Bean Huo <beanhuo@micron.com>, Can Guo <cang@codeaurora.org>,
        Daejun Park <daejun7.park@samsung.com>,
        Lukas Bulwahn <lukas.bulwahn@gmail.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 16/17] ABI: sysfs-driver-ufs: Add another What for
 platform drivers
Thread-Topic: [PATCH 16/17] ABI: sysfs-driver-ufs: Add another What for
 platform drivers
Thread-Index: AQHXs6f5Yuh6nmqV7E6a9UXcE4axwKu6iE1g
Date:   Wed, 29 Sep 2021 06:32:40 +0000
Message-ID: <DM6PR04MB6575E3C9D13E8B41627240FDFCA99@DM6PR04MB6575.namprd04.prod.outlook.com>
References: <cover.1632750608.git.mchehab+huawei@kernel.org>
 <bc61469964bbcabe38d12aa88f2734d38a8741e5.1632750608.git.mchehab+huawei@kernel.org>
In-Reply-To: <bc61469964bbcabe38d12aa88f2734d38a8741e5.1632750608.git.mchehab+huawei@kernel.org>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?gdui1lvC3X8wSLQV+ZLgunSQqOCC0xXb3Dn2ZCcUnQQw9qY0g2apmQKJgaHa?=
 =?us-ascii?Q?CBZ7H/QFwJSt92KX9DS7Wmo8krzEdRcPLaXouh/J51stueTb8whsKAGvSvxy?=
 =?us-ascii?Q?mEEeycHh58GcqP+ud8/i+k/239A+lN/xxy48r/X+ep+qnOtUlgD/UzusQPWh?=
 =?us-ascii?Q?GX7MzSFl1GtsfJTpt8VdsT+BizeI9Srqk2u62Hc9MBVhGt1qg5H8trfVVzEX?=
 =?us-ascii?Q?t4XltXJukYvnbyB2T6Rs5hA/52m7h8LHYKJDeQg40HVy9ezDaq3fURe9zORE?=
 =?us-ascii?Q?ccdfGH0Ymfz5ySSDBONvyymP2nHN+ZHi9Nwcv4qyVVGh/cqi1M8JqrFHOuUk?=
 =?us-ascii?Q?dUn0oxwFbY05YUYU+RGCgSH77vruco6oXVAm4kc69GFh++6ToDRHHZ41z8ps?=
 =?us-ascii?Q?woVNHIDxynN6Z6ZmegZm2GUX+904GuOmLT9DkwCJOtZmHWmbwmCgMCHkQ41t?=
 =?us-ascii?Q?ke+fao0cHnyTwkF0zXDqGoOn9SeLRLDNQ1+eRp+0vs37w9zeeHcqdz1eBusb?=
 =?us-ascii?Q?uaVyCixrRODJA7EY6OzzXo+z042bxRI/pj0Ey2lGbr/iTWto2I1L3KdhSEQq?=
 =?us-ascii?Q?D2PItbp1pG/5LMp2rOhyTmow+sObF6+XFpIelktjZhtLqBOJ9amjTcAYL+M6?=
 =?us-ascii?Q?EiBHombDB2QQA+v/n0qZkZrjmfPA+PW6H1QV4MsDojfPfEu9OMIFdS2mo4L9?=
 =?us-ascii?Q?ToY7pLF6lJWvauTfwMfhEcRx/GyPXJgLmKKmAJ/SM1zBRPStrhyjpxQTlwIt?=
 =?us-ascii?Q?zbVDetlZuBfM8kYD8HkNv1AqHIF21L2dKVuF3kdeIBO+kXCOhoHhyb5g4X0C?=
 =?us-ascii?Q?HdMc8mNT+JXC7X+PTDaZr8RjYVauNRG9DxDhuLR1a+bZk9wAKteenwLc8zhx?=
 =?us-ascii?Q?suzIioSDc+hkzTRo9B37nBG6jLfcHH+WH50wi5fQcOOxxIEE7HnEmzfQE/m0?=
 =?us-ascii?Q?idUpfqZgI1+zjq2x+layde+my9S3Dcp0rp1hL0rbu1gOX/rnqJbU7JfeWgts?=
 =?us-ascii?Q?R7XxjMtPZgY8NUfaZtirqPb8lLg/bnUQKwD4DHt0X+twsQp9spDGAnr+efXX?=
 =?us-ascii?Q?rgtcs63n3vvUuH//aXV0gwdJfbNxwf27z+KfqANPwFOi9v32hXhCmf7WHfgf?=
 =?us-ascii?Q?Ki9XeL4X0ZiF2VB4pLVAj9vhsA5B3KTNrdP3dIWW0HDLpliSDU4pg+tcT/+i?=
 =?us-ascii?Q?/kAjIc6arfnkKXsUFRmcfNGPg9K/aQifIB1qiRCWaH9PyZclXrebRxxpm7lD?=
 =?us-ascii?Q?NkwU2dD1K4uc/WGX+J8tQICg4TdyR0/BZCwGzZhMCUJDmzEr6Sa/lyRQ1YFv?=
 =?us-ascii?Q?VTLwZJQtK9magHXFYNmA2J+w?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR04MB6575.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fc16b692-69d8-4ab3-9f77-08d98312efbf
X-MS-Exchange-CrossTenant-originalarrivaltime: 29 Sep 2021 06:32:40.3354
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: SzqMRTagJhi4rcIbAP4X4fHYeMJL9juBxQS254H17I4B8r/8u5KwyHJfK1O6maKoXCIcJDzsPIEAiTGEJv5zQA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR04MB4843
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> The current what expressions:
>         What:          /sys/bus/platform/drivers/ufshcd/*/...
>=20
> Doesn't actually match what (some?) platform drivers actually
> export. For instance, drivers/scsi/ufs/ufs-hisi.c actually
> creates the sysfs struct for ufshcd inside this directory:
>=20
>         /sys/devices/platform/soc/ff3c0000.ufs
>=20
> Which has those aliases:
>=20
>         /sys/devices/virtual/devlink/platform:fff35000.crg_ctrl--
> platform:ff3c0000.ufs/consumer/
>         /sys/bus/platform/drivers/ufshcd-hisi/ff3c0000.ufs/
>         /sys/bus/platform/devices/soc/ff3c0000.ufs/
>         /sys/bus/platform/devices/ff3c0000.ufs/
>=20
> So, add another What: for such files that will match the
> device ufs entries, e. g.:
>=20
>         What:          /sys/bus/platform/devices/*.ufs/
On some platforms the suffix is .ufshc
So Maybe use:
/sys/bus/platform/devices/*.ufs*/

Thanks,
Avri
>=20
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>