From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Subject: Re: Linux-3.14-rc2: Order of serial node compatibles in DTS files. Date: Tue, 11 Feb 2014 17:41:20 -0600 Message-ID: <1392162080.6733.404.camel@snotra.buserror.net> References: <20140206082635.GA7048@visitor2.iram.es> <20140207101036.GA823@visitor2.iram.es> <20140210110342.GA15806@visitor2.iram.es> <20140211072606.GA26514@visitor2.iram.es> <63AEBD99-AA87-4FD7-BBDA-0CE419959F14@kernel.crashing.org> <52FAA97F.4060600@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <52FAA97F.4060600-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sebastian Hesselbarth Cc: Kumar Gala , Stephen N Chivers , Chris Proctor , linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, Arnd Bergmann , devicetree List-Id: devicetree@vger.kernel.org On Tue, 2014-02-11 at 23:51 +0100, Sebastian Hesselbarth wrote: > On 02/11/2014 11:33 PM, Kumar Gala wrote: > > Hmm, > > > > Wondering if this caused the issue: > > > > commit 105353145eafb3ea919f5cdeb652a9d8f270228e > > Author: Sebastian Hesselbarth > > Date: Tue Dec 3 14:52:00 2013 +0100 > > > > OF: base: match each node compatible against all given matches first > > [adding Arnd on Cc] > > Could be. I checked tty/serial/of_serial.c and it does not provide a > compatible for "fsl,ns16550". Does reverting the patch fix the issue > observed? > > I don't think the missing compatible is causing it, but of_serial > provides a DT match for .type = "serial" just to fail later on > with the error seen above. > > The commit in question reorders of_match_device in a way that match > table order is not relevant anymore. This can cause it to match > .type = "serial" first here. > > Rather than touching the commit, I suggest to remove the problematic > .type = "serial" from the match table. It is of no use anyway. Regardless of whether .type = "serial" gets removed, it seems wrong for of_match_node() to accept a .type-only match (or .name, or anything else that doesn't involve .compatible) before it accepts a compatible match other than the first in the compatible property. -Scott -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html