From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757303AbbCCXrq (ORCPT ); Tue, 3 Mar 2015 18:47:46 -0500 Received: from mailserver04.net.ipp.pt ([193.136.58.124]:44370 "EHLO mailserver04.net.ipp.pt" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756661AbbCCXro (ORCPT ); Tue, 3 Mar 2015 18:47:44 -0500 X-AuditID: c1883a7c-f799e6d000001238-28-54f6481dc3e5 From: =?UTF-8?q?Cl=C3=A1udio=20Maia?= To: thomas.petazzoni@free-electrons.com, noralf@tronnes.org, gregkh@linuxfoundation.org Cc: devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCH 2/2] Staging: fbtft: fb_pcd8544: Fixed coding style errors and warnings Date: Tue, 3 Mar 2015 23:47:19 +0000 Message-Id: <1425426439-17358-2-git-send-email-crrm@isep.ipp.pt> X-Mailer: git-send-email 1.7.9.5 In-Reply-To: <1425426439-17358-1-git-send-email-crrm@isep.ipp.pt> References: <1425426439-17358-1-git-send-email-crrm@isep.ipp.pt> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrCIsWRmVeSWpSXmKPExsVysMOqSlfW41uIwdJnphaXd81hc2D0+LxJ LoAxissmJTUnsyy1SN8ugSvj5OT5LAXf5Som3v3F3MC4UKSLkZNDQsBEYv+rj6wQtpjEhXvr 2boYuTiEBM4zSjw8/ZANJCEk4C6x4sgKdojEciaJJzP/MoIk2AQsJK7e62QGsUUEoiX+LnwM NolZwE7i/KOdLCC2sEC4xK0zV8BsFgFVid6dq8BqeAUcJB7fP83UxcgBtFlBYs4kG5Awp4Cj xJm/95kg9jpIfHi1jB2iXFDi5MwnLCDlzALqEuvnCUFskpdo3jqbeQKj4CwkVbMQqmYhqVrA yLyKUTw3MTOnOLUIGCwGJnp5qSV6mQUFegUlmxghgVmzg/HfG8tDjAIcjEo8vC84v4YIsSaW FVfmHmKU5GBSEuV1sfgWIsSXlJ9SmZFYnBFfVJqTWnyIUYKDWUmE97EBUI43JbGyKrUoHyYl zcGiJM6buSYzQEggPbEkNTs1tSC1CCYrw8GhJMG73g2oUbAoNT21Ii0zpwQhzcTBCTKcB2j4 CZAa3uKCxNzizHSI/ClGRSlx3pcgCQGQREZpHlwvLHG8YhQHekWY18sdqIoHmHTgul8BDWYC GnxL8QvI4JJEhJRUA2Pb6dd6rmsaDE6Y6MpfY9px8U2XTeh9z4vuZwRvHU/9sPm6xLY7zx0/ hG/59GCqgoZhusrvlf+LBP+9WqvjkqFXZnOy0pplWsqPX0+XPbR5cdkwVuHAtfcMaW81H8kd 9DoWq5jrdDJqW/+vuSF9abMmMq+Z8/yHdEjN/0RX+zW3Zztcindqa1ykxFKckWioxVxUnAgA 6zzVtfcCAAA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patch fixes the following errors by refactoring the comments' style: WARNING: line over 80 characters ERROR: code indent should use tabs where possible Signed-off-by: Cláudio Maia --- drivers/staging/fbtft/fb_pcd8544.c | 107 ++++++++++++++++++++---------------- 1 file changed, 59 insertions(+), 48 deletions(-) diff --git a/drivers/staging/fbtft/fb_pcd8544.c b/drivers/staging/fbtft/fb_pcd8544.c index f25053c..fddb9cd 100644 --- a/drivers/staging/fbtft/fb_pcd8544.c +++ b/drivers/staging/fbtft/fb_pcd8544.c @@ -34,7 +34,7 @@ #define WIDTH 84 #define HEIGHT 48 #define TXBUFLEN (84*6) -#define DEFAULT_GAMMA "40" /* gamma is used to control contrast in this driver */ +#define DEFAULT_GAMMA "40" /* gamma controls the contrast in this driver */ static unsigned tc; module_param(tc, uint, 0); @@ -51,43 +51,50 @@ static int init_display(struct fbtft_par *par) par->fbtftops.reset(par); - /* Function set */ - write_reg(par, 0x21); /* 5:1 1 - 2:0 PD - Powerdown control: chip is active - 1:0 V - Entry mode: horizontal addressing - 0:1 H - Extended instruction set control: extended - */ - - /* H=1 Temperature control */ - write_reg(par, 0x04 | (tc & 0x3)); /* - 2:1 1 - 1:x TC1 - Temperature Coefficient: 0x10 - 0:x TC0 - */ - - /* H=1 Bias system */ - write_reg(par, 0x10 | (bs & 0x7)); /* - 4:1 1 - 3:0 0 - 2:x BS2 - Bias System - 1:x BS1 - 0:x BS0 - */ - - /* Function set */ - write_reg(par, 0x22); /* 5:1 1 - 2:0 PD - Powerdown control: chip is active - 1:1 V - Entry mode: vertical addressing - 0:0 H - Extended instruction set control: basic - */ - - /* H=0 Display control */ - write_reg(par, 0x08 | 4); /* - 3:1 1 - 2:1 D - DE: 10=normal mode - 1:0 0 - 0:0 E - */ + /* Function set + * + * 5:1 1 + * 2:0 PD - Powerdown control: chip is active + * 1:0 V - Entry mode: horizontal addressing + * 0:1 H - Extended instruction set control: extended + */ + write_reg(par, 0x21); + + /* H=1 Temperature control + * + * 2:1 1 + * 1:x TC1 - Temperature Coefficient: 0x10 + * 0:x TC0 + */ + write_reg(par, 0x04 | (tc & 0x3)); + + /* H=1 Bias system + * + * 4:1 1 + * 3:0 0 + * 2:x BS2 - Bias System + * 1:x BS1 + * 0:x BS0 + */ + write_reg(par, 0x10 | (bs & 0x7)); + + /* Function set + * + * 5:1 1 + * 2:0 PD - Powerdown control: chip is active + * 1:1 V - Entry mode: vertical addressing + * 0:0 H - Extended instruction set control: basic + */ + write_reg(par, 0x22); + + /* H=0 Display control + * + * 3:1 1 + * 2:1 D - DE: 10=normal mode + * 1:0 0 + * 0:0 E + */ + write_reg(par, 0x08 | 4); return 0; } @@ -97,16 +104,20 @@ static void set_addr_win(struct fbtft_par *par, int xs, int ys, int xe, int ye) fbtft_par_dbg(DEBUG_SET_ADDR_WIN, par, "%s(xs=%d, ys=%d, xe=%d, ye=%d)\n", __func__, xs, ys, xe, ye); - /* H=0 Set X address of RAM */ - write_reg(par, 0x80); /* 7:1 1 - 6-0: X[6:0] - 0x00 - */ - - /* H=0 Set Y address of RAM */ - write_reg(par, 0x40); /* 7:0 0 - 6:1 1 - 2-0: Y[2:0] - 0x0 - */ + /* H=0 Set X address of RAM + * + * 7:1 1 + * 6-0: X[6:0] - 0x00 + */ + write_reg(par, 0x80); + + /* H=0 Set Y address of RAM + * + * 7:0 0 + * 6:1 1 + * 2-0: Y[2:0] - 0x0 + */ + write_reg(par, 0x40); } static int write_vmem(struct fbtft_par *par, size_t offset, size_t len) -- 1.7.9.5