On Wed, 2009-12-02 at 10:07 +0900, Tejun Heo wrote: > On 12/02/2009 09:36 AM, Benjamin Herrenschmidt wrote: > > This is a libata driver for the "macio" IDE controller used on most Apple > > PowerMac and PowerBooks. It's a libata equivalent of drivers/ide/ppc/pmac.c > > > > It supports all the features of its predecessor, including mediabay hotplug > > and suspend/resume. It should also support module load/unload. > > > > The timing calculations have been simplified to use pre-calculated tables > > compared to drivers/ide/pmac.c and it uses the new mediabay interface > > provided by a previous patch. > > Other than the media bay locking thing I mentioned before, it looks > good to me and the media bay thing can be dealth with further patches, > so... > > Acked-by: Tejun Heo <tj@xxxxxxxxxx> Thanks. I'm not sure what media bay locking problem you mentioned tho... The way the driver does the locking now is that I block the media-bay thread (lock/unlock_media_bay calls) around initial registration and driver removal which blocks the callback event. At any other time, the callback event will just schedule EH so things should be fine. I've left the debouncing in the mediabay code as well for both insertion and removal so there should be no issue here with glitches. Cheers, Ben. -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
