From mboxrd@z Thu Jan 1 00:00:00 1970 From: Santosh Shilimkar Subject: Re: [GIT PULL] ARM: OMAP5: hwmod, prm/cm data files and updates for 3.10 Date: Mon, 15 Apr 2013 11:50:04 +0530 Message-ID: <516B9C14.5070406@ti.com> References: <1363768852-18125-1-git-send-email-santosh.shilimkar@ti.com> <20130401170539.GK10155@atomide.com> <515BA76D.3000302@ti.com> <515D6020.2030103@ti.com> <20130404165208.GI10155@atomide.com> <515DB110.9000503@ti.com> <515F00B8.3000501@ti.com> <79CD15C6BA57404B839C016229A409A83EC4A81D@DBDE01.ent.ti.com> <51654DC5.2040909@ti.com> <79CD15C6BA57404B839C016229A409A83EC55521@DBDE01.ent.ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: Received: from bear.ext.ti.com ([192.94.94.41]:40602 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753030Ab3DOGR6 (ORCPT ); Mon, 15 Apr 2013 02:17:58 -0400 In-Reply-To: <79CD15C6BA57404B839C016229A409A83EC55521@DBDE01.ent.ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Hiremath, Vaibhav" Cc: Tony Lindgren , Paul Walmsley , "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "Kristo, Tero" , "Menon, Nishanth" , "Nayak, Rajendra" , "Valentin, Eduardo" , "Anna, Suman" , "Bedia, Vaibhav" On Monday 15 April 2013 10:36 AM, Hiremath, Vaibhav wrote: > >> -----Original Message----- >> From: Shilimkar, Santosh >> Sent: Wednesday, April 10, 2013 5:02 PM [..] >>>> From b95dd33fe59b8e77727eb3b1717d763bbf9a2893 Mon Sep 17 00:00:00 >> 2001 >>>> From: Sricharan R >>>> Date: Fri, 5 Apr 2013 20:39:12 +0530 >>>> Subject: [PATCH] ARM: AM33XX: hwmod data: Clean up the data file >>>> >>>> - The IO resource information like dma request lines, irq number and >>>> ocp address space can be populated via dt blob. So such data can be >>>> stripped >>>> from SOC hwmod data file. >>>> >>>> - The devices like adc, mailbox, gpmc which are missing the device >>>> tree bindings, hwmod data is not added since AM33XX is DT only >> build. >>>> When such devices add the dt bindings, respective hwmod data can be >>>> added along with it. >>>> >>> This seems unnecessary churn to me. DT bindings for most of the >> devices >>> which you mentioned above are submitted and are at various stages of >> review >>> process. >>> >>> ADC: >>> >>> GPMC: >>> >>> PWM: >>> >> The modules are dropped as per what is going for 3.10 merge window. >> Above 3 modules can be retained if the DT conversion patches are >> under review and can go along with this patch most likely for 3.11. >> >>> >>>> - The hwmod like firewall etc which are not useful are also dropped. >>>> >>>> This gets us around ~2000 loc of negative diff. Patch is boot tested >> on >>>> AM335X EVM. >>>> >>> I would not recommend to get into unnecessary code churn in the >> future just >>> to reduce temp Number of Lines of code. This will also kill our >> autogeneration >>> concept as well. >>> >> It doesn't break any concept. We just autogenrate what is *useful* >> rather. >> BTW, I didn't find any srcipt to auto-generate the AM33XX data so we >> have >> to manually do the updates. Can you send me a pointer if you have a >> sript >> for this. With script it is much simpler to clean-up the data. >> >> >>> I would suggest you to just alone drop base-addr, irq and dma >> references >>> from hwmod entries. >>> >> That we are doing anyways. Apart from that we should also clean-up data >> which is not used and useful. Why do you need unused data like firewall >> and >> friends ? >> >> So as I understood, you would like to keep the data for ADC, PWM and >> GPMC >> which is fine by me. We just need those DT bindings in place so that >> they >> go together. Who is following the DT patches for these ? >> >> Thanks for looking into it Vaibhav. >> > Are you planning to send updated version of this? > I would rather prefer to review next version. > > Please let me know if you need any help here. > Yes :-) It will be great if you take the patch forward and update it based on the discussion. Regards, Santosh