From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 093E1C433E0 for ; Thu, 11 Mar 2021 13:19:39 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 64AB064ECB for ; Thu, 11 Mar 2021 13:19:38 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 64AB064ECB Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=ti.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Tu0nuQbCqq70/fYrwMobqS8CaoV5YruRyaKhDQBaQIQ=; b=CuNetjOXo/qGtccYHnUghmvii 7iXm+HW6YuWO20WS/76NoEavh84+KkDFp1eGRJfmLuvlIgwnTJuA0jljf0TEN02M9GbrsyozkjW9u h2DFobqXjW4iptgWEqJtLCj2ZcsiBCfHnsKAT3DV606rlNQr9y1N6yxdTvMFSy1faN7GnG/mmduGx 5n5eTpq2pnJWHw0XcMSk/zshwAtGyhhGNk9odI9d51YTG3T6iNp0/UCS+Q3F8ueWfwO/K4pc/DZI/ JQq+gIgvuwHOCh6CbXbevsNyyghujfrBvpuO3wUe+xN+v0DC7kGioUTVCKI/IYjwk8DllwglXWmX5 CJPfbU0mA==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lKLCW-009CHL-NV; Thu, 11 Mar 2021 13:18:04 +0000 Received: from fllv0016.ext.ti.com ([198.47.19.142]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lKLCR-009CGc-Rl for linux-arm-kernel@lists.infradead.org; Thu, 11 Mar 2021 13:18:02 +0000 Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 12BDHs9b044916; Thu, 11 Mar 2021 07:17:54 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1615468674; bh=oebGbjZf1CUZSOme8bgvj9xynU4AedYVMPSzbHOpfgk=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=dfUepoWa1DnnFpu7YlsrBHQ3qoa401BIeSUHnvJ5vDIfBHYktrLBEN6Upm3uA/Hpp RqOBf0z4UQz/5M8lawJMmlNUULwnTRqtpopVrxQZChx54n13mUm34KRPcGcUdVly9/ NX2vjcG/MN1Wyb8KR7+svkwx8aNN0F59ckSdkAMU= Received: from DFLE113.ent.ti.com (dfle113.ent.ti.com [10.64.6.34]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 12BDHsRA044775 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Thu, 11 Mar 2021 07:17:54 -0600 Received: from DFLE110.ent.ti.com (10.64.6.31) by DFLE113.ent.ti.com (10.64.6.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2; Thu, 11 Mar 2021 07:17:54 -0600 Received: from fllv0040.itg.ti.com (10.64.41.20) by DFLE110.ent.ti.com (10.64.6.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2 via Frontend Transport; Thu, 11 Mar 2021 07:17:54 -0600 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 12BDHsPQ124750; Thu, 11 Mar 2021 07:17:54 -0600 Date: Thu, 11 Mar 2021 07:17:54 -0600 From: Nishanth Menon To: Jan Kiszka CC: Tero Kristo , Rob Herring , , , , Le Jin , Bao Cheng Su , Vignesh Raghavendra Subject: Re: [PATCH v4 3/3] arm64: dts: ti: Add support for Siemens IOT2050 boards Message-ID: <20210311131754.i5ewls6hgeitcgre@astonish> References: <9bff40f434e5298890e5d139cc36cc46a0ca2d76.1615369068.git.jan.kiszka@siemens.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <9bff40f434e5298890e5d139cc36cc46a0ca2d76.1615369068.git.jan.kiszka@siemens.com> User-Agent: NeoMutt/20171215 X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210311_131800_210985_AA51B10C X-CRM114-Status: GOOD ( 12.90 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 10:37-20210310, Jan Kiszka wrote: > From: Jan Kiszka > + spidev@0 { > + compatible = "rohm,dh2228fv"; > + spi-max-frequency = <20000000>; > + reg = <0>; Jan, As part of my final sanity checks, I noticed that we missed this: is a checkpatch warning WARNING: DT compatible string "rohm,dh2228fv" appears un-documented -- check ./Documentation/devicetree/bindings/ #629: FILE: arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi:581: compatible = "rohm,dh2228fv"; I cannot pick up nodes that are'nt documented as yaml in Documentation/devicetree I know this is irritating to find such nodes that already have previous users and the person coming last gets to deal with "new rules".. but sorry for catching this so late. Here are the options that come to mind: option 1) - drop the node and resubmit. option 2) - get the documentation into linux master tree and then submit the patches. I think we should just drop the node and resubmit - since this is a more intrusive change and I don't have your platform handy, I am going to suggest you make a call :( Additionally please install yamlint and dtbs_schema -> run dtbs_check. I see more than a few warnings there which may need some closer look. A full log against linux-next is here: https://pastebin.ubuntu.com/p/qR69h28c5f/ PS: https://github.com/nmenon/kernel_patch_verify/blob/master/kpv I have been using my script to verify with kpv -C -V -n num_patches and then digging through the logs. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D)/Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel