Messages in this thread | ![/](/images/icornerl.gif) | | Date | Fri, 25 Apr 2003 12:15:17 +0200 | From | Jean Delvare <> | Subject | Re: [PATCH 2.4] dmi_ident made public |
| |
> > If this patch is accepted and applied, I'll work together with Peter > > to get the three above-mentioned modules simplified, as well as any > > other I may have missed. Also, I'll take care of porting this patch > > to the 2.5 series, since it also belongs there. > > i2c-piix4 in the 2.5 kernel tree does not need this patch, as > everything it needs to detect IBM laptops is already made public. See > the current 2.5 releases to verify this.
Thanks for pointing this out. I took a look and could actually verify it. This made me think again about the whole problem. I wondered wether all other modules could be fixed that way (in which case my patch wouldn't make sense), but it turns out that neither i8k nor omke can use the same trick as the one you used for i2c-piix4. This is due to the fact that i2c-piix4 only needs a flag (does the system match a given DMI "mask"? yes/no) whereas the other modules need the actual data. Peter, correct me if I'm wrong.
What's more, I plan to write another module that exports the DMI data, as scanned at boot time, to userland (via sysfs), and such a module definitely requires that the DMI data is made public in dmi_scan.
The good thing is that I think I now understand a bit better what Alan meant yesterday by "register multiple DMI tables for boot time scanning to set further flags in the dmi properties post scan". It must be what you did for i2c-piix4, and isn't what I need there if my analysis is correct.
So, I still think my patch makes sense and should be applied.
-- Jean Delvare http://www.ensicaen.ismra.fr/~delvare/ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| ![\](/images/icornerr.gif) |