Re: [PATCH] ACPI: Enable SCI_EMULATE to manually simulate physicalhotplug testing.

From: Toshi Kani
Date: Thu Sep 13 2012 - 20:28:30 EST


> > >> +
> > >> + /*
> > >> + * Check for internal object and make sure there is a handler
> > >> + * registered for this object
> > >> + */
> > >> + obj_desc = acpi_ns_get_attached_object(node);
> > >> + if (obj_desc) {
> > >> + if (obj_desc->common_notify.notify_list[0]) {
> > >
> > > Is the above check necessary? acpi_ev_queue_notify_request() sets up to
> > > call the global handler, acpi_gbl_global_notify[0], even if the object
> > > does not have a local handler registered.
> >
> > Not sure.
> >
> > maybe Len or other acpi guyes could answer your questions.
>
> I think this check should be removed, but would like someone to
> verify...


Hi Yinghai,

Attached is my suggested update to your patch. It allows a SCI to be
sent to any object, and therefore can be used for testing the global
notify handler. Some drivers such as dock.c only register their handler
to the global notify handler. I also made a few minor changes. I have
been testing with this update and it is working fine. I like this
feature, so I hope we can make progress with this update.

Thanks,
-Toshi