Re: [PATCH v2 1/3] misc: servo-pwm: driver for controlling servo motors via PWM

Greg Kroah-Hartman
2023-02-17T15:48:32Z

On Fri, Feb 17, 2023 at 04:39:01PM +0100, Angelo Compagnucci wrote:
> On Fri, Feb 17, 2023 at 4:36 PM Greg Kroah-Hartman <
> gregkh@linuxfoundation.org> wrote:
>
> > On Fri, Feb 17, 2023 at 03:57:28PM +0100, Angelo Compagnucci wrote:
> > > This patch adds a simple driver to control servo motor position via
> > > PWM signal.
> > > The driver allows to set the angle from userspace, while min/max
> > > positions duty cycle and the motor degrees aperture are defined in
> > > the dts.
> > >
> > > Signed-off-by: Angelo Compagnucci <angelo@amarulasolutions.com>
> > > ---
> > > MAINTAINERS | 6 ++
> > > drivers/misc/Kconfig | 11 +++
> > > drivers/misc/Makefile | 1 +
> > > drivers/misc/servo-pwm.c | 149 +++++++++++++++++++++++++++++++++++++++
> >
> > You add sysfs files but do not document them in Documentation/ABI/ which
> > is required. Please do so in your next version of this patch.
> >
>
> https://patchwork.ozlabs.org/project/linux-pwm/patch/20230217145731.3018148-2-angelo@amarulasolutions.com/
>
> This patch [2/3] should do what you're asking.

Why was I only cc:ed on this one patch?

confused,

greg k-h