Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
Date:   Fri, 13 Jan 2023 11:20:34 +0100
From:   Michael Walle <michael@walle.cc>
To:     Rob Herring <robh+dt@kernel.org>
Cc:     Robert Marko <robimarko@gmail.com>,
        Frank Rowand <frowand.list@gmail.com>,
        Peng Fan <peng.fan@nxp.com>,
        Miquel Raynal <miquel.raynal@bootlin.com>,
        Alexander Stein <alexander.stein@ew.tq-group.com>,
        devicetree@vger.kernel.org,
        Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
        linux-kernel@vger.kernel.org, Aisheng Dong <aisheng.dong@nxp.com>
Subject: Re: [PATCH] of: property: fix #nvmem-cell-cells parsing
In-Reply-To: <CAL_JsqLN0jd0nT7rC7SBMh2zLsAvGCafknnS_oqJJj+=c1m9pg@mail.gmail.com>
References: <20230110233056.3490942-1-michael@walle.cc>
 <167357352392.592020.6790951746345716129.robh@kernel.org>
 <CAL_JsqLN0jd0nT7rC7SBMh2zLsAvGCafknnS_oqJJj+=c1m9pg@mail.gmail.com>
User-Agent: Roundcube Webmail/1.4.13
Message-ID: <729fb97c96216168f73d1a7f01bfd70d@walle.cc>
Content-Type: text/plain; charset=US-ASCII;
 format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

>> > 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 <peng.fan@nxp.com>
>> > Signed-off-by: Michael Walle <michael@walle.cc>
>> > Tested-by: Robert Marko <robimarko@gmail.com>
>> > ---
>> >  drivers/of/property.c | 8 ++++----
>> >  1 file changed, 4 insertions(+), 4 deletions(-)
>> >
>> 
>> Applied, thanks!
> 
> Or not. 67b8497f005f is not in my tree, so
> 
> Acked-by: Rob Herring <robh@kernel.org>

Thanks! Srinivas, can you pick it up to get rid of the errors/warnings
in linux-next?

-michael