From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Subject: RE: [PATCH v2 05/17] OMAP2,3 DSS Driver register moved to mach_omap2 Date: Wed, 22 Dec 2010 13:36:00 +0200 Message-ID: <1293017760.6829.11.camel@tubuntu> References: <1291031503-21036-1-git-send-email-svadivu@ti.com> <1291031503-21036-6-git-send-email-svadivu@ti.com> <1292939360.22441.57.camel@tubuntu> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from smtp.nokia.com ([147.243.128.24]:50396 "EHLO mgw-da01.nokia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751597Ab0LVLgH (ORCPT ); Wed, 22 Dec 2010 06:36:07 -0500 In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "ext Guruswamy, Senthilvadivu" Cc: "khilman@deeprootsystems.com" , "paul@pwsan.com" , "Hiremath, Vaibhav" , "linux-omap@vger.kernel.org" On Wed, 2010-12-22 at 16:55 +0530, ext Guruswamy, Senthilvadivu wrote: > > > -----Original Message----- > > From: Tomi Valkeinen [mailto:tomi.valkeinen@nokia.com] > > Sent: Tuesday, December 21, 2010 7:19 PM > > To: Guruswamy, Senthilvadivu > > Cc: khilman@deeprootsystems.com; paul@pwsan.com; Hiremath, Vaibhav; linux- > > omap@vger.kernel.org > > Subject: Re: [PATCH v2 05/17] OMAP2,3 DSS Driver register moved to > > mach_omap2 > > > > On Mon, 2010-11-29 at 17:21 +0530, ext Guruswamy Senthilvadivu wrote: > > > From: Senthilvadivu Guruswamy > > > > > > Move DSS driver register from board file to devices.c > > > Changed device name from omapdss to omap_display as the > > > driver takes care of panel information. > > > > Here you do two separate things, adding the omap_display_init() and > > changing the driver name. The latter should be in the previous patch, I > > suppose. > [Senthil] Yes, I can change the driver name first and then omap_display_init() as 2 small patches. When I do this I have to do similarly for board files too. Then the board file related changes alone would be 3 separate patches - 1. driver name change, 2. omap_display_init, 3. Regulator initialization being done with driver name > > If this is fine, then I can send v3 version of the patch set. Try to do one logical change in one patch, so for example when you change the driver name, change it in the clock files, board files and in the driver. That way the kernel stays whole after each patch. Tomi