From patchwork Thu Aug 17 00:35:02 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Rini X-Patchwork-Id: 9904789 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 8806560386 for ; Thu, 17 Aug 2017 00:35:43 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 79E911FF2D for ; Thu, 17 Aug 2017 00:35:43 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 6EB6828A30; Thu, 17 Aug 2017 00:35:43 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.1 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, RCVD_IN_DNSWL_LOW, RCVD_IN_SORBS_SPAM autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id D19421FF2D for ; Thu, 17 Aug 2017 00:35:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date: Subject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To: References:List-Owner; bh=8tZzxlIORxUT9aJuaiNPYpuDKy5KtRsOtZX35nbkCf4=; b=sS3 3CsdP3qzFSCD+WTBW0DlPsIkiIpHt/f0bmqEL+A1mlmQd2zgLjDIErwwSCwOVJ8EzYNbNq4uiws3X 6WQ7TH440Dn7xqJfTMj/+hKdd0Lwv/Ar2o2Asd2qpoC/QtyxfycRnOsbBLBxzO5/kF5futza1Pt4G Gdv6bWqPdRgPgdbnvtWgyb/LejLLUhGBjv92Hr9hlfE3uBXY8dDOT4aXn866yzajERSb19bmjcdS3 qCltF1T5FkbVUosMYeRoYt1yCfIiQaZihOH8NMBXy+RNfqc0ExRfoSIVgw8Bt3RgM0LV8ECxDr9nf JXOCimZM72k0R1YyztkNdmIBIi4M15Q==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1di8mb-0003Nr-6Y; Thu, 17 Aug 2017 00:35:33 +0000 Received: from mail-yw0-f193.google.com ([209.85.161.193]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1di8mV-0002Ed-Qy for linux-arm-kernel@lists.infradead.org; Thu, 17 Aug 2017 00:35:30 +0000 Received: by mail-yw0-f193.google.com with SMTP id s143so3261550ywg.4 for ; Wed, 16 Aug 2017 17:35:06 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=P3zlcBetkE7+IIUuwCNAwDnJ7JSyVrfYjETUzoBMdqw=; b=EvPFTICA68pomyRbfapigtQ+ZhBd8HnUfnBhPxr5TTzjL7+/bugnEjVN8oUwRuo25y VHbbo8UyH6Gtt/2ATwzNHpYQLP4uYn/KShV+6EobzlGSp8xpQrANWFuu+kG5XFVj/XlG tbZnyisqzcBuzNXxwu8oY3dFCAo1zx5NHtSyp5IY97szywQcnHIXFA3i1Ahu7O0Zr9Og Ki3MsRK4rvNsAJRJP1trxasJ9A9Y2umnxm+7o6OoucGgDxf/q79s5dQ18l3300le4OeA wamchInp+W1/NxMt08xsRxRLBGRyd9j/Rghg6Vq+8qlohXPTaqGyaggBPvTbve5Cx7Pq 5lWg== X-Gm-Message-State: AHYfb5jHA1xjz3y9ifiQru7mQ/Y7SBCiumQ63gU0ruYMZMI0rrpkCdQ+ gl/3hbkbjw1qIw== X-Received: by 10.37.162.4 with SMTP id b4mr3070399ybi.178.1502930105509; Wed, 16 Aug 2017 17:35:05 -0700 (PDT) Received: from localhost.localdomain (cpe-174-106-0-58.ec.res.rr.com. [174.106.0.58]) by smtp.gmail.com with ESMTPSA id s126sm758725ywe.49.2017.08.16.17.35.04 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 16 Aug 2017 17:35:04 -0700 (PDT) From: Tom Rini To: devicetree@vger.kernel.org Subject: [PATCH] dts: Make it easier to enable __symbols__ generation in dtb files Date: Wed, 16 Aug 2017 20:35:02 -0400 Message-Id: <1502930102-8648-1-git-send-email-trini@konsulko.com> X-Mailer: git-send-email 1.9.1 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170816_173528_058590_BCAEF018 X-CRM114-Status: GOOD ( 15.67 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Nishanth Menon , Mark Rutland , Masahiro Yamada , Drew Fustini , Frank Rowand , linux-kbuild@vger.kernel.org, Sekhar Nori , Russell King , Rob Herring , linux-kernel@vger.kernel.org, Tero Kristo , Lokesh Vutla , Tomi Valkeinen , Michal Marek , Jason Kridner , Robert Nelson , linux-arm-kernel@lists.infradead.org MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP This introduces the variabe DTC_EXTRA_FLAGS to allow for additional flags to be passed to dtc. While this can have many uses (such as easier testing of new warning flags) the use case I wish to spell out here is passing of -@ to enable __symbols__ to be included in the resulting dtb and thus 'stacked' overlays to be able to be used. When passing in -@ this will increase the size of the dtb (and resident memory usage) in exchange for allowing more types of overlays to be applied to the dtb file prior to passing it to Linux and additional functionality within the kernel when OF_OVERLAY is enabled and in tools outside of the kernel. Cc: Jason Kridner Cc: Drew Fustini Cc: Robert Nelson Cc: Lokesh Vutla Cc: Frank Rowand Cc: Rob Herring Cc: Mark Rutland Cc: Russell King Cc: Masahiro Yamada Cc: Michal Marek CC: linux-kbuild@vger.kernel.org Cc: devicetree@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Tom Rini --- This would be v3 of my attempt to enable __symbols__ more widely (v2, for people not on the CC, enabled it always, but for some boards). To be clear, this version of the patch is more flexible in that it allows for arbitrary flags to be passed to dtc, which can have other uses. But, I'm still spelling out -@ being the motivation behind this for clarity. Both Frank and Rob were uncomfortable with v2, and Frank suggested this approach to make it easier for users / distributions / etc to opt-in to this functionality. For clarity, Frank was suggesting (at least at first) perhaps also a tie-in CONFIG option, which I have not done here as I feel that's adding extra hurdles to something that must already be opt-in. Given that today many of the immediate uses are for "pre-boot" application of overlays (SoM + carrier + add-ons or EVM + LCD for example), I believe this is an acceptable trade-off to make currently. --- scripts/Makefile.lib | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib index 58c05e5d9870..04434ffe3370 100644 --- a/scripts/Makefile.lib +++ b/scripts/Makefile.lib @@ -293,7 +293,7 @@ DTC_FLAGS += -Wnode_name_chars_strict \ -Wproperty_name_chars_strict endif -DTC_FLAGS += $(DTC_FLAGS_$(basetarget)) +DTC_FLAGS += $(DTC_FLAGS_$(basetarget)) $(DTC_EXTRA_FLAGS) # Generate an assembly file to wrap the output of the device tree compiler quiet_cmd_dt_S_dtb= DTB $@