From: <tr...@ff...> - 2011-01-16 17:01:38
|
Author: adi Date: 2011-01-16 09:01:31 -0800 (Sun, 16 Jan 2011) New Revision: 1948 Modified: trunk/libffado/tests/test-ffado.cpp Log: Fix ListDevices on Juju stack Patch by Stefan Richter. He writes: The fix is straightforward: Let ListDevices continue at the next node ID when the configuration ROM of a lower node could not be fetched or parsed successfully. Modified: trunk/libffado/tests/test-ffado.cpp =================================================================== --- trunk/libffado/tests/test-ffado.cpp 2011-01-14 15:53:13 UTC (rev 1947) +++ trunk/libffado/tests/test-ffado.cpp 2011-01-16 17:01:31 UTC (rev 1948) @@ -208,14 +208,12 @@ printf(" Node id GUID VendorId ModelId Vendor - Model\n"); for (int i = 0; i < service.getNodeCount(); i++) { ConfigRom crom(service, i); - if (!crom.initialize()) - break; - - printf(" %2d 0x%s 0x%08X 0x%08X %s - %s\n", - i, crom.getGuidString().c_str(), - crom.getNodeVendorId(), crom.getModelId(), - crom.getVendorName().c_str(), - crom.getModelName().c_str()); + if (crom.initialize()) + printf(" %2d 0x%s 0x%08X 0x%08X %s - %s\n", + i, crom.getGuidString().c_str(), + crom.getNodeVendorId(), crom.getModelId(), + crom.getVendorName().c_str(), + crom.getModelName().c_str()); } } |