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 5BCF5C38141 for ; Fri, 20 Jan 2023 17:48:55 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230150AbjATRsy (ORCPT ); Fri, 20 Jan 2023 12:48:54 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38580 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230389AbjATRst (ORCPT ); Fri, 20 Jan 2023 12:48:49 -0500 Received: from mail-wr1-x42d.google.com (mail-wr1-x42d.google.com [IPv6:2a00:1450:4864:20::42d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6D895881E6 for ; Fri, 20 Jan 2023 09:48:41 -0800 (PST) Received: by mail-wr1-x42d.google.com with SMTP id y1so1022736wru.2 for ; Fri, 20 Jan 2023 09:48:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=usZwUafmssHHjBMbZ97d1dp/H4o4iCVGkXuyEhBu/eE=; b=Xk8v6S4B9l/1/rZC5Acfrnhs5lgw/5B2wVyXT0iRjq2/6CzwmVk2HVGvuCnXq25YXm 7cXqKvaFgFecajORnMSICkCtzhDysvIuMaPZJ94snTgQGUbtynCWhpOtJmthFeTgqkJn llBYq1ZjfKp4J1H0YZJfFqVXXGwTAq1xXzPKWxBCDC6HCQfgYdlxa0QDpYlkmhZB6SBy YisYi/DGL2scYRLQwGhRtDm6Ze9Mp2Odt8idZOLZCMsASSiqCankWGGxlzgfMiSmbLEe ENPMDZ6CzrCrFOb7UgoyE58gi+nHNarCy+wsA7liMfKbFz8pYZ4jVL2MIz8Lm5451MHV QoQA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc: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=usZwUafmssHHjBMbZ97d1dp/H4o4iCVGkXuyEhBu/eE=; b=KZMwypMnphGn4w6OPltuQAD7FURio8H/W3+c/fu8cZ/YSRT6Muk2Blsm2opm1YdrYt i1Ig7w71Zyo0KgqKGN9PTGPGg5WTglDfrCXbNM1Ds6MwWdjUWD2ZFKNAHr8R5dR0Ohjs K5xGhSNpli68xOiZGTdIayztJ+5oUJ2PZKC9HGlIdD1SSCHiWzvMG5u2UwkSKERUx1Zl Kykb+NGqw2ZK9ucEQUt5GSrdIV1POBeUlXnjfvnfWTEFt2okXP0rivG0C2LaJsbHr0g9 BJp/TLQ3RoNiEzo6Baz3ygt2BIlGyYs4Oz4ZYNm1iLuHKEZZjLiefGEfg3fCD5f4IQXA CBVQ== X-Gm-Message-State: AFqh2kp6L/mqfq4rpMP4FeqdiRd/lB6MjeuQ/glnjwhhqvNF7weZnB5+ t1i5GzxoEwfTk9el63hINrRrog== X-Google-Smtp-Source: AMrXdXuScCkgBt6ufGU4Q+zUCLg+jabHQ01J+OwfTLnAkLy8rziGHZ9jITRpp+e1sZ2s8g6Fe6Q6EQ== X-Received: by 2002:adf:e70a:0:b0:2b5:90e:cfa5 with SMTP id c10-20020adfe70a000000b002b5090ecfa5mr13962181wrm.29.1674236920097; Fri, 20 Jan 2023 09:48:40 -0800 (PST) Received: from [192.168.1.195] ([5.133.47.210]) by smtp.googlemail.com with ESMTPSA id r12-20020a5d4e4c000000b002be1d1c0bd7sm9540535wrt.93.2023.01.20.09.48.39 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 20 Jan 2023 09:48:39 -0800 (PST) Message-ID: Date: Fri, 20 Jan 2023 17:48:38 +0000 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.2 Subject: Re: [PATCH] of: property: fix #nvmem-cell-cells parsing Content-Language: en-US To: Michael Walle , Rob Herring , Frank Rowand , Robert Marko , Alexander Stein , Peng Fan , Miquel Raynal Cc: devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Aisheng Dong References: <20230110233056.3490942-1-michael@walle.cc> From: Srinivas Kandagatla In-Reply-To: <20230110233056.3490942-1-michael@walle.cc> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 10/01/2023 23:30, Michael Walle wrote: > Commit 67b8497f005f ("of: property: make #.*-cells optional for simple > props") claims to make the cells-name property optional for simple > properties, but changed the code for the wrong property, i.e. for > DEFINE_SUFFIX_PROP(). Fix that. > > Fixes: 67b8497f005f ("of: property: make #.*-cells optional for simple props") > Reported-by: Peng Fan > Signed-off-by: Michael Walle > Tested-by: Robert Marko > --- Applied thanks, --srini > drivers/of/property.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/of/property.c b/drivers/of/property.c > index 8d9ba20a8f90..95b838185b2f 100644 > --- a/drivers/of/property.c > +++ b/drivers/of/property.c > @@ -1202,8 +1202,8 @@ static struct device_node *parse_prop_cells(struct device_node *np, > if (strcmp(prop_name, list_name)) > return NULL; > > - if (of_parse_phandle_with_args(np, list_name, cells_name, index, > - &sup_args)) > + if (__of_parse_phandle_with_args(np, list_name, cells_name, 0, index, > + &sup_args)) > return NULL; > > return sup_args.np; > @@ -1256,8 +1256,8 @@ static struct device_node *parse_suffix_prop_cells(struct device_node *np, > if (strcmp_suffix(prop_name, suffix)) > return NULL; > > - if (__of_parse_phandle_with_args(np, prop_name, cells_name, 0, index, > - &sup_args)) > + if (of_parse_phandle_with_args(np, prop_name, cells_name, index, > + &sup_args)) > return NULL; > > return sup_args.np;