Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 5 May 2023 11:51:03 +0700
From:   Bagas Sanjaya <bagasdotme@gmail.com>
To:     James Seo <james@equiv.tech>, Jean Delvare <jdelvare@suse.com>,
        Guenter Roeck <linux@roeck-us.net>,
        Jonathan Corbet <corbet@lwn.net>
Cc:     linux-hwmon@vger.kernel.org, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [RFC 07/11] Documentation/hwmon: Revise sysfs interface
 specification
Message-ID: <ZFSLN6KmGgfW4YzE@debian.me>
References: <20230504075752.1320967-1-james@equiv.tech>
 <20230504075752.1320967-8-james@equiv.tech>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="85j7XFAgyxDylWtE"
Content-Disposition: inline
In-Reply-To: <20230504075752.1320967-8-james@equiv.tech>
Precedence: bulk


--85j7XFAgyxDylWtE
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, May 04, 2023 at 12:57:48AM -0700, James Seo wrote:
> +**RO**    Read-only value.
> +**WO**    Write-only value.
> +**RW**    Read/write value.

Why are the permissions be emphasized?

> +| | ``pwm[1-*]_auto_point[1-*]_pwm``        | **RW** | PWM temperature c=
urve definition    |
> +| | ``pwm[1-*]_auto_point[1-*]_temp``       |        | for chips that as=
sociate trip       |
> +| | ``pwm[1-*]_auto_point[1-*]_temp_hyst``  |        | points with PWM o=
utput channels.    |
> ++-------------------------------------------+--------+------------------=
-------------------+
> +| | ``temp[1-*]_auto_point[1-*]_pwm``       | **RW** | PWM temperature c=
urve definition    |
> +| | ``temp[1-*]_auto_point[1-*]_temp``      |        | for chips that as=
sociate trip       |
> +| | ``temp[1-*]_auto_point[1-*]_temp_hyst`` |        | points with tempe=
rature channels.   |

Dunno if replacing line blocks with bullet lists should improve the semantic
of three sysfs files having the same description.

Thanks.

--=20
An old man doll... just what I always wanted! - Clara

--85j7XFAgyxDylWtE
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYKAB0WIQSSYQ6Cy7oyFNCHrUH2uYlJVVFOowUCZFSLNwAKCRD2uYlJVVFO
o/iiAP9iRiM37NSCrYnWxOpiAozg/UqXIm4W8AZVo/2uFptc/gD9Fc/hfDLuhUXw
J0N/LrUM66JwC/CC6kQm8Sd9buz6TgM=
=J291
-----END PGP SIGNATURE-----

--85j7XFAgyxDylWtE--