From mboxrd@z Thu Jan 1 00:00:00 1970 From: Venkatraman S Subject: Re: [PATCH v8 2/2] omap hsmmc: adaptation of sdma descriptor autoloading feature Date: Sat, 15 May 2010 00:13:06 +0530 Message-ID: References: <004501caed38$03e7b9f0$544ff780@am.dhcp.ti.com> <000601caee06$b2c8a9b0$544ff780@am.dhcp.ti.com> <4BE4694F.3060901@ti.com> <4BE6DD8D.6030201@gmail.com> <4BE8056C.6080106@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mail-pv0-f174.google.com ([74.125.83.174]:43007 "EHLO mail-pv0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757445Ab0ENSnH convert rfc822-to-8bit (ORCPT ); Fri, 14 May 2010 14:43:07 -0400 In-Reply-To: <4BE8056C.6080106@ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Nishanth Menon , "Chikkature Rajashekar, Madhusudhan" , "Shilimkar, Santosh" , kishore kadiyala Nishanth Menon wrote: > On 05/10/2010 07:31 AM, Venkatraman S wrote: >> >> Nishanth Menon =A0wrote: > >>>>> Please see [1] for SOC specific feature handling. any reasons we = can't >>>>> handle it by adding a new feature? >>>>> >>>>> [1] >>>>> >>>>> >>>>> http://git.kernel.org/?p=3Dlinux/kernel/git/torvalds/linux-2.6.gi= t;a=3Dblob;f=3Darch/arm/plat-omap/include/plat/cpu.h#l439 >>>>> >>>> >>>> Thanks. I can add a new feature here, but I see that the API is ti= ed >>>> to OMAP3, whereas the DMA feature is common >>>> to 3630, OMAP4 and mostly everything after that. I can work on an >>>> upgrade, but do you see that >>>> as a dependency and done on the context of this patch ? >>>> Regards, >>>> Venkat. >>> >>> Yes, I am aware that the current APIs are tied to OMAP3, no reason = that >>> we >>> cant introduce a OMAP version independent feature.. Yes, IMHO, this= is an >>> SOC specific feature that has no place in a platform data.. lets no= t >>> misuse >>> that. >>> Regards, >>> NM >> Just a note that I have updated my patch series based on all the comments except that the 'features' framework has to be created to be utilised by this patch series. That would be a separate patch, which I am working on, and will post on monday. Regards, Venkat. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html