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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3514FC001B0 for ; Mon, 24 Jul 2023 07:16:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229770AbjGXHQR (ORCPT ); Mon, 24 Jul 2023 03:16:17 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54176 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229492AbjGXHQP (ORCPT ); Mon, 24 Jul 2023 03:16:15 -0400 Received: from mail-lf1-x12d.google.com (mail-lf1-x12d.google.com [IPv6:2a00:1450:4864:20::12d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3A3BC94 for ; Mon, 24 Jul 2023 00:16:14 -0700 (PDT) Received: by mail-lf1-x12d.google.com with SMTP id 2adb3069b0e04-4f9fdb0ef35so5957663e87.0 for ; Mon, 24 Jul 2023 00:16:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1690182972; x=1690787772; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=LJjiLpTiLcLmlDY6EXYdVhyrOy6hWSuU/gTL98LQWAs=; b=Gd24sNIj0oDtV+j8vvlaIzwzpX5g16e5B0su1m38z5Eh4/ymC4AQyHc4iOL5GSnsKg qqh/KFLB2/ApPKkloUyObuj40XmZRV7Bho4iuxDzwtmNeVG3NtBWe7vYy3Y+VWEcZwLL kb7deTAzE3D6/yorBkWrFraCGa0GTCTaXK11D1S2Y7uwJKARNr8LfXmUpUuIcwFYSikR dq2WhX0ZRnoGaV2qXPozC5zF2vF2fBRPS8r4wgOgTrhTZXZeYRpyQoE8kSRut4Y1DZM6 JPFRwhRmXz3Ih+rUTUNRvXegagbcIBRLq+09CAKjRFeGIa6wiBiv/ErzETd/AHJpSoBc drig== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690182972; x=1690787772; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=LJjiLpTiLcLmlDY6EXYdVhyrOy6hWSuU/gTL98LQWAs=; b=XZnOpdDdZJ0x/isn7mw1S/vDEhSmCoFQ7j6gtHc2UL4g0ZzD9ohyVQ65FglWWOytv4 p3mWNht1oco/EuG6VsMqViU8cQKYwKveTByvlIUN0/CdfKh9e2ga18RIDeGF0e/Lkdll mue78D34KUOUa0aJowOnh1hkOqbrW5z6DOAyQrbLreRTiuzBa7djm2MPOWa15IciIRYg 8MD59mr4rdVq/bdaWfQWloGF6TRLWaPvWgx25vsxU+/xJ68A9zzxZcEKTUjODybmRpVA FXtqkXg+a3k2nWFIB5nA4a/nZCMZW8kvXhmmdONu+EehEzfr47e1FVradTZBt85qEt2t AeIQ== X-Gm-Message-State: ABy/qLYvfk2gwii/v9lDqYRmbACQPhGComFET+ZTlICR2O+DJCyj64ph wbGDqwh7EEsGIGUjl3Sc5lcR6FsrASj41uuVbp737A== X-Google-Smtp-Source: APBJJlHFEusndSawbs0I417rvOGJ25UK9yb415P0WNw1Hwu1JevZtKCiq9KD/FsQvDX2NH7fwJ7rgA== X-Received: by 2002:ac2:4d96:0:b0:4fd:d172:fc2c with SMTP id g22-20020ac24d96000000b004fdd172fc2cmr4036921lfe.21.1690182972321; Mon, 24 Jul 2023 00:16:12 -0700 (PDT) Received: from [192.168.1.20] ([178.197.223.104]) by smtp.gmail.com with ESMTPSA id b1-20020a05640202c100b0051e06693590sm5708715edx.91.2023.07.24.00.16.09 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 24 Jul 2023 00:16:11 -0700 (PDT) Message-ID: Date: Mon, 24 Jul 2023 09:16:08 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Subject: Re: [EXT] Re: [PATCH v4 1/7] dt-bindings: arm: fsl: add se-fw binding doc Content-Language: en-US To: Pankaj Gupta , "shawnguo@kernel.org" , "s.hauer@pengutronix.de" , "kernel@pengutronix.de" , "clin@suse.com" , "conor+dt@kernel.org" , "pierre.gondois@arm.com" , Jacky Bai , Clark Wang , Wei Fang , Peng Fan , Bough Chen , "festevam@gmail.com" , dl-linux-imx , "davem@davemloft.net" , "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Gaurav Jain , "alexander.stein@ew.tq-group.com" , Sahil Malhotra , Aisheng Dong , Varun Sethi References: <20230712121219.2654234-1-pankaj.gupta@nxp.com> <20230712121219.2654234-2-pankaj.gupta@nxp.com> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 24/07/2023 08:37, Pankaj Gupta wrote: >>> +examples: >>> + - | >>> + ele_mu: ele_mu { >> >> No underscores in node names, generic node names, e.g. firmware. Look at >> existing code. > > Changed from: > - ele_mu to ele-mu. > - "ele_mu {" to "se-fw {" Still not generic. Why do you change it twice? You understand I talk here about node name? > > Name of yaml file, is se-fw.yaml. > >> >>> + compatible = "fsl,imx93-ele"; >>> + mbox-names = "tx", "rx"; >>> + mboxes = <&s4muap 2 0 >>> + &s4muap 3 0>; >> >> Two items, not one. > > Corrected it to "mboxes= = <&s4muap 2 0 &s4muap 3 0>;" > >> >>> + fsl,mu-did = <1>; >>> + fsl,mu-id = <1>; >>> + }; >> >> Plus you clearly did not test the binding and DTS. You said you did some >> internal review, so I assume this also includes some testing. How did you test >> your DTS? >> > > Each version is tested before sent for review here. > I have tested the DTS file by compiling it and loading the DTB to the board. > Executed test on the board. That's not enough and your colleagues should tell you that... Read our guides for bindings. It does not look like you tested the DTS against bindings. Please run `make dtbs_check` (see Documentation/devicetree/bindings/writing-schema.rst or https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/ for instructions). It does not look like you tested the bindings, at least after quick look. Please run `make dt_binding_check` (see Documentation/devicetree/bindings/writing-schema.rst for instructions). Maybe you need to update your dtschema and yamllint. Best regards, Krzysztof