From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH v3 0/3] input: pwm-beeper: add feature to set volume level Date: Thu, 19 Jan 2017 13:37:00 -0800 Message-ID: <20170119213700.GC13542@dtor-ws> References: <0e2dec4d-7550-9495-f12a-020391183304@exceet.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-input-owner@vger.kernel.org To: Frieder Schrempf Cc: robh@kernel.org, pawel.moll@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, luis@debethencourt.com, linux-input@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, David Lechner List-Id: devicetree@vger.kernel.org On Thu, Jan 19, 2017 at 04:24:07PM +0100, Frieder Schrempf wrote: > Make the driver accept switching volume levels via sysfs. > This can be helpful if the beep/bell sound intensity needs > to be adapted to the environment of the device. > > The number of volume levels available and their values can > be specified via device tree (similar to pwm-backlight). > > The volume adjustment is done by changing the duty cycle of > the pwm signal. I wonder how this all will mesh up with beepers that have dedicated amplifiers (support is being added by David Lechner). Thanks. -- Dmitry