Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp26499534rwd; Mon, 3 Jul 2023 10:28:57 -0700 (PDT) X-Google-Smtp-Source: APBJJlGiiGHqO5b2NN3JM+VZhqFFYY9TyCS3qVY4x1dBf2nHR2etAavYEuAQwQ7EZlV0osouAeM1 X-Received: by 2002:a05:6a00:1ad1:b0:666:e42c:d5fb with SMTP id f17-20020a056a001ad100b00666e42cd5fbmr15109673pfv.32.1688405337418; Mon, 03 Jul 2023 10:28:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1688405337; cv=none; d=google.com; s=arc-20160816; b=XLPpfy1t22v/ulEd2BFiOaxjaG/Jdyew8G5a/grNALHQwSEaOB7knVQ7wuAJEuMzOn 7cH/U+VbiuTWl9mHQwBszdj9oRlnsJmlMNxVOuUSfq2yEOO2WPfK1ZuJq1nisQe3oBUn hl2ebN1s1IyVqWo93JgsyO8UxA3HVubGt7y4sdgWnjUuiXdjoGJEfnVsO2QKuCmGrn1T M7M/sAi+OmYG039mUhVSrs71OfqqMaFLh6cZY/m2IQhIF9DCT9VEeCKSQJpEzrTlfGHn VHjyKH+SuU+rTOREDxBt9z8REWnzHXpminwoZWeQyh7tsrIhqNGVszixWcs5yMyVvOTT GJuw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=vHaTg43ZN4pIqH0XjXUyIHPFQb3nQdBrMYDjetNLFHM=; fh=9+mKr/pjNBElGUYGhNXmqxQo1H2YxpiCNthRVorW2oI=; b=dLS5i+8+T720r3M2lBR8N9pEKnar7T6kDPxXUwIaYSqfodXOECncNEAY3WghYatUxO mj+nm4JurVHKt14HS9khbUoMGxwKeQ9BUtygU5YctWWsi9OY1SW6qovyXa/zjaRgjRlS y54RIVe8iAsRL5+QSUnhAouryKTCXq90/B/4Fj9BWYTOguFITcM38spxhvtjalD7vKaO OoJbLyjcbsv+ldPKncv3TIuj9iExP5Y4vYz/tI3ntL2zcce9C9OHWTNLQmbKmBjerzz+ vcn1Lo7qf2OqG2lw1nvekvmcegqm1JvjjqrMCWv3rAeLPJHz2p/1EuhnKrK9Ekyfeoqz blTw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@wanadoo.fr header.s=t20230301 header.b=UYuVx8uN; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id r7-20020aa79887000000b0068259969675si7831334pfl.297.2023.07.03.10.28.43; Mon, 03 Jul 2023 10:28:57 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@wanadoo.fr header.s=t20230301 header.b=UYuVx8uN; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231166AbjGCRCA (ORCPT + 99 others); Mon, 3 Jul 2023 13:02:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35770 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230102AbjGCRBw (ORCPT ); Mon, 3 Jul 2023 13:01:52 -0400 Received: from smtp.smtpout.orange.fr (smtp-24.smtpout.orange.fr [80.12.242.24]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 99A73E69 for ; Mon, 3 Jul 2023 10:01:44 -0700 (PDT) Received: from [192.168.1.18] ([86.243.2.178]) by smtp.orange.fr with ESMTPA id GMvbqaLPIL1SxGMvcqewci; Mon, 03 Jul 2023 19:01:36 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wanadoo.fr; s=t20230301; t=1688403696; bh=vHaTg43ZN4pIqH0XjXUyIHPFQb3nQdBrMYDjetNLFHM=; h=Date:Subject:To:Cc:References:From:In-Reply-To; b=UYuVx8uN3j2zQ1Mj6NoQNPv2Vn3M3UmX2kUAmwh91xHCBmSln+5lvS0Xkskkuw8kg KKootEW+nAqL5ZxwXoLZAai/2L1plig0O9M/qQZjdmRNUNziTh5t8qZF7frgly9HlH 9fCQV1+wNvF0JqWAWyiz/WfMQf1SIQemHnDTejaK+LiFDKuia2kw7GtOpgZNd7eyQq nXc84LC9poLQwvAbwB5yDUbS5nzK2cMqsvfaFSjTvyUUBSSxwu0G7kgRG9SCjiAyA7 ZKZMKkd/WHajeNDRsc09Xl6oUxOLrmUdvQc8k2ayx82AubPdhuOPZgyF5ZbKaUmrBq XNGv8ZIj0ny/Q== X-ME-Helo: [192.168.1.18] X-ME-Auth: Y2hyaXN0b3BoZS5qYWlsbGV0QHdhbmFkb28uZnI= X-ME-Date: Mon, 03 Jul 2023 19:01:36 +0200 X-ME-IP: 86.243.2.178 Message-ID: Date: Mon, 3 Jul 2023 19:01:31 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH] powerpc: powermac: Use of_get_cpu_hwid() to read CPU node 'reg' To: Michael Ellerman , Benjamin Herrenschmidt , Nicholas Piggin , Christophe Leroy , Rob Herring Cc: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org References: <20230319145931.65499-1-robh@kernel.org> <168836201902.50010.13066091729121047041.b4-ty@ellerman.id.au> Content-Language: fr, en-US From: Christophe JAILLET In-Reply-To: <168836201902.50010.13066091729121047041.b4-ty@ellerman.id.au> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.2 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_PASS,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Le 03/07/2023 à 07:26, Michael Ellerman a écrit : > On Sun, 19 Mar 2023 09:59:31 -0500, Rob Herring wrote: >> Replace open coded reading of CPU nodes' "reg" properties with >> of_get_cpu_hwid() dedicated for this purpose. >> >> > > Applied to powerpc/next. > > [1/1] powerpc: powermac: Use of_get_cpu_hwid() to read CPU node 'reg' > https://git.kernel.org/powerpc/c/bc1cf75027585f8d87f94e464ee5909acf885a8c > > cheers > Hi, I guess, that it does not really matter, but shouldn't the of_node_put() be *after* the "reset_io = *rst;" statements to be absolutely safe? (This change is in my backlog and I have apparently never proposed it) CJ