From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Sverdlin Subject: Re: [RFC] [PATCH 3/3] IRQ: irq domain: defer of irq ressoure resolve at platform_drv_probe Date: Mon, 24 Jun 2013 11:47:05 +0200 Message-ID: <51C81599.3020407@nsn.com> References: <20130528145219.GA30411@game.jcrosoft.org> <1369753729-12997-1-git-send-email-plagnioj@jcrosoft.com> <1369753729-12997-3-git-send-email-plagnioj@jcrosoft.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1369753729-12997-3-git-send-email-plagnioj-sclMFOaUSTBWk0Htik3J/w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: ext Jean-Christophe PLAGNIOL-VILLARD Cc: devicetree-discuss-mnsaURCQ41sdnm+yROfE0A@public.gmane.org, Ralf Baechle , Rob Herring List-Id: devicetree@vger.kernel.org Hi! On 05/28/2013 05:08 PM, ext Jean-Christophe PLAGNIOL-VILLARD wrote: > Today in the current of implementation we populate all the ressources > at of_platform_populate time. But this leed to a chicken-egg dilemat > some the irq present in DT are from platform_device too. And you can > not resolve them as of_platform_populate. So delay the populate of irq > at platform_drv_probe. > > And if the irq_domain is not yet present just defer the probe (GPIO as example) > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD Tested-by: Alexander Sverdlin Tested the whole series. This is definitely a necessary feature. I've tested it on MIPS64 system with merge-able device-subtrees. If you would come with the new patch series, please put me on CC, I'll test it again. > Cc: Grant Likely > Cc: Rob Herring > Cc: Arnd Bergmann > Cc: Linus Walleij > Cc: Benjamin Herrenschmidt > Cc: Ralf Baechle > Cc: Nicolas Ferre > --- > drivers/base/platform.c | 5 +++++ > drivers/of/platform.c | 29 +++++++++++++++++++++++++++-- > include/linux/of_platform.h | 7 +++++++ > 3 files changed, 39 insertions(+), 2 deletions(-) ... -- Best regards, Alexander Sverdlin.