From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nishanth Menon Subject: Re: [RFC][PATCH] OMAP3: introduce OMAP3630 Date: Thu, 8 Oct 2009 09:40:35 -0500 Message-ID: <4ACDF9E3.4010701@ti.com> References: <[PATCH][RFC] OMAP3630: Create architecture macros and config> <1254977223-4236-1-git-send-email-nm@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from comal.ext.ti.com ([198.47.26.152]:46181 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932366AbZJHOlP (ORCPT ); Thu, 8 Oct 2009 10:41:15 -0400 In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Premi, Sanjeev" Cc: "Pandita, Vikram" , "Shilimkar, Santosh" , linux-omap , "Chikkature Rajashekar, Madhusudhan" , "Pais, Allen" , "Gadiyar, Anand" , "Cousson, Benoit" , Kevin Hilman , "Aguirre Rodriguez, Sergio Alberto" , Tony Lindgren Premi, Sanjeev had written, on 10/08/2009 09:23 AM, the following: >> -----Original Message----- >> From: Pandita, Vikram >> Sent: Thursday, October 08, 2009 7:01 PM >> To: Shilimkar, Santosh; Menon, Nishanth; linux-omap >> Cc: Chikkature Rajashekar, Madhusudhan; Pais, Allen; Gadiyar, >> Anand; Cousson, Benoit; Kevin Hilman; Premi, Sanjeev; Aguirre >> Rodriguez, Sergio Alberto; Tony Lindgren >> Subject: RE: [RFC][PATCH] OMAP3: introduce OMAP3630 >> >> >> >>> -----Original Message----- >>> From: Shilimkar, Santosh >>>> diff --git a/arch/arm/plat-omap/include/mach/cpu.h b/arch/arm/plat- >>>> omap/include/mach/cpu.h >>>> index 431fec4..af1080f 100644 >>>> --- a/arch/arm/plat-omap/include/mach/cpu.h >>>> +++ b/arch/arm/plat-omap/include/mach/cpu.h >>>> @@ -383,6 +383,12 @@ IS_OMAP_TYPE(3430, 0x3430) >>>> #define OMAP3430_REV_ES2_1 0x34302034 >>>> #define OMAP3430_REV_ES3_0 0x34303034 >>>> #define OMAP3430_REV_ES3_1 0x34304034 >>>> +/* NOTE: Add 36xx series below >>>> + * If additional 34xx series are added, OMAP3430_REV_ESXXXX can be >>>> + * added above the 3630 defines and series renumbered to ensure >>>> + * rev() > checks to work >>>> + */ >>>> +#define OMAP3630_REV_ES1_0 0x34305034 >>>> >>>> #define OMAP443X_CLASS 0x44300034 >>> Was expecting that this patch will add "cpu_is_omap36xx()" in cpu.h >>> apart from above. Is this handled in another patch ? >> Idea is to re-use all 34xx code for 36xx, as per the mail >> thread on list, and given in reference. >> Hence at run time, the check could be: >> >> if (omap_rev() == OMAP3630_REV_ES1_0) >> xxxxx >> >> cpu_is_omap34xx() will be true for 36xx as well. > > [sp] This case seems quite similar to the OMAP35x. > Can you look at this thread: > > http://marc.info/?l=linux-omap&m=125372581804902&w=2 > > It applies equally well here as well... > I will be submitting updated patch tomorrow. yes, any specifics should be feature based IMHO. we will need to extend the feature list. -- Regards, Nishanth Menon