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 lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BB9FDC43334 for ; Wed, 29 Jun 2022 16:18:13 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4LY6840dMZz3dQx for ; Thu, 30 Jun 2022 02:18:12 +1000 (AEST) Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=kernel.crashing.org (client-ip=63.228.1.57; helo=gate.crashing.org; envelope-from=segher@kernel.crashing.org; receiver=) Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) by lists.ozlabs.org (Postfix) with ESMTP id 4LY67Y2Tfyz3bvb for ; Thu, 30 Jun 2022 02:17:43 +1000 (AEST) Received: from gate.crashing.org (localhost.localdomain [127.0.0.1]) by gate.crashing.org (8.14.1/8.14.1) with ESMTP id 25TGD56V017150; Wed, 29 Jun 2022 11:13:05 -0500 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id 25TGD3U2017144; Wed, 29 Jun 2022 11:13:03 -0500 X-Authentication-Warning: gate.crashing.org: segher set sender to segher@kernel.crashing.org using -f Date: Wed, 29 Jun 2022 11:13:03 -0500 From: Segher Boessenkool To: Krzysztof Kozlowski Subject: Re: [PATCH v3 02/12] powerpc: wiiu: device tree Message-ID: <20220629161302.GG25951@gate.crashing.org> References: <20220622131037.57604-1-ash@heyquark.com> <20220628133144.142185-1-ash@heyquark.com> <20220628133144.142185-3-ash@heyquark.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.4.2.3i X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, linkmauve@linkmauve.fr, linux-kernel@vger.kernel.org, rw-r-r-0644@protonmail.com, robh+dt@kernel.org, paulus@samba.org, Ash Logan , krzysztof.kozlowski+dt@linaro.org, j.ne@posteo.net, linuxppc-dev@lists.ozlabs.org, joel@jms.id.au Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" On Wed, Jun 29, 2022 at 11:58:18AM +0200, Krzysztof Kozlowski wrote: > On 28/06/2022 15:31, Ash Logan wrote: > > + model = "nintendo,wiiu"; > > It's not compatible, but user-visible string, e.g. "Nintendo Wii U" The "model" property in OF is documented as: --- “model” S Standard property name to define a manufacturer’s model number. prop-encoded-array: Text string, encoded with encode-string. A manufacturer-dependent string that generally specifies the model name and number (including revision level) for this device. The format of the text string is arbitrary, although in conventional usage the string begins with the name of the device’s manufacturer as with the “name” property. Although there is no standard interpretation for the value of the “model” property, a specific device driver might use it to learn, for instance, the revision level of its particular device. See also: property, model. Used as: " XYZCO,1416-02" encode-string " model" property --- > > + cpus { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + /* TODO: Add SMP */ > > + PowerPC,espresso@0 { > > Node name should be generic, so "cpu". Unless something needs the > specific node name? This is how most other PowerPC firmwares do it. The PowerPC processor binding is older than the generic naming practice, so CPU nodes have device_type "cpu" instead. This is a required property btw, with that value. (There is no requirement on the names of the CPU nodes). There is no added value in generic naming for CPU nodes anyway, since you just find them as the children of the "/cpus" node :-) Segher