From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752330AbbCYWN0 (ORCPT ); Wed, 25 Mar 2015 18:13:26 -0400 Received: from mailgw1.uni-kl.de ([131.246.120.220]:44767 "EHLO mailgw1.uni-kl.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752260AbbCYWNW convert rfc822-to-8bit (ORCPT ); Wed, 25 Mar 2015 18:13:22 -0400 Date: Wed, 25 Mar 2015 21:10:59 +0100 From: Thomas =?UTF-8?B?TmllZGVycHLDvG0=?= To: Maxime Ripard Cc: plagnioj@jcrosoft.com, tomi.valkeinen@ti.com, kernel@pengutronix.de, shawn.guo@linaro.org, robh+dt@kernel.org, linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCHv5 09/11] fbdev: ssd1307fb: Add module parameter to set the initial contrast Message-ID: <20150325211059.1ae68f8e@maestro.intranet> In-Reply-To: <20150324221636.GE23664@lukather> References: <1423261694-5939-1-git-send-email-niederp@physik.uni-kl.de> <1427232238-21099-1-git-send-email-niederp@physik.uni-kl.de> <1427232238-21099-10-git-send-email-niederp@physik.uni-kl.de> <20150324221636.GE23664@lukather> Organization: TU Kaiserslautern X-Mailer: Claws Mail 3.10.1 (GTK+ 2.24.26; i686-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Tue, 24 Mar 2015 15:16:36 -0700 schrieb Maxime Ripard : > On Tue, Mar 24, 2015 at 10:23:56PM +0100, Thomas Niederprüm wrote: > > This patch adds the module parameter "contrast" to determine the > > contrast value that is used to initialize the display. This > > setting applies to all instances of the driver. > > > > Signed-off-by: Thomas Niederprüm > > Do we still need this patch given that the next one is doing the exact > same thing? I was kind of expecting this objection. I added this module parameter since it suits my use case where I always run the display at the lowest possible contrast. With this module parameter I can specify this need at module load time and I do not need udev or to touch the backlight controls manually every time the driver is loaded. I see that this is probably a (too) weak argument, so if nobody complains I will remove this patch for the next round. Thomas