From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932397Ab3AISBW (ORCPT ); Wed, 9 Jan 2013 13:01:22 -0500 Received: from mga09.intel.com ([134.134.136.24]:5751 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932384Ab3AISBU (ORCPT ); Wed, 9 Jan 2013 13:01:20 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.84,439,1355126400"; d="scan'208";a="270120532" Date: Wed, 9 Jan 2013 20:04:28 +0200 From: Mika Westerberg To: Vinod Koul Cc: Andy Shevchenko , Linus Walleij , Andy Shevchenko , linux-kernel@vger.kernel.org, Viresh Kumar , spear-devel Subject: Re: [resend][PATCH 01/16] dw_dmac: call .probe after we have a device in place Message-ID: <20130109180427.GA13897@intel.com> References: <20130108155423.GJ19691@intel.com> <1357719435-29955-1-git-send-email-andriy.shevchenko@linux.intel.com> <1357719435-29955-2-git-send-email-andriy.shevchenko@linux.intel.com> <20130109134757.GN19691@intel.com> <20130109153848.GT19691@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20130109153848.GT19691@intel.com> 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 Wed, Jan 09, 2013 at 07:38:48AM -0800, Vinod Koul wrote: > On Wed, Jan 09, 2013 at 04:56:45PM +0200, Andy Shevchenko wrote: > > On Wed, Jan 9, 2013 at 3:47 PM, Vinod Koul wrote: > > > On Wed, Jan 09, 2013 at 10:17:00AM +0200, Andy Shevchenko wrote: > > >> If we don't yet have the platform device for the driver when it is being loaded > > >> we fail to probe the driver. So instead of calling probe() directly we call > > >> platform_driver_register(). It will call the probe() immediately if we have the > > >> device but also makes the driver to work on platforms where the platform device > > >> is created later. > > > Why wont you have the platform device? > > > > It's still the same device as before. The only difference is in > > boot/initialization sequence. > > > > > Is this only due to your approach of > > > creating the device in PCI probe? > > > > No. We had an issue when driver was initialized (and with old version > > immediately probed) > > just before it happened in the platform devices. In such case .probe() fails. > > Above patch makes sure that platform devices are already in the > > internal structures and we could match them safely. > hmmm, but that its still not clear why that would happen. Driver init is called > later when all device creation should already be done. If the platform device is not yet created (like it might be when it is enumerated from ACPI for example), the probe fails completely. Certainly not the situation we want. Calling platform_driver_register() handles both cases properly.