From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH v9 06/10] ata: zpodd: check zero power ready status Date: Fri, 28 Dec 2012 16:16:35 -0500 Message-ID: <20121228211635.GB3062@mtj.dyndns.org> References: <1353906191.2523.25.camel@dabdike> <21511277.LLinyDpbAK@vostro.rjw.lan> <20121128013928.GB15971@htj.dyndns.org> <1354092969.2276.49.camel@dabdike> <20121203081321.GA9990@mint-spring.sh.intel.com> <1354523143.2307.2.camel@dabdike.int.hansenpartnership.com> <20121203162323.GB19802@htj.dyndns.org> <50D2AB1D.9050602@intel.com> <20121225171723.GI10220@mtj.dyndns.org> <50DA55F6.8080706@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <50DA55F6.8080706@intel.com> Sender: linux-pm-owner@vger.kernel.org To: Aaron Lu Cc: James Bottomley , "Rafael J. Wysocki" , linux-pm@vger.kernel.org, Jeff Garzik , Alan Stern , Jeff Wu , Aaron Lu , linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org, linux-acpi@vger.kernel.org List-Id: linux-acpi@vger.kernel.org Hello, On Wed, Dec 26, 2012 at 09:42:14AM +0800, Aaron Lu wrote: > > This is really a round-about way to find out the matching device and > > it wouldn't work if the disk device nests deeper. Doesn't really look > > like a good idea to me. > > I don't quite understand the 'disk device nests deeper' case, can you > please elaborate? My understanding is, as long as the disk's part0 > device has a parent, this function should work. For LLDs want to take Hmmm, maybe I misread but it looked like it wouldn't work if there are intermediate nodes between the parent device and part0. It might not happen for sata but I don't think it's a good idea to assume that the part0 and hardware device are connected directly. In general, it's a quite roundabout way to do it. Let's just push it through SCSI. That's how everything else is routed after all. It's confusing to do this one differently. Thanks. -- tejun