From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH] cpuidle: kirkwood: Move out of mach directory, add DT. Date: Mon, 11 Feb 2013 11:41:40 +0000 Message-ID: <20130211114140.D47AF3E1E7B@localhost> References: <1356698844-4220-1-git-send-email-andrew@lunn.ch> <20130208213441.014343E2C27@localhost> <20130210185812.GI20212@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130210185812.GI20212@lunn.ch> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org Cc: Andrew Lunn , Jason Cooper , len.brown@intel.com, devicetree-discuss@lists.ozlabs.org, rafael.j.wysocki@intel.com, linux ARM List-Id: devicetree@vger.kernel.org On Sun, 10 Feb 2013 19:58:12 +0100, Andrew Lunn wrote: > > > +* QNAP Power Off > > > + > > > +QNAP NAS devices have a microcontroller controlling the main power > > > +supply. This microcontroller is connected to UART1 of the Kirkwood and > > > +Orion5x SoCs. Sending the charactor 'A', at 19200 baud, tells the > > > +microcontroller to turn the power off. This driver adds a handler to > > > +pm_power_off which is called to turn the power off. > > > + > > > +Required Properties: > > > +- compatibile: Should be "qnap,power-off" > > > > It would do well to have the model number embedded into this compatible > > string since it is unlikely that every single QNAP device will use the > > same controller interface. > > Hi Grant > > All QNAP devices known to mainline use this same controller interface. > > So that would be ts109, ts209, ts409, ts119, ts219, ts419, plus a few > other models which we don't differentiate, like ts219P and ts219P+. > > Any suggestions what to use in the compatible string? for the ts109: compatible = "qnap,ts109-poweroff"; for the others: compatible = "qnap,ts[model]-poweroff", "qnap,ts109-poweroff"; That is the usual way things are done. The newer model claims compatibility with the older. Some have also tried to use a 'generic' value for claiming compatibility (ie. "qnap,tsxxx-poweroff") but I don't recommend that because generic values aren't 'anchored' to any real piece of hardware. Instead 'qnap,ts109-poweroff' is used as the 'generic' version with the advantage that when new hardware appears that doesn't conform you simply drop the string for the older version. It is also recommended to also include the specific model number simply as a mechanism to test for quirks if ever needed... not critical though for system level devices since the top level system model property can also be used for that. g.