[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y++hw3QTma3BGpwB@kroah.com>
Date: Fri, 17 Feb 2023 16:48:19 +0100
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: Angelo Compagnucci <angelo@...rulasolutions.com>
Cc: Angelo Compagnucci <angelo.compagnucci@...il.com>,
Derek Kiernan <derek.kiernan@...inx.com>,
Dragan Cvetic <dragan.cvetic@...inx.com>,
Arnd Bergmann <arnd@...db.de>,
Thierry Reding <thierry.reding@...il.com>,
Uwe Kleine-König
<u.kleine-koenig@...gutronix.de>, linux-kernel@...r.kernel.org,
linux-pwm@...r.kernel.org
Subject: Re: [PATCH v2 1/3] misc: servo-pwm: driver for controlling servo
motors via PWM
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@...uxfoundation.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@...rulasolutions.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
Powered by blists - more mailing lists