Received-SPF: pass (google.com: domain of linux-kernel+bounces-25576-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Precedence: bulk
MIME-Version: 1.0
References: <20240115004847.22369-1-warthog618@gmail.com> <20240115004847.22369-2-warthog618@gmail.com>
In-Reply-To: <20240115004847.22369-2-warthog618@gmail.com>
From: Andy Shevchenko <andy.shevchenko@gmail.com>
Date: Mon, 15 Jan 2024 03:11:07 +0200
Message-ID: <CAHp75Vc=oZK=VR6vZ+kTapq5WpySgKVjAfuO6mxcPqPfQe6VRg@mail.gmail.com>
Subject: Re: [PATCH v2 1/9] Documentation: gpio: add chardev userspace API documentation
To: Kent Gibson <warthog618@gmail.com>
Cc: linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, 
	linux-doc@vger.kernel.org, brgl@bgdev.pl, linus.walleij@linaro.org, 
	andy@kernel.org, corbet@lwn.net
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Mon, Jan 15, 2024 at 2:49=E2=80=AFAM Kent Gibson <warthog618@gmail.com> =
wrote:
>
> Add documentation for the GPIO character device userspace API.
>
> Added to the userspace-api book, but also provide a link from the
> admin-guide book, as historically the GPIO documentation has been
> there.

..

> +.. note::
> +   Do NOT abuse userspace APIs to control hardware that has proper kerne=
l
> +   drivers. There may already be a driver for your use case, and an exis=
ting
> +   kernel driver is sure to provide a superior solution to bitbashing
> +   from userspace.
> +
> +   Read Documentation/driver-api/gpio/drivers-on-gpio.rst to avoid reinv=
enting
> +   kernel wheels in userspace.
> +
> +   Similarly, for multi-function lines there may be other subsystems, su=
ch as
> +   Documentation/spi/index.rst, Documentation/i2c/index.rst,
> +   Documentation/driver-api/pwm.rst, Documentation/w1/index.rst etc, tha=
t
> +   provide suitable drivers and APIs for your hardware.

Very good note and would be nice to convince users to follow it:
https://stackoverflow.com/q/77683532/2511795
(OOH you might be proud it's Rust, OTOH as described above)

--=20
With Best Regards,
Andy Shevchenko