From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756376AbbBGQCb (ORCPT ); Sat, 7 Feb 2015 11:02:31 -0500 Received: from mailgw1.uni-kl.de ([131.246.120.220]:32961 "EHLO mailgw1.uni-kl.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751714AbbBGQC3 convert rfc822-to-8bit (ORCPT ); Sat, 7 Feb 2015 11:02:29 -0500 Date: Sat, 7 Feb 2015 17:05:03 +0100 From: Thomas =?UTF-8?B?TmllZGVycHLDvG0=?= To: Maxime Ripard Cc: linux-fbdev@vger.kernel.org, plagnioj@jcrosoft.com, tomi.valkeinen@ti.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH 5/8] fbdev: ssd1307fb: Add module parameter bitsperpixel. Message-ID: <20150207170503.045490aa@maestro.intranet> In-Reply-To: <20150207112043.GO2079@lukather> References: <1423261694-5939-1-git-send-email-niederp@physik.uni-kl.de> <1423261694-5939-6-git-send-email-niederp@physik.uni-kl.de> <20150207112043.GO2079@lukather> Organization: TU Kaiserslautern X-Mailer: Claws Mail 3.10.1 (GTK+ 2.24.24; 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 Sat, 7 Feb 2015 12:20:43 +0100 schrieb Maxime Ripard : > On Fri, Feb 06, 2015 at 11:28:11PM +0100, niederp@physik.uni-kl.de > wrote: > > From: Thomas Niederprüm > > > > This patch adds a module parameter 'bitsperpixel' to adjust the > > colordepth of the framebuffer. All values >1 will result in memory > > map of the requested color depth. However only the MSB of each > > pixel will be sent to the device. The framebuffer identifies itself > > as a grayscale display with the specified depth. > > I'm not sure this is the right thing to do. > > The bits per pixel for this display is rightfully defined, used and > reported to the userspace, why would you want to change that? > You are right of course. The display is 1bpp and it reports to be 1 bpp. The problem is that there is almost no userspace library that can handle 1 bit framebuffers correctly. So it is nice if the framebuffer (optionally) can expose itself as 8 bits per pixel grayscale to the userspace program. As an example this allows to run DirectFB on the framebuffer, which is not possible out of the box for 1bpp. Also note that if do not set the module parameter at load time the framebuffer will be 1bpp. So you have to actively set that module parameter to make the framebuffer pretend to be more than 1bpp. In any case I don't cling to that patch, I just thought it was a nice feature. > Maxime >