Changeset 583 for trunk/libffado/src/genericavc

Timestamp:

08/29/07 02:19:04 (15 years ago)

Author:

ppalmers

Message:

- Clean up the FFADODevice constructor to remove the nodeid that

can be retrieved from the configrom.

- Implement new method of creating devices: a device is supposed

to implement the createDevice static method that should return
an instance of itself or a subclass of itself. This should ease
support for manufacturer specific customizations since we don't
have to put them into the device manager class.

Files:

• trunk/libffado/src/genericavc/avc_avdevice.cpp (modified) (2 diffs)
• trunk/libffado/src/genericavc/avc_avdevice.h (modified) (1 diff)

trunk/libffado/src/genericavc/avc_avdevice.cpp

@@ -56 +56 @@
-};
-
-
-
-
-
+
+
+
-    , m_model( NULL )
-
-{
-    debugOutput( DEBUG_LEVEL_VERBOSE, "Created GenericAVC::AvDevice (NodeID %d)\n",
-nodeId
+configRom->getNodeId()
-    addOption(Util::OptionContainer::Option("snoopMode",false));
-}
@@ -92 +91 @@
-
-    return false;
+}
+
+FFADODevice *
+AvDevice::createDevice( Ieee1394Service& ieee1394Service,
+                        std::auto_ptr<ConfigRom>( configRom ))
+{
+    return new AvDevice(ieee1394Service, configRom );
-}
-

trunk/libffado/src/genericavc/avc_avdevice.h

@@ -47 +47 @@
-class AvDevice : public FFADODevice, public AVC::Unit {
-public:
-
-
-
+
+
-    virtual ~AvDevice();
-
-    static bool probe( ConfigRom& configRom );
-    virtual bool discover();
+    static FFADODevice * createDevice( Ieee1394Service& ieee1394Service,
+                                        std::auto_ptr<ConfigRom>( configRom ));
-    
-    virtual void setVerboseLevel(int l);