From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <benh@kernel.crashing.org>
Received: from gate.crashing.org (gate.crashing.org [63.228.1.57])
 (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by lists.ozlabs.org (Postfix) with ESMTPS id 6B18A1A0009
 for <linuxppc-dev@lists.ozlabs.org>; Mon,  7 Jul 2014 16:56:31 +1000 (EST)
Message-ID: <1404716172.16156.8.camel@pasglop>
Subject: Re: [PATCH v4] powerpc/powernv: hwmon driver for power, fan rpm,
 voltage and temperature
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Guenter Roeck <linux@roeck-us.net>
Date: Mon, 07 Jul 2014 16:56:12 +1000
In-Reply-To: <53B76220.9090900@roeck-us.net>
References: <20140704105343.22437.52125.stgit@localhost.localdomain>
 <53B76220.9090900@roeck-us.net>
Content-Type: text/plain; charset="UTF-8"
Mime-Version: 1.0
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
 lm-sensors@lm-sensors.org, sbhat@linux.vnet.ibm.com,
 Neelesh Gupta <neelegup@linux.vnet.ibm.com>, linuxppc-dev@lists.ozlabs.org,
 jdelvare@suse.de
List-Id: Linux on PowerPC Developers Mail List <linuxppc-dev.lists.ozlabs.org>
List-Unsubscribe: <https://lists.ozlabs.org/options/linuxppc-dev>,
 <mailto:linuxppc-dev-request@lists.ozlabs.org?subject=unsubscribe>
List-Archive: <http://lists.ozlabs.org/pipermail/linuxppc-dev/>
List-Post: <mailto:linuxppc-dev@lists.ozlabs.org>
List-Help: <mailto:linuxppc-dev-request@lists.ozlabs.org?subject=help>
List-Subscribe: <https://lists.ozlabs.org/listinfo/linuxppc-dev>,
 <mailto:linuxppc-dev-request@lists.ozlabs.org?subject=subscribe>

On Fri, 2014-07-04 at 19:25 -0700, Guenter Roeck wrote:
> > +cooling-fan#8-data {
> > +     sensor-id = <0x7052107>;
> > +     phandle = <0x10000028>;
> > +     linux,phandle = <0x10000028>;
> > +     compatible = "ibm,opal-sensor-cooling-fan";
> 
> phandle and linux-phandle are neither documented nor used. Either
> document or drop.

They are core device-tree stuff, as such they are used potentially
anywhere and afaik can be / are generated by DTC but yes, this is
not the place to document them.

Or rather, "phandle" is, "linux,phandle" is an older variant used
for backward compatibility that our firmware still generates but
definitely something we can strip from the doc.

Cheers,
Ben,