[{"id":1769405,"web_url":"http://patchwork.ozlabs.org/comment/1769405/","msgid":"<CAPnjgZ1Mm5tT+f3iNcaAgaoW7ajss0ruY7EhEaSCd8mRGTOZoQ@mail.gmail.com>","list_archive_url":null,"date":"2017-09-15T19:25:33","subject":"Re: [U-Boot] [PATCH v2 14/16] dtoc: Support properties containing\n\tmultiple phandle values","submitter":{"id":12501,"url":"http://patchwork.ozlabs.org/api/people/12501/","name":"Simon Glass","email":"sjg@google.com"},"content":"At present dtoc has a very simplistic view of phandles. It assumes that\na property has only a single phandle with a single argument (i.e. two\ncells per property).\n\nThis is not true in many cases. Enhance the implementation to scan all\nphandles in a property and to use the correct number of arguments (which\ncan be 0, 1, 2 or more) when generating the C code. For the struct\ndefinitions, use a struct which can hold the maximum number of arguments\nused by the property.\n\nSigned-off-by: Simon Glass <sjg@chromium.org>\n---\n\nChanges in v2: None\n\n include/dt-structs.h             |  5 +++++\n tools/dtoc/dtb_platdata.py       | 21 ++++++++++++++-------\n tools/dtoc/dtoc_test_phandle.dts | 17 +++++++++++++++--\n tools/dtoc/test_dtoc.py          | 27 ++++++++++++++++++++++++---\n 4 files changed, 58 insertions(+), 12 deletions(-)\n\nApplied to u-boot-fdt thanks!","headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"uytv36HS\"; dkim-atps=neutral"],"Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xv5YW6rkFz9s7c\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSat, 16 Sep 2017 05:48:31 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid A3C86C22486; Fri, 15 Sep 2017 19:33:45 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id E51E7C2200F;\n\tFri, 15 Sep 2017 19:33:42 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 7829AC21FDD; Fri, 15 Sep 2017 19:27:16 +0000 (UTC)","from mail-io0-f170.google.com (mail-io0-f170.google.com\n\t[209.85.223.170])\n\tby lists.denx.de (Postfix) with ESMTPS id F1820C22009\n\tfor <u-boot@lists.denx.de>; Fri, 15 Sep 2017 19:25:37 +0000 (UTC)","by mail-io0-f170.google.com with SMTP id l15so10831613iol.8\n\tfor <u-boot@lists.denx.de>; Fri, 15 Sep 2017 12:25:37 -0700 (PDT)","from 480794996271 named unknown by gmailapi.google.com with\n\tHTTPREST; Fri, 15 Sep 2017 12:25:34 -0700"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H2, T_DKIM_INVALID,\n\tUNPARSEABLE_RELAY autolearn=unavailable\n\tautolearn_force=no version=3.4.0","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;\n\ts=20161025; \n\th=mime-version:sender:from:in-reply-to:references:date:message-id\n\t:subject:to:cc;\n\tbh=ToqNeFu6CXlkNZiH8N+1wVn4MlrCpTrzm3YVR53vVaQ=;\n\tb=uytv36HScRHh6giE0Vh0em8Sh66pOiQYdHTHK1oBo7O+k8yPGGCFFB5e9a5zh8Wfma\n\tE+w9i1B91pJURCEDpCTjEf1f2uUgIiN00khc4fQ3kK1surqs6VBlczceX9kJ34aP6WUa\n\t3/jb6LU92EUfylweGu2M8HiWNl6rwHjJPVRmE8eLcQ1+V+vVycC6hJc7fC/ePC67DPwQ\n\t0d6YND6RPeIjhTjm/ToJcJCwpjxevCOElqKBMW1XSAIFEfKr/ip8m+Ax76ydFP+ZCNck\n\t67LArjLe33NbNXS/jbMtfOZAud9EW9xwIPTw43PT/5xox8HMQkj5PrNsdQR+jQd9hxgo\n\tiRgQ==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:sender:from:in-reply-to:references\n\t:date:message-id:subject:to:cc;\n\tbh=ToqNeFu6CXlkNZiH8N+1wVn4MlrCpTrzm3YVR53vVaQ=;\n\tb=FYENrhxacrao+/N+iRQ9uPYNbXaSUaWc0xCa4HDkzbkq3ED7HWkOHMVpiu15w2lYv9\n\tmSoZpMYJ77GJbBWWiu2lYGbbCvP5OqZDEaCB7iBV6VqBDbvxAWrB0h0HMtATgCmYBChG\n\tiYtrNaPKJwbPPeVy3KlgELRj1aEBcKqrr9vnFdE79wPit9CuS2WJmNxO1I1IJvd5kwjZ\n\taKbbuYjUlXSw3WqC+adHeolpN5E3e3jDgEwXEGB8OGgCPKtikqmJ9mOLEuh+9kZWkg+r\n\tf7S24B1Fkl5R9S3RrORIcYw1LLpjBeO4ttW0tcIOqjzkobDcEa0nhXB0XmSSY3ullEsj\n\tQvFA==","X-Gm-Message-State":"AHPjjUipUWHVBbCEhWhvoedQgvA7CAXbWvT6b91LU8qd3qQeH8d1Ah5R\n\t2oH2cFYQvk71YSxQgIRYjZEzli6MoLPJTzkVhvdk4w==","X-Google-Smtp-Source":"AOwi7QCJ7u3k1eF+jQBKbEhiyQFdxjPBFFkMS2blyNzDCEMmbS6yj1LUmbuTdmA1TRpYSzjMPBpCIL4CrW9F8y6sSH0=","X-Received":"by 10.202.170.204 with SMTP id\n\tt195mr13975200oie.277.1505503536583; \n\tFri, 15 Sep 2017 12:25:36 -0700 (PDT)","MIME-Version":"1.0","From":"sjg@google.com","In-Reply-To":"<20170829201601.64312-15-sjg@chromium.org>","References":"<20170829201601.64312-15-sjg@chromium.org>\n\t<20170829201601.64312-1-sjg@chromium.org>","Date":"Fri, 15 Sep 2017 12:25:33 -0700","X-Google-Sender-Auth":"4x-y_hQ-HRB5jnXd5ShHX53q9BY","Message-ID":"<CAPnjgZ1Mm5tT+f3iNcaAgaoW7ajss0ruY7EhEaSCd8mRGTOZoQ@mail.gmail.com>","To":"Simon Glass <sjg@chromium.org>","Cc":"Tom Rini <trini@konsulko.com>, U-Boot Mailing List <u-boot@lists.denx.de>","Subject":"Re: [U-Boot] [PATCH v2 14/16] dtoc: Support properties containing\n\tmultiple phandle values","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<http://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n\t<mailto:u-boot-request@lists.denx.de?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>"}}]