Revert "cinterion: bail earlier if the plugin doesn't support the port"

This reverts commit 1e1bfbf1d8.

Aleksander says this might break RS232<->USB converter connected
Cinterion modems, so we'll need to handle this issue another way.
This commit is contained in:
Dan Williams
2011-07-07 10:28:41 -05:00
parent 1e1bfbf1d8
commit c70d6d565d

View File

@@ -123,29 +123,12 @@ supports_port (MMPluginBase *base,
{
GUdevDevice *port;
guint32 cached = 0;
const char *subsys, *name;
guint16 vendor = 0;
/* Can't do anything with non-serial ports */
port = mm_plugin_base_supports_task_get_port (task);
if (strcmp (g_udev_device_get_subsystem (port), "tty"))
return MM_PLUGIN_SUPPORTS_PORT_UNSUPPORTED;
/* If we can get the vendor ID from udev then we can short-circuit
* the vendor name checks.
*/
subsys = g_udev_device_get_subsystem (port);
name = g_udev_device_get_name (port);
if (mm_plugin_base_get_device_ids (base, subsys, name, &vendor, NULL)) {
/* Vendors: Cinterion (0x1e2d)
* Siemens (0x0681)
* Also including 0 just in case the ports are platform serial ports
* which this plugin does need to check support for.
*/
if (vendor != 0x1e2d && vendor != 0x0681 && vendor != 0)
return MM_PLUGIN_SUPPORTS_PORT_UNSUPPORTED;
}
/* First thing to check in this plugin is if we got capabilities already.
* This is because we have a later check of the probed vendor, which is
* taken also during port probing.