Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755550AbcDGKVL (ORCPT ); Thu, 7 Apr 2016 06:21:11 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:41649 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754880AbcDGKVI (ORCPT ); Thu, 7 Apr 2016 06:21:08 -0400 Subject: Re: [PATCH] of/platform: Allow secondary compatible match in of_dev_lookup To: Rob Herring , Tony Lindgren References: <1459546504-32668-1-git-send-email-tony@atomide.com> <20160401214053.GQ9329@atomide.com> CC: Frank Rowand , Grant Likely , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-omap , Felipe Balbi From: Grygorii Strashko Message-ID: <5706348C.1040709@ti.com> Date: Thu, 7 Apr 2016 13:21:00 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2127 Lines: 49 On 04/07/2016 07:52 AM, Rob Herring wrote: > On Fri, Apr 1, 2016 at 4:40 PM, Tony Lindgren wrote: >> * Tony Lindgren [160401 14:37]: >>> We currently try to match of_dev_auxdata based on compatible, >>> IO address, and device name. But in some cases we have multiple >>> instances of drivers that can use the same auxdata. >>> >>> Let's add an additional secondary lookup for generic compatible >>> match for auxdata if no device specific match is found. This does >>> not change the existing matching, and still allows adding device >>> specific auxdata. >>> >>> This simplifies things as specifying the IO address and device >>> name is prone errors as it requires maintaining an in kernel >>> database for each SoC. >> >> And here's what I can apply later on to get rid of some >> ifdeffery. >> >> I'm also planning to move some of the legacy omap hwmod >> functionality into proper device drivers, so can generic >> pdata for that too. > > Why can't the platform data be moved into the driver given that it > appears to be only SoC family specific? Auxdata was somewhat intended > to be temporary. It appears there is already some per compatible match > data for these OMAP parts in the driver. > Most probably this is required to pass some data from parent device to children when parent dev instantiate children from DT, at least I've experimented with this in mostly similar way (I've not added second pass and did break in the first if !phys_addr, but Tony's patch is more correct). For example, - USB dwc3 platform/integration layer dev creates DWC3-core device (of_platform_populate()) - DWC3-core device creates xhci device - USB dwc3 platform/integration layer dev can dynamically get rev info from HW and identify limitation/erratas/quirks which need to be applied to to its children. - DWC3-core can dynamically get rev info ... I was not able to find any other proper way to pass this (platfrom)data to children, except using Auxdata. And, as per Felipe Balbi, It was the major reason why now DWC3-core device creates xhci device manually. -- regards, -grygorii