From mboxrd@z Thu Jan 1 00:00:00 1970 From: Igor Grinberg Subject: Re: [PATCH v3 3/3] ARM: OMAP: TI814X: Create board support and enable build for TI8148 EVM Date: Tue, 04 Oct 2011 11:00:44 +0200 Message-ID: <4E8ACB3C.6020401@compulab.co.il> References: <1317258570-30494-1-git-send-email-hemantp@ti.com> <4E88542B.4010206@compulab.co.il> <2A3DCF3DA181AD40BDE86A3150B27B6B03B4EC5D52@dbde02.ent.ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from 50.23.254.54-static.reverse.softlayer.com ([50.23.254.54]:44279 "EHLO softlayer.compulab.co.il" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755120Ab1JDJAy (ORCPT ); Tue, 4 Oct 2011 05:00:54 -0400 In-Reply-To: <2A3DCF3DA181AD40BDE86A3150B27B6B03B4EC5D52@dbde02.ent.ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Pedanekar, Hemant" Cc: "linux-omap@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Tony Lindgren , "Hilman, Kevin" , Paul Walmsley On 10/03/11 18:45, Pedanekar, Hemant wrote: > Hi Igor, > > Igor Grinberg wrote on Sunday, October 02, 2011 5:38 PM: > >> Hi Hemant, >> >> On 09/29/11 04:09, Hemant Pedanekar wrote: >>> This patch adds minimal support and build configuration for TI8148 EVM. >>> Also adds support for low level debugging on UART1 console on the EVM. >>> >>> Note that existing TI8168 EVM file (board-ti8168evm.c) is updated with >>> machine info for TI8148 EVM and renamed as board-ti81xxevm.c. >> >> Should we really rename the existing file? >> Shouldn't we just stick to the name of the file submitted first? >> (e.g. board-ti8168evm.c) and just add the support for the new >> TI8148 EVM in to the existing file? > > But won't this be misleading? Misleading? For whom? Actually, I don't really care how you call that file. What I care (and I think not just me) is uniformity, so if we decide to rename all those files that have multiple boards supported in them, I'm fine with it. So pros for my proposed approach would be: 1) Currently, there are already board files with multiple boards supported in them that follow the approach and renaming them is really unnecessary. 2) git log will not break. 3) boards that cannot be named after the convention like 81xx but can be added to the same file will not require further renaming (like 82x8 - I don't really know if that will exist, just wondering). 4) This renaming is really what Linus likes ;) cons: 1) Misleading? Currently, I don't think this renaming is good for anything, especially that majority of the board stuff should be transformed to the DT descriptors. -- Regards, Igor.