From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Date: Tue, 11 Sep 2018 03:34:10 +0000 Subject: Re: [PATCH 21/21] fbdev: fsl-diu: get cpu node with of_get_cpu_node Message-Id: List-Id: References: <20180905193738.19325-1-robh@kernel.org> <20180905193738.19325-22-robh@kernel.org> In-Reply-To: <20180905193738.19325-22-robh@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Rob Herring , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, Bartlomiej Zolnierkiewicz On 9/5/18 2:37 PM, Rob Herring wrote: > #ifdef CONFIG_NOT_COHERENT_CACHE > - np = of_find_node_by_type(NULL, "cpu"); > + np = of_get_cpu_node(0, NULL); This #ifdef means that it's only compiled on an MPC5121, which is a very dead platform. of_get_cpu_node() looks okay to me, but I'm going to have to assume that you know what you're doing. Acked-by: Timur Tabi