From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jon Hunter Subject: Re: [PATCH v5 04/14] ARM: OMAP2+: gpmc: minimal driver support Date: Wed, 13 Jun 2012 12:05:11 -0500 Message-ID: <4FD8C847.20002@ti.com> References: <54e643eb1b4dbefcbb52580fa582043bf4f0da3d.1339419492.git.afzal@ti.com> <4FD6586A.9060706@ti.com> <4FD782F7.9010003@ti.com> <20120613120717.GP12766@atomide.com> <20120613134032.GW12766@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from devils.ext.ti.com ([198.47.26.153]:50280 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752326Ab2FMRFN (ORCPT ); Wed, 13 Jun 2012 13:05:13 -0400 In-Reply-To: <20120613134032.GW12766@atomide.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Tony Lindgren Cc: "Mohammed, Afzal" , "paul@pwsan.com" , "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" On 06/13/2012 08:40 AM, Tony Lindgren wrote: > * Mohammed, Afzal [120613 06:16]: >> Hi Tony, >> >> On Wed, Jun 13, 2012 at 17:37:17, Tony Lindgren wrote: >>> * Jon Hunter [120612 11:01]: >>>> >>>> On 06/12/2012 02:16 AM, Mohammed, Afzal wrote: >> >>>>> Does having minor revision add any value ?, at least as of now, >>>>> I do not see any. >>>> >>>> May be not but it does not hurt either. >>> >>> Yeah might as well show the full revision. >> >> We are already showing full version, but only storing major, it seems you >> want to store minor version along with major version in the same variable, >> right ? > > Ah OK, yeah might as well just store the minor version. Actually, you really need the major version. However, IMO if you are going to store it, then store the whole thing, you have allocated 32-bits and the major and minor combined are 8-bits. Jon From mboxrd@z Thu Jan 1 00:00:00 1970 From: jon-hunter@ti.com (Jon Hunter) Date: Wed, 13 Jun 2012 12:05:11 -0500 Subject: [PATCH v5 04/14] ARM: OMAP2+: gpmc: minimal driver support In-Reply-To: <20120613134032.GW12766@atomide.com> References: <54e643eb1b4dbefcbb52580fa582043bf4f0da3d.1339419492.git.afzal@ti.com> <4FD6586A.9060706@ti.com> <4FD782F7.9010003@ti.com> <20120613120717.GP12766@atomide.com> <20120613134032.GW12766@atomide.com> Message-ID: <4FD8C847.20002@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 06/13/2012 08:40 AM, Tony Lindgren wrote: > * Mohammed, Afzal [120613 06:16]: >> Hi Tony, >> >> On Wed, Jun 13, 2012 at 17:37:17, Tony Lindgren wrote: >>> * Jon Hunter [120612 11:01]: >>>> >>>> On 06/12/2012 02:16 AM, Mohammed, Afzal wrote: >> >>>>> Does having minor revision add any value ?, at least as of now, >>>>> I do not see any. >>>> >>>> May be not but it does not hurt either. >>> >>> Yeah might as well show the full revision. >> >> We are already showing full version, but only storing major, it seems you >> want to store minor version along with major version in the same variable, >> right ? > > Ah OK, yeah might as well just store the minor version. Actually, you really need the major version. However, IMO if you are going to store it, then store the whole thing, you have allocated 32-bits and the major and minor combined are 8-bits. Jon