From patchwork Mon Sep 13 14:51:46 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rob Herring X-Patchwork-Id: 509772 Delivered-To: patch@linaro.org Received: by 2002:a02:c816:0:0:0:0:0 with SMTP id p22csp634998jao; Mon, 13 Sep 2021 07:55:17 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzhPyzyZaiiwj66XAERzFFcqQ8ZyVbCwJ4SJx1/X8bOqaLE7BQ4LB+HFnyszwWymodxVyq3 X-Received: by 2002:a5e:8a4a:: with SMTP id o10mr9435836iom.11.1631544917468; Mon, 13 Sep 2021 07:55:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1631544917; cv=none; d=google.com; s=arc-20160816; b=Q5OZx3lk+/hb7kjqZB3Z21jUOivKzKrseQ97fBgOKVxFmSmj2lex4iwPssDg1G4F/j OcYo7j9iOi8upBMQLADtME3ocv5NF3OYe8OPWAs2VhQ+b0JxlgecPAiW2hC2TQRRmfPA rTXqCNjA92Q7WlPLGJyXF11BmKS0f7iL5ZePFOzSms5qlSB3QyR7px/Xp7yRi1zonVCo 1MkawSnX6X0Hg7v2b3EbKQV9+z9wrfXZ9vx4qTwI6XwVf6U2hwWsFQhq4gbpl48xZzxk OpFE7vfqcF09izo9JjV7hSNtkeuR7JefuhRRTsk7u4YivYng8UsQ9aEHCfWr88jX42CA pWMA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=2k8H413Cyw4gFlMV7mUeANkoOjizWyGiq1vrf4EX2kc=; b=B4V+F1bObSX/W9h1nrEw3fxmdSFzDAN3FnKvK4UcpkDEIQZrKlTid3xy7r1elJ++Xv BWaBCOT4Zj4vbiVjqGyUHXJhp2DOxxUQKBlXLoOAlnS43wcGrN+uI1CYUiFDmoaKXOBd 8vontxlwkTCLKKVo1imyq/vxSimV+FDmkfWMpYorhx4dUFyH/bY7Td79ztGRfzTi9/H9 DHeExC+Jr2uPJp7SqPslD4nWhqw9ZxWyi06g0cLnsit0RCwvJQxjC56fAYap/rweYVkl 1xY2b/DdBrrtH6K5QlqBmSBakl7ghF1Bl2IGa4QbmMh1lWnWOGKHjOrxUxZR/d2oCYzD 4utg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of devicetree-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=devicetree-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id d11si6894675jaf.24.2021.09.13.07.55.17; Mon, 13 Sep 2021 07:55:17 -0700 (PDT) Received-SPF: pass (google.com: domain of devicetree-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of devicetree-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=devicetree-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1346916AbhIMO4a (ORCPT + 6 others); Mon, 13 Sep 2021 10:56:30 -0400 Received: from mail-ot1-f45.google.com ([209.85.210.45]:42637 "EHLO mail-ot1-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346634AbhIMOxI (ORCPT ); Mon, 13 Sep 2021 10:53:08 -0400 Received: by mail-ot1-f45.google.com with SMTP id c19-20020a9d6153000000b0051829acbfc7so13629175otk.9; Mon, 13 Sep 2021 07:51:51 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=2k8H413Cyw4gFlMV7mUeANkoOjizWyGiq1vrf4EX2kc=; b=NgTDjkauVRbbCHfj07cScPyHzBsaxb8WMKI6Zb96P9pqGPguIUxtqbO6RUcR+8muGQ gbhvcnOlAMRAmZ5A0kT5JzpXQHnnNnWibbnQyt02++Al5Wa1SQZUseuQMdxAeh+4guwh RDUAozCadtjo+sAtjyQFDqXEgOI2eMTPofu6OvfACr70K0KHYoXFI7xLG+RsYSWAzQjq k/Sg4fw14zFK9hhxkMrKICG8mS5lOxptq45uRtg8WqlY1hJoV5I8R8EsYoMi1yqzeg5s pTB5Wn3PagzNyFVR5yn8llVmnrkqkVkLHwN1pIuhJRK57Yyv4hGWViXB0PN0p1pBQ4/U LNhQ== X-Gm-Message-State: AOAM5307TgSFac2aKIBqnDtHFANJiJLt+mM5ywajdERCQa6Lct4G2FLl aepCAh4NEXgdSZsaRBEA/dmIXG0cYg== X-Received: by 2002:a9d:5f8e:: with SMTP id g14mr9983292oti.37.1631544711041; Mon, 13 Sep 2021 07:51:51 -0700 (PDT) Received: from xps15.herring.priv (66-90-148-213.dyn.grandenetworks.net. [66.90.148.213]) by smtp.googlemail.com with ESMTPSA id d10sm1897462ooj.24.2021.09.13.07.51.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Sep 2021 07:51:48 -0700 (PDT) From: Rob Herring To: devicetree@vger.kernel.org Cc: Dmitry Baryshkov , Tom Rini , Masahiro Yamada , linux-kbuild@vger.kernel.org Subject: [PATCH v2] kbuild: Enable DT schema checks for %.dtb targets Date: Mon, 13 Sep 2021 09:51:46 -0500 Message-Id: <20210913145146.766080-1-robh@kernel.org> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org It is possible to build a single dtb, but not with DT schema validation enabled. Enable the schema validation to run for %.dtb and %.dtbo targets. Anyone building a dtb for a specific platform *should* pay attention to schema warnings. This could be supported with a separate %.dt.yaml target instead. However, the .dt.yaml format is considered an intermediate format and could possibly go away at some point if schema checking is integrated into dtc. Also, the plan is to enable the schema checks by default once platforms are free of warnings, and this is a move in that direction. Cc: Dmitry Baryshkov Cc: Tom Rini Cc: Masahiro Yamada Cc: linux-kbuild@vger.kernel.org Signed-off-by: Rob Herring --- v2: - Also enable schema checks on %.dtbo targets --- Makefile | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) -- 2.30.2 Acked-by: Masahiro Yamada diff --git a/Makefile b/Makefile index 7cfe4ff36f44..c57a7657c8dd 100644 --- a/Makefile +++ b/Makefile @@ -1402,17 +1402,17 @@ endif ifneq ($(dtstree),) -%.dtb: include/config/kernel.release scripts_dtc - $(Q)$(MAKE) $(build)=$(dtstree) $(dtstree)/$@ +%.dtb: dt_binding_check include/config/kernel.release scripts_dtc + $(Q)$(MAKE) $(build)=$(dtstree) $(dtstree)/$@ $(dtstree)/$*.dt.yaml -%.dtbo: include/config/kernel.release scripts_dtc - $(Q)$(MAKE) $(build)=$(dtstree) $(dtstree)/$@ +%.dtbo: dt_binding_check include/config/kernel.release scripts_dtc + $(Q)$(MAKE) $(build)=$(dtstree) $(dtstree)/$@ $(dtstree)/$*.dt.yaml PHONY += dtbs dtbs_install dtbs_check dtbs: include/config/kernel.release scripts_dtc $(Q)$(MAKE) $(build)=$(dtstree) -ifneq ($(filter dtbs_check, $(MAKECMDGOALS)),) +ifneq ($(filter dtbs_check %.dtb %.dtbo, $(MAKECMDGOALS)),) export CHECK_DTBS=y dtbs: dt_binding_check endif