Attached is a small cleanup of the parameter parsing in the ide driver. This patch replaces some of the string parsing code with get_options() and it starts cleaning up the code that checks for specific chipsets. I tried to keep the chipset error checking exactly as it was before, and now the remaining code looks a little strange. It does something like this: if (parameter is a chipset name) { if (chipset type is already defined for selected port) goto bad_option; if ((chipset parameter != dc4030) && (hw!=ide0) ) goto bad_channel; if ((chipset parameter != dc4030) && chipset type is already defined for ide1) goto bad_option; } Can some of this be removed? It would allow the remaining code to be simplified.. Gerald