From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751834AbdCAFX5 (ORCPT ); Wed, 1 Mar 2017 00:23:57 -0500 Received: from mail-out.m-online.net ([212.18.0.10]:46736 "EHLO mail-out.m-online.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750747AbdCAFX1 (ORCPT ); Wed, 1 Mar 2017 00:23:27 -0500 X-Auth-Info: VJaoc1EmByfc0SVRo6udSVKgQRRIJQkBMaWZ5+YETg4= Reply-To: hs@denx.de Subject: Re: [v2] Input: pwm-beeper: support customized freq for SND_BELL References: <1487579863-7256-1-git-send-email-hs@denx.de> <06af466d-ac78-ea38-77d2-3bf32a6e157c@lechnology.com> <58B5084A.9000709@denx.de> <58B51517.4050003@denx.de> To: Dmitry Torokhov Cc: David Lechner , linux-input@vger.kernel.org, Guan Ben , Mark Jonas , devicetree@vger.kernel.org, Thierry Reding , linux-kernel@vger.kernel.org, Boris Brezillon , Rob Herring , Manfred Schlaegl , Mark Rutland From: Heiko Schocher Message-ID: <58B65687.3090801@denx.de> Date: Wed, 1 Mar 2017 06:05:11 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Dimitry, Am 28.02.2017 um 07:20 schrieb Dmitry Torokhov: > On February 27, 2017 10:13:43 PM PST, Heiko Schocher wrote: >> Hello David, >> >> Am 28.02.2017 um 06:30 schrieb David Lechner: >>> On 02/27/2017 11:19 PM, Heiko Schocher wrote: >>>> Hello David, >>>> >>>> Am 27.02.2017 um 19:11 schrieb David Lechner: >>>>> On 02/20/2017 02:37 AM, Heiko Schocher wrote: >>>>>> From: Guan Ben >>>>>> >>>>>> extend the pwm-beeper driver to support customized frequency >>>>>> for SND_BELL from device tree. >>>>>> >>>>>> Signed-off-by: Guan Ben >>>>>> Signed-off-by: Mark Jonas >>>>>> [hs@denx.de: adapted to 4.10-rc7] >>>>>> Signed-off-by: Heiko Schocher >>>>>> --- >>>>>> >>>>>> Changes in v2: >>>>>> - add comment from Rob Herring: >>>>>> rename property name "bell-frequency" to "beeper-hz" >>>>> >>>>> Is there a separate patch for the devicetree bindings >> documentation? >>>> >>>> No, it is in this patch ... In the meantime I got an >>>> Acked-by from Rob Herring ... >>>> >>> >>> What I should have said is that, in general, I think it is preferred >> to have the device tree >>> bindings as a separate patch from the code changes. >> >> I can split this into 2 patches ... > > Don't - since Rob already asked DT change you don't need to post to device tree list again and I'll merge them back together since they do not make sense separately. Ok, I post a v3 with the changes suggested from David and Rob soon, without splitting. Thanks! bye, Heiko -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany