From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751708AbcIIOKs (ORCPT ); Fri, 9 Sep 2016 10:10:48 -0400 Received: from mail-yw0-f169.google.com ([209.85.161.169]:33273 "EHLO mail-yw0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751092AbcIIOKq (ORCPT ); Fri, 9 Sep 2016 10:10:46 -0400 Date: Fri, 9 Sep 2016 10:10:55 -0400 From: Tom Rini To: Rob Herring Cc: Frank Rowand , Tony Lindgren , Grant Likely , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-omap , Nishanth Menon , Tero Kristo Subject: Re: [PATCHv2] of: Add generic handling for ePAPR 1.1 fail-sss states Message-ID: <20160909141055.GZ4990@bill-the-cat> References: <20160829223542.18871-1-tony@atomide.com> <57C74306.9020901@gmail.com> <20160831214151.wzq7y54xxs2qj422@atomide.com> <57D1B668.3070001@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Sep 08, 2016 at 09:43:03PM -0500, Rob Herring wrote: > On Thu, Sep 8, 2016 at 2:05 PM, Frank Rowand wrote: > > On 09/08/16 06:38, Rob Herring wrote: > >> On Wed, Aug 31, 2016 at 4:41 PM, Tony Lindgren wrote: > >>> * Frank Rowand [160831 13:51]: [snip] > >>>> Why not just create a new property that describes the hardware? > >>>> Perhaps something like: > >>>> > >>>> incomplete = "pins_output", "buggy_dma"; > >>> > >>> New property for incomplete works for me. Rob, got any comments here? > >> > >> Pins not muxed out or connected on the board has to be the #1 reason > >> for disabled status. I don't think we need or want another way to > >> express that. > > > > How is that expressed now? > > status = "disabled"; It might be worth repeating that the problem that needs to be solved is that we have an IP block that exists in some sense but needs to be shut down (or some similar case the driver can handle) rather than ignored like today. This is not the same as having 5 UARTs available from the SoC but only one is actually in some useful physical state, so list all 5 in the dtsi, status disabled, and then in the board dts file enable the exposed ones. -- Tom