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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2BEEEC433F5 for ; Wed, 13 Apr 2022 09:46:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To: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=ZrIZiLa4iz1SIx5rGLtv1txKNV5bVwd3R0FoHTEuiBc=; b=kDPghHEv0xAzyc +Pp77qI4mOEH55aGeX6iNtx9JOuvtlhVAOAdsV83q0RXrJqS0Wod/2p6Hh8+Zp7YQzpQurIHkInu3 fTHVA4vLrIcF3EIVj59owVdbsowctqMGWH+zn0xNE7gf53Rdv6wRCwEsiKKhPL6kltIlLIZRp6+bj wE5T4Dt4BPfKcgL2zlSmiA5tesqpcCIOOJPv7o8G4OkD3MFghYzCnYUOHE0tjjWWqhllbFmJk2/AY Yqwf/xyJe34US7bdZ48e/fLbXmVRnwoM9lPBmilJ5GNvlAXf2cYfAwKusTtBEo88orW8Hrlyr1s7O 9CrhSlw7BFOlJi1JTOqQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1neZYa-000PlG-33; Wed, 13 Apr 2022 09:45:00 +0000 Received: from mail-gv0che01on2072c.outbound.protection.outlook.com ([2a01:111:f403:4012::72c] helo=CHE01-GV0-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1neZYV-000Pgq-JS for linux-arm-kernel@lists.infradead.org; Wed, 13 Apr 2022 09:44:58 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WKEi2jGGoPhjIfp54tpw+Rs902hqqvk6RadP/J1AB0IkvLImRppuCg8Em3aTF0Bk0n6XPc/ysWGuHrA1Xj4/Uf6tbaVgLz7EewzBKVceL6coT8O8fAIWjBRaUeVH3SFXH2r5qbzjk+/V7gRkyvbUrCZXNVHytpjVEUuP7Bjy4tSW8Qqme8/eiyjvS8S2wedUjGzXzmeLSEonDEVtPQYkk/qJpv3cuy8mU3nvOgljNBzwrrJRg97rg4Qf9GgPw9b//v/VfpPKvTVsffyTxPvz65SsvEhfrR3YmNLghiTBWtULCKIepC5NzFDrBvWamNwSp1XJE2LD9A9+MKvxsrOYcQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=8kza2s4IMdOjK1ExzbGi/SpZ02UcjQwlbw/HJi8/QkU=; b=Zrkqi56gb4TB1sXKU8P4KhoREAa+ENr7zMg4FEu/9mrGayOaw+mPnlzfxhJ3aEL99Ete93CKEr9POhKdtbzeVyqRlzyWQAu009FCqto+vSlVwmuE7S05NvhE9hrPRHFoP4hCoW1TZTjJgBjaJfkCYRx4Rtb2b1E34LhAKYh66sWwaIW2I6bXd9OqWSmnbLmkOfSF3H7cmw0UDK0RNI83JSb6PKBJ+25nDOM4gyscpfpvvVPRaxdlbzjbhuP1Rrc9koCLn34aAWzCE+eCqtjWIzXwnBwNBenjwA2rK6AgqorGwyIbaRWpYm4jgY4PYJD6CRYcJZNSFUy3eZPVCFQ41g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=toradex.com; dmarc=pass action=none header.from=toradex.com; dkim=pass header.d=toradex.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=toradex.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=8kza2s4IMdOjK1ExzbGi/SpZ02UcjQwlbw/HJi8/QkU=; b=nS40uOA3d03ZXisC5fWvD49wflqQJtYAZDQKfrrzXv8E/fYNbvMnhYmNjl9qbOtjRbI63bCshQ4JrXlBXkQoH+YCW2n2Xkr4fWzPXDiOankskxhul67SKA0opm/xSsUFrp3vPUsY9m8Q0vchYBtOnkC2hRXxH0tJEwmSb9Liyug= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=toradex.com; Received: from ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:2e::8) by GV0P278MB0018.CHEP278.PROD.OUTLOOK.COM (2603:10a6:710:1d::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5144.30; Wed, 13 Apr 2022 09:44:49 +0000 Received: from ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM ([fe80::f465:3051:c795:3c2]) by ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM ([fe80::f465:3051:c795:3c2%7]) with mapi id 15.20.5144.030; Wed, 13 Apr 2022 09:44:49 +0000 Date: Wed, 13 Apr 2022 11:44:49 +0200 From: Francesco Dolcini To: Marek Vasut Cc: Francesco Dolcini , linux-arm-kernel@lists.infradead.org, Fabio Estevam , Marcel Ziswiler , Peng Fan , Shawn Guo , NXP Linux Team Subject: Re: [PATCH 2/2] arm64: dts: imx8mm: Add i.MX8M Mini Toradex Verdin based Menlo board Message-ID: <20220413094449.GB118560@francesco-nb.int.toradex.com> References: <20220407202456.244917-1-marex@denx.de> <20220407202456.244917-2-marex@denx.de> <20220408064657.GA80730@francesco-nb.int.toradex.com> <55a1e747-9b6e-98da-969c-7592b29a293b@denx.de> <20220410084636.GB115930@francesco-nb.int.toradex.com> <753d9e1b-cbc9-bdb8-036a-89293583271b@denx.de> Content-Disposition: inline In-Reply-To: <753d9e1b-cbc9-bdb8-036a-89293583271b@denx.de> X-ClientProxiedBy: ZR0P278CA0142.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:40::21) To ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:2e::8) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 588dbb79-c469-455d-80ab-08da1d3240aa X-MS-TrafficTypeDiagnostic: GV0P278MB0018:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: mLSAk2/Na4XA99Mm/DC5gk8n2EMIwDuPOz1H3rF0Krrm5fCkj08/n5QGUQWhElN9B3GJMK93Rw9X+XjZ6+CEJOptQ8+qetdHQzQlGAHUdIYMKqeZHOxA3gfK9czYkiy1sZ7WJWxEekwQmhFEwtI77dZOhqa6hscMlHFaiIfmc4VDOxm4D0Pe+7/s/nRJN01nseLches6KEx5uaSLcNfFINItww7KROj5bFBVL16zKZjA7ek2q1CiyW51Iyra/NxFQKUN1mV5Z3eubaPkB6b5c9iVlhSKA0JOcUpP9RGZz15VHwUkoAkEL/83wSSHFe0fjaqtb0DNsbgW2Yep/mnoKzzjEiur+fYpMvy2HhEGyHjX9lY/hRyW5GPV8m1t9Ebk/MbK2XKzzdU1c3LLsEX8vv9ZcjgoUd3+MkDn2mdudS0crYVwxVl+b11exPHn56GRcjdxo6XjCX7h5UDuARlUDcEffihbk11V0gi16xaddCmvTcJFeX2u6yifqh/GZQjD6PRIXfnlJJ6lP4gmEeespRDuM12jt0CjmgZT2vzFO6hgsgGH0BKEwTTvqFtXFxweg3aI7sNA/PvZ8xMxStiBXFdG2UWr1g46jViuK0las7ZzchzXFXZDuJWp8rZOsdtXEE2r3saKgJx6Y1Sqe3wC7ZbstZP0Us/o3xQJAn95zjluD8CapFpkufkhB4GQGnooFc9IhQjEn+Cs0P85YhwPXnLCXhRFv1/pF852PrU1qh1TrE5I5P9FT7nNTQYCT6m0QhDQr+d3gerSSFrDGrnEFw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(13230001)(4636009)(366004)(346002)(1076003)(26005)(186003)(6512007)(6506007)(2906002)(5660300002)(508600001)(83380400001)(6486002)(8936002)(86362001)(33656002)(66476007)(66946007)(66556008)(8676002)(4326008)(38100700002)(38350700002)(52116002)(53546011)(44832011)(54906003)(6916009)(32563001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?sU8FbmHU59PrO2dyF2aj1qqqcppqROsaKV9tKfecnSnGHJ4Ws4iGgBttokvY?= =?us-ascii?Q?XM9ZcQhwa5gMTzU9nd7g8KLgBufQg+omzUdLXMIZ8zApTA3riWbPQB8Rnf5a?= =?us-ascii?Q?OxddHFVhNM6rgsp3NT+gAwUbVofHPoBNt1QZmQ5XuXKhImLW3quY9N+Akyt+?= =?us-ascii?Q?6Jt2vQD1OJ6FJ4qXMSnkJUR+Oxj2WOW9fPB9wR/unZiA0yGraRLifuirku97?= =?us-ascii?Q?Q20h08TLzUyHCU+TTTJMaSZ7Xo1ovr6ulRSJhUppbLUswj7wGVwXSiqUOcMj?= =?us-ascii?Q?wzNj6OhialCgceJ/sX2R2eynrWfiMZh4xdwxDMvtQvnqqe0CoFNVLLq9p1H1?= =?us-ascii?Q?Ie7spf5YVYVr0zoG3T41/9hWQSmBg16DsAaJnU2qw5GN5KMdPYtMkHbvBe3E?= =?us-ascii?Q?IvBsxr45Q9nfEByMGhaWel79pszK51Kl6NY4QrZuUnWgtC9KOj8GQMUzcuis?= =?us-ascii?Q?/1cMVlF4fb20lFsaIUqNbhuRY8v7LlC/NaOBWbvPqPCkJ/DGnRyeSgZxEkvA?= =?us-ascii?Q?phAK+nMG1YasVwVxIiisz14c8nzF+Pdkz0LDaobdwgT/FvP+CUKuQXtSZXP6?= =?us-ascii?Q?1RKr9Tgv0DLTMyHtiADyrsTfuRNLUELceHgiTeWY1NcII1NMiWdwX7UhlOUg?= =?us-ascii?Q?GbDOsOOFvpvWc60zJSdKWYeElk4OK7ML8ltSmuvDfkT6Qm59vs3ofmCg0lj9?= =?us-ascii?Q?ZNNg3WKrQEhA0IPHhnYsari6ql4eQnnGgoMYr2E6E4CQykYnfjbUCqgE2MpB?= =?us-ascii?Q?hKWBAILDEt0bbBQ9hGOSaJ/cDEVbncdNt9nCAgSG9fnNqUiF0EzeL79K+V3J?= =?us-ascii?Q?gUQSsUyFv2lofDwsaGCvSbt2vrmaqw7aJGbmVPMR+sluUR6+kcSR76NlfRlY?= =?us-ascii?Q?gWQOHJnVjHdrzOn2fepKRxT3GT4G113mmZfUh3+R4320nZb7YTC6UgWriOoy?= =?us-ascii?Q?JMGoq/wWGWxM4QIolEiaIdrA6h2ZKUOtlqvb3mqylZ2SBs95FerYwAVa60SY?= =?us-ascii?Q?KpPuVXKHezDTI7CVdi1YvP+AZVu8mYaPeQZSwfdMClrz61DFss8TjDHt7D1q?= =?us-ascii?Q?elx9Wlbjhj6//WqPvSUYbUq23TT4nGL56ULTbWWxaQqq0LiDtXXlG3Tw7ZY3?= =?us-ascii?Q?XTZj+PS2MxTAtsd07AKzIne/XZLeFI4uPX2OZ5UDO+jtu3Axc2KnU//K3+BZ?= =?us-ascii?Q?rYuoZYwlem6AtwG2HcUXtI5BFxtbCzyCxZy2LrQ/vUhOD8vINrfijP0stccB?= =?us-ascii?Q?g+y/3SYHNkp2+TNeaF/dMC1sYTOIy5oC9p0b3PPQ0Ltj2QvzYGrHYhx6m0Hm?= =?us-ascii?Q?g/JybDIrd1eXfmdkA7y0kPypEnWcvrWW8+6tD0pOG+zyJTdE9205lRK4lYSo?= =?us-ascii?Q?kX70B+SxrsZXfpC11tru8AUyb3ntWFn3uSWQ9wJrQOMFyRrjyhQu9NKvjBlv?= =?us-ascii?Q?YiAAc8qs3jbcJ3ovncvYclYyW49VsBLjXTKcoJEJZhWF8ZqiLq0ZwX4UAhSq?= =?us-ascii?Q?tf0JQoAdOgRQtk30om553f4PnxYcqzeoUIX7FcyqoC0t8dwBAMlKIcXtD7en?= =?us-ascii?Q?B0Ltiu0MPUFGbSFBbUtcGJFoOfTnB2dyxBylaZBcWO4lK3P7qUnhensv2k0i?= =?us-ascii?Q?0Xd0cnNlZEYmpuLFv4GzHcuY6wVRIlZk7WSyablgm9WaLth55Ibqu3PqhjF2?= =?us-ascii?Q?DtuXAhhHoj/fUsCKcwjcim+PoOxoUIKBidrq3sKoYBYwhn6k7OXv/wTn14pP?= =?us-ascii?Q?XYjp6cx/3xmJo4S83+0ujurBmHcRCnY=3D?= X-OriginatorOrg: toradex.com X-MS-Exchange-CrossTenant-Network-Message-Id: 588dbb79-c469-455d-80ab-08da1d3240aa X-MS-Exchange-CrossTenant-AuthSource: ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Apr 2022 09:44:49.7926 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: d9995866-0d9b-4251-8315-093f062abab4 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: JOmBR7w62v877pqlQq2TexILfhuRVQQlArZtwy4EZBCtsSVPoKWcthgp9JHJC1GXknXc3h8Rgdu8XLRjWExrgoZi2eRWa1PBb1/ESnb3OoY= X-MS-Exchange-Transport-CrossTenantHeadersStamped: GV0P278MB0018 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220413_024455_877000_5696A111 X-CRM114-Status: GOOD ( 27.76 ) 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 Hello Marek On Sun, Apr 10, 2022 at 11:37:52PM +0200, Marek Vasut wrote: > On 4/10/22 10:46, Francesco Dolcini wrote: > > On Fri, Apr 08, 2022 at 05:02:15PM +0200, Marek Vasut wrote: > > > On 4/8/22 08:46, Francesco Dolcini wrote: > > > > > + /delete-node/ gpio-keys; > > > > would it be better if we had a label in the imx8mm-verdin.dtsi and we > > > > could just set status=disabled here? > > > > > > It would be better if there was Verdin SoM dtsi and Verdin carrier board dts > > > which includes the SoM dtsi. Right now, it seems these two things are > > > conflated a bit. > > > > > > There are no GPIO buttons on the Verdin SoM, there are some on the Dahlia > > > carrier board I think. > > > > The GPIO keys, for example the suspend button, are part of Verdin family > > SODIMM connector pinout/definition (see related datasheets). In the SoM > > dtsi we implement our standard family definition. > > > > Of course, you are free to redefine this in any way you prefer. In > > general the way we envision this is to just enable/disable in the > > carrier board or overlay dts, this is the reason I proposed to add > > a label there. I do not see any real value on deleting the node at all, > > just some potential for additional maintenance burden. > > There are two reasons for not adding DT nodes for hardware which is not > populated: > - You are polluting the DT with unused nodes representing hardware which > can never be present on the system, that makes the DT bigger and more > complex, for no reason. > - Once DTOs enter the picture, these so far only useless nodes and > properties actively become a problem. You cannot delete either node or > property from a base DT using a DTO, because neither /delete-node/ nor > /delete-property/ can be encoded into the DTO blob . Ok, I understand your arguments and I agree they are fully valid. We (Toradex) had some discussion about in the past and we still see benefit on the way we are doing it never the less. - The SoM dtsi representing not only the functionality implemented into the SoM, but the whole connector pinout to the carrier makes very easy to just include a different som.dtsi in the carrier board dts and just switch SoM, for example from a colibri-imx6 to a colibri-imx7. - We avoid code duplication - Even if the DTO cannot `delete`, it can disable a node. We do our best to have label and keep nodes disabled when functionality is not self-contained in the SoM. This is working for us pretty well so far and the majority of the users of ours modules rely on that, changing it now would just be too disruptive. I would propose that you go with the delete-node as you are doing and we keep the verdin.dtsi as it is. Thanks once more for the very good discussion, I hope that my proposal works fine for you. Francesco _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel