Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965249AbbD0VPB (ORCPT ); Mon, 27 Apr 2015 17:15:01 -0400 Received: from smtp43.i.mail.ru ([94.100.177.103]:41246 "EHLO smtp43.i.mail.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965095AbbD0VO6 (ORCPT ); Mon, 27 Apr 2015 17:14:58 -0400 X-Greylist: delayed 14609 seconds by postgrey-1.27 at vger.kernel.org; Mon, 27 Apr 2015 17:14:58 EDT Message-ID: <553EA6B9.5040309@list.ru> Date: Tue, 28 Apr 2015 00:14:33 +0300 From: Stas Sergeev User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Pavel Machek CC: linux-leds@vger.kernel.org, Linux kernel , Stas Sergeev Subject: Re: [PATCH v2 0/3] leds: blink resolution improvements References: <553E6CF5.4030601@list.ru> <20150427205441.GB1301@xo-6d-61-c0.localdomain> In-Reply-To: <20150427205441.GB1301@xo-6d-61-c0.localdomain> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam: Not detected X-Mras: Ok Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1622 Lines: 36 Hi. 27.04.2015 23:54, Pavel Machek пишет: > Hi! > >> The following patches improve the precision of led >> timer trigger and add the delay unit control. >> That allows to make PWM brightness control with timer >> trigger. > Are you sure that is good idea? Doing LED pwm with main cpu is quite harsh... Do you remember the pc-speaker driver? :) In fact I tried the maximal possible freq on armada-xp board (it doesn't go above approx 100KHz, perhaps this all the HW timers can do on that board) and I haven't measured any noticeable CPU load with htop. > We already have "brightness" for... well... brightness level. > > Would it make sense to have an option (Kconfig?) to do PWM using timer on hardware > that does not do hardware PWM? I think it would make sense, but not for the "timer" trigger. Maybe for "none" trigger, or inventing another trigger called "pwm". For the timer trigger I would pretty much like my approach to stay. The reason is that the PWM I need to do, is not strictly a PWM - it needs the ON period in range of tens or hundreds of milliseconds, while the OFF period is in a couple of usecs (or vice-versa). No generic PWM driver can afford such distribution, so I'd like to have the "timer" trigger for the unusual things like that. As I said, your idea can still be implemented for "none" trigger or some new trigger, so its still all for good. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/