From patchwork Fri Aug 31 16:47:12 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vaibhav Hiremath X-Patchwork-Id: 1393961 Return-Path: X-Original-To: patchwork-linux-omap@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id EF8483FF1B for ; Fri, 31 Aug 2012 16:47:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754555Ab2HaQre (ORCPT ); Fri, 31 Aug 2012 12:47:34 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:42595 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754336Ab2HaQrX convert rfc822-to-8bit (ORCPT ); Fri, 31 Aug 2012 12:47:23 -0400 Received: from dbdp20.itg.ti.com ([172.24.170.38]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id q7VGlFBp027111; Fri, 31 Aug 2012 11:47:16 -0500 Received: from DBDE71.ent.ti.com (localhost [127.0.0.1]) by dbdp20.itg.ti.com (8.13.8/8.13.8) with ESMTP id q7VGlDW3013023; Fri, 31 Aug 2012 22:17:13 +0530 (IST) Received: from DBDE01.ent.ti.com ([fe80::d5df:c4b5:9919:4e10]) by DBDE71.ent.ti.com ([fe80::692c:15fd:9507:b54%21]) with mapi id 14.01.0323.003; Fri, 31 Aug 2012 22:17:13 +0530 From: "Hiremath, Vaibhav" To: "Hiremath, Vaibhav" , Tony Lindgren CC: Russell King , Paul Walmsley , "Cousson, Benoit" , "linux-arm-kernel@lists.infradead.org" , "linux-omap@vger.kernel.org" Subject: RE: Without MACH_ option Early printk (DEBUG_LL) Thread-Topic: Without MACH_ option Early printk (DEBUG_LL) Thread-Index: Ac2HiJVWVgGww90QTvCNDMQfv0yxcP//s90A//+gFLCAAGU3AP//oWFQ//87izA= Date: Fri, 31 Aug 2012 16:47:12 +0000 Message-ID: <79CD15C6BA57404B839C016229A409A83EA9EBD1@DBDE01.ent.ti.com> References: <5040D03B.2050304@ti.com> <20120831155226.GL1303@atomide.com> <79CD15C6BA57404B839C016229A409A83EA9EB0F@DBDE01.ent.ti.com> <20120831161122.GO1303@atomide.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.24.133.105] MIME-Version: 1.0 Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org On Fri, Aug 31, 2012 at 22:07:37, Hiremath, Vaibhav wrote: > On Fri, Aug 31, 2012 at 21:41:22, Tony Lindgren wrote: > > * Hiremath, Vaibhav [120831 09:06]: > > > On Fri, Aug 31, 2012 at 21:22:26, Tony Lindgren wrote: > > > > * Vaibhav Hiremath [120831 07:55]: > > > > > Hi Russell & Tony, > > > > > > > > > > AM335X EVM (based on AM33XX device) only supports DT boot mode and > > > > > doesn't have CONFIG_MACH_AM335XEVM option defined. Some time back during > > > > > baseport submission we had aligned that, we won't create separate EVM > > > > > options, killing the board file all-together. > > > > > > > > > > Having said that, the early printk option (DEBUG_LL) is broken, the > > > > > auto-generated file "./include/generated/mach-types.h" still refers to > > > > > CONFIG_MACH_AM335XEVM option, > > > > > > > > The way we're heading is that the DEBUG_LL options will only work for > > > > one hardcoded machine where you need to select the uart type and address > > > > in Kconfig. Or just patch it in. > > > > > > > > > #ifdef CONFIG_MACH_AM335XEVM > > > > > # ifdef machine_arch_type > > > > > # undef machine_arch_type > > > > > # define machine_arch_type __machine_arch_type > > > > > # else > > > > > # define machine_arch_type MACH_TYPE_AM335XEVM > > > > > # endif > > > > > # define machine_is_am335xevm() (machine_arch_type == MACH_TYPE_AM335XEVM) > > > > > #else > > > > > # define machine_is_am335xevm() (0) > > > > > #endif > > > > > > > > > > > > > > > So I am thinking of changing the config_xxx option to SOC_AM33XX or > > > > > ARCH_OMAP2PLUS, something like below, > > > > > > > > > > am335xevm SOC_AM33XX AM335XEVM 3589 > > > > > > > > > > OR > > > > > > > > > > am335xevm ARCH_OMAP2PLUS AM335XEVM 3589 > > > > > > > > > > > > > > > Can you comment on this? Based on that I will submit the patch. > > > > > > > > I think that would at minimum break things for autogenerated > > > > mach-types.h where if only some other non-am335xevm machine is > > > > selected (like omap-generic) things don't get optimized out any > > > > longer as they currently do. > > > > > > > > > > Agreed. In that case the first option should work here, right? > > > > It gets messy if we start mixing mach and soc defines there.. > > > > How about just add a hidden Kconfig option to mach-omap2/Kconfig > > that always selects MACH_TYPE_AM335XEVM if SOC_AM33XX is set? > > Great, this is what I had in my mind but since it is hidden option I thought > may not be right thing to do. > I was just thinking in the direction that, it should be logical and fine if > SOC_AM33XX is used for all AM33xx based machines, isn't it? > > Anyway, I think we are on same page here, I will add it and submit the patch > ASAP. > > > Or does that require that MACHINE_START is there as well? > > > > I do not think so, they are not related to each other, this option is > required only during decompression. > I have tested it on BeagleBone and it is working. > Can you please review below patch? If you think its ok, I will send the patch - Thanks, Vaibhav --- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig index fccbdf0..eccafb4 100644 --- a/arch/arm/mach-omap2/Kconfig +++ b/arch/arm/mach-omap2/Kconfig @@ -104,6 +104,9 @@ config SOC_AM33XX select CPU_V7 select ARM_CPU_SUSPEND if PM select MULTI_IRQ_HANDLER + select MACH_AM335XEVM + select MACH_AM335XIAEVM + select MACH_TAM335X config OMAP_PACKAGE_ZAF bool @@ -140,6 +143,15 @@ config MACH_OMAP_GENERIC Support for generic TI OMAP2+ boards using Flattened Device Tree. More information at Documentation/devicetree +config MACH_AM335XEVM + bool + +config MACH_AM335XIAEVM + bool + +config MACH_TAM335X + bool + config MACH_OMAP2_TUSB6010 bool depends on ARCH_OMAP2 && SOC_OMAP2420