Re: sysfs regression on Supermicro X7DB8+

[Zhang Rui <rui.zhang@intel.com>]

> > > What is "PNPIDNON" (below) supposed to mean?
> > > 
> > For devices with PNPID, we use PNPID:instance_no as the bus_id.
> > But for others without PNPID, we set PNPIDNON:instance_no as their
> > bus_id instead.
> 
> There must be a better way to name a device that has no  PNPid   than PNPIDNON:#
> 
> How about "device:#"?
Sounds good. Will do.
> Also,  this for the first time exposes the internal fake ones
> 
> /* _HID definitions */
> 
> #define ACPI_POWER_HID                  "ACPI_PWR"
> #define ACPI_PROCESSOR_HID              "ACPI_CPU"
> #define ACPI_SYSTEM_HID                 "ACPI_SYS"
> #define ACPI_THERMAL_HID                "ACPI_THM"
> #define ACPI_BUTTON_HID_POWERF          "ACPI_FPB"
> #define ACPI_BUTTON_HID_SLEEPF          "ACPI_FSB"
> 
> If we are going to expose these, then we should spend a moment to think about how
> understandable they'll be appearing in the file system.  ACPI_FSB doesn't bring
> "Fixed Function Sleep Button" to mind when I see it...
> 
> Any reason we can't re-use "ACPI0007" for ACPI_PROCESSOR_HID
> It is already reserved in the spec.
> 
> BTW. anybody know what PNP0C08 is supposed to be?
> Seems it is reserved OS use to describe generic ACPI resources
> that are not associated with a particular device.
> 
> Note that while I don't think we are constrained here, standard PNP-id's must be of the form AAA####
> where the A's are letters and the #'s are hex.  ACPI-id's must be of the form ACPI####
> where #'s are again hex.  It would be nice if we could work within those constraints
> and not conflict with the standard in case something is expecting those formats.
> eg. IBM specific devices are IBM####, Toshiba are TOS####.
> But the down-side is that the #### doesn't tell the reader much -- particularly
> if it is non-standard.
All these PNPids are from BIOS, and we expose them to userspace
directly.
Now all ACPI devices are registered before any ACPI driver is found. So
how can we recognize the platform specific devices?
> But if we are not going to stay within the standard format and we are going to blow the 8-character
> convention, then I vote for strings that actually make sense to human,
> say something like this:
> 
> #define ACPI_POWER_HID                  "power_resource"
> #define ACPI_PROCESSOR_HID              "processor"
> #define ACPI_SYSTEM_HID                 "system"
> #define ACPI_THERMAL_HID                "thermal"
> #define ACPI_BUTTON_HID_POWERF          "button_power_fixed"
> #define ACPI_BUTTON_HID_SLEEPF          "button_sleep_fixed"
That's true. I'll convert these fake PNPids to more interesting strings.

Then I have an idea about the other ones. We can also convert the PNPids
reserved in the spec to such kind of strings.
E.g.	"PNP0C0D,PNP0C0C,PNP0C0E"	"button"
	"ACPI0003"			"ac_adapter"
	"PNP0C0A"			"battery"
....
We can use an array to maintain the name mapping between PNPid and
bus_id.
Like this:
struct acpi_name_mapping {
 char PNPid[9];	/*both fake PNPids and PNPids reserved by the spec*/
 char bus_id[20]; /* names expose to userspace */
 int instance_no;
}
static struct acpi_name_mapping acpi_name_mapping_table[] = {
	{"PNP0C0A", "battery",0},
	{"ACPI_PWER_HID", "powe_resource",0},
	....
	NULL,
};
Then we can expose such kind of strings to userspace instead of PNPids.
Though it's more friendly, the load is heavy.
The problem is whether it's worth doing.