On Tue, 20 Mar 2007 16:53:21 +0100
Holger Macht <hmacht@xxxxxxx> wrote:
> On Mon 19. Mar - 11:04:12, Kristen Carlson Accardi wrote:
> > On Sun, 18 Mar 2007 19:55:30 +0100
> > Holger Macht <hmacht@xxxxxxx> wrote:
> >
> > > On Sun 18. Mar - 15:36:52, Henrique de Moraes Holschuh wrote:
> > > > On Sun, 18 Mar 2007, Holger Macht wrote:
> > > > > those ThinkPads where it is needed. Afterwards it does the corresponding
> > > > > dock/undock request on ibm_acpi. And this works reliably good what I can
> > > > > see from the feedback I already got. But for this to work, userspace would
> > > >
> > > > It should work with the generic bay device too, but I have no ideas about
> > > > dock. But you'll need to deal with udev with the new bay device, something
> > > > I am not too happy about. These things are ACPI events, they should remain
> > > > so unless all other ACPI events are going to become uevents.
> > >
> > > It doesn't work, I've already tried. The bay driver only emits an event if
> > > you really try to remove the bay, but not on docking/undocking.
> > >
> > > Regards,
> > > Holger
> > >
> >
> > this *should* work. The Bay driver registers with the dock driver to get
> > dock events:
> >
> > /* if we are on a dock station, we should register for dock
> > * notifications.
> > */
> > if (bay_is_dock_device(handle)) {
> > bay_dprintk(handle, "Is dependent on dock\n");
> > register_hotplug_dock_device(handle, bay_notify, new_bay);
> > }
> >
>
> But is_dock_device(...) for both the bay and for the parent handle return
> false. I'm using an X60 here, so bay_notify is never registered. I
> couldn't find the reason in the short time I was looking at it, though.
>
> Regards,
> Holger
>
Works for me on my X60 - have you tested on the latest rc kernel? Also,
let me know what your BIOS sets your SATA controller to. I can try to
duplicate your issue and get it to work.
-
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
