From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1754200AbbCBNPh (ORCPT <rfc822;w@1wt.eu>);
	Mon, 2 Mar 2015 08:15:37 -0500
Received: from mail-wg0-f50.google.com ([74.125.82.50]:36929 "EHLO
	mail-wg0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751483AbbCBNPg (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Mon, 2 Mar 2015 08:15:36 -0500
Message-ID: <1425302132.3928.17.camel@gmail.com>
Subject: Re: [PATCH V2] Staging: fbtft: fix coding style errors
From: Matteo Semenzato <mattew8898@gmail.com>
Reply-To: 20150302092904.GD5386@mwanda
To: Dan Carpenter <dan.carpenter@oracle.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Date: Mon, 02 Mar 2015 14:15:32 +0100
In-Reply-To: <20150302092904.GD5386@mwanda>
References: <1425133894-28251-1-git-send-email-mattew8898@gmail.com>
	 <1425145050-486-1-git-send-email-mattew8898@gmail.com>
	 <20150302092904.GD5386@mwanda>
Content-Type: text/plain; charset="UTF-8"
X-Mailer: Evolution 3.12.11 
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Il giorno lun, 02/03/2015 alle 12.29 +0300, Dan Carpenter ha scritto:
> On Sat, Feb 28, 2015 at 06:37:30PM +0100, Matteo Semenzato wrote:
> >  static int hy28b_init_sequence[] = {
> > -	-1,0x00e7,0x0010,-1,0x0000,0x0001,-1,0x0001,0x0100,-1,0x0002,0x0700,
> > -	-1,0x0003,0x1030,-1,0x0004,0x0000,-1,0x0008,0x0207,-1,0x0009,0x0000,
> > -	-1,0x000a,0x0000,-1,0x000c,0x0001,-1,0x000d,0x0000,-1,0x000f,0x0000,
> > -	-1,0x0010,0x0000,-1,0x0011,0x0007,-1,0x0012,0x0000,-1,0x0013,0x0000,
> > -	-2,50,-1,0x0010,0x1590,-1,0x0011,0x0227,-2,50,-1,0x0012,0x009c,-2,50,
> > -	-1,0x0013,0x1900,-1,0x0029,0x0023,-1,0x002b,0x000e,-2,50,
> > -	-1,0x0020,0x0000,-1,0x0021,0x0000,-2,50,-1,0x0050,0x0000,
> > -	-1,0x0051,0x00ef,-1,0x0052,0x0000,-1,0x0053,0x013f,-1,0x0060,0xa700,
> > -	-1,0x0061,0x0001,-1,0x006a,0x0000,-1,0x0080,0x0000,-1,0x0081,0x0000,
> > -	-1,0x0082,0x0000,-1,0x0083,0x0000,-1,0x0084,0x0000,-1,0x0085,0x0000,
> > -	-1,0x0090,0x0010,-1,0x0092,0x0000,-1,0x0093,0x0003,-1,0x0095,0x0110,
> > -	-1,0x0097,0x0000,-1,0x0098,0x0000,-1,0x0007,0x0133,-1,0x0020,0x0000,
> > -	-1,0x0021,0x0000,-2,100,-3 };
> > +	-1, 0x00e7, 0x0010,
> > +	-1, 0x0000, 0x0001,
> > +	-1, 0x0001, 0x0100,
> > +	-1, 0x0002, 0x0700,
> > +	-1, 0x0003, 0x1030,
> > +	-1, 0x0004, 0x0000,
> > +	-1, 0x0008, 0x0207,
> > +	-1, 0x0009, 0x0000,
> > +	-1, 0x000a, 0x0000,
> > +	-1, 0x000c, 0x0001,
> > +	-1, 0x000d, 0x0000,
> > +	-1, 0x000f, 0x0000,
> > +	-1, 0x0010, 0x0000,
> > +	-1, 0x0011, 0x0007,
> > +	-1, 0x0012, 0x0000,
> > +	-1, 0x0013, 0x0000,
> > +	-2, 50,
> > +	-1, 0x0010, 0x1590,
> > +	-1, 0x0011, 0x0227,
> > +	-2, 50,
> > +	-1, 0x0012, 0x009c,
> > +	-2, 50,
> > +	-1, 0x0013, 0x1900,
> > +	-1, 0x0029, 0x0023,
> > +	-1, 0x002b, 0x000e,
> > +	-2, 50,
> > +	-1, 0x0020, 0x0000,
> > +	-1, 0x0021, 0x0000,
> > +	-2, 50,
> > +	-1, 0x0050, 0x0000,
> > +	-1, 0x0051, 0x00ef,
> > +	-1, 0x0052, 0x0000,
> > +	-1, 0x0053, 0x013f,
> > +	-1, 0x0060, 0xa700,
> > +	-1, 0x0061, 0x0001,
> > +	-1, 0x006a, 0x0000,
> > +	-1, 0x0080, 0x0000,
> > +	-1, 0x0081, 0x0000,
> > +	-1, 0x0082, 0x0000,
> > +	-1, 0x0083, 0x0000,
> > +	-1, 0x0084, 0x0000,
> > +	-1, 0x0085, 0x0000,
> > +	-1, 0x0090, 0x0010,
> > +	-1, 0x0092, 0x0000,
> > +	-1, 0x0093, 0x0003,
> > +	-1, 0x0095, 0x0110,
> > +	-1, 0x0097, 0x0000,
> > +	-1, 0x0098, 0x0000,
> > +	-1, 0x0007, 0x0133,
> > +	-1, 0x0020, 0x0000,
> > +	-1, 0x0021, 0x0000,
> > +	-2, 100,
> > +	-3
> > +};
> 
> For some reason you put the line breaks at the negative numbers?  Why?
> 
> regards,
> dan carpenter
> 
-1, -2 and -3 are the init sequence markers: -1: command begin, -2:
millisecond delay, -3: end of init sequence