--- a/usbmgmt/usbmgrtest/t_usbmanager_suite/Stub3CC/src/Stub3CC.cpp Thu Jul 15 20:42:20 2010 +0300
+++ b/usbmgmt/usbmgrtest/t_usbmanager_suite/Stub3CC/src/Stub3CC.cpp Tue Aug 31 17:01:47 2010 +0300
@@ -1,5 +1,5 @@
/*
-* Copyright (c) 1997-2009 Nokia Corporation and/or its subsidiary(-ies).
+* Copyright (c) 1997-2010 Nokia Corporation and/or its subsidiary(-ies).
* All rights reserved.
* This component and the accompanying materials are made available
* under the terms of "Eclipse Public License v1.0"
@@ -22,11 +22,10 @@
@file
*/
-#include "Stub3CC.h"
#include <usb_std.h>
#include <es_ini.h>
#include <d32usbc.h>
-#include <usb/usblogger.h>
+#include "Stub3CC.h"
#ifdef __FLOG_ACTIVE
_LIT8(KLogComponent, "STUB3CC");
@@ -34,6 +33,11 @@
#include "usbmaninternalconstants.h"
+#include "OstTraceDefinitions.h"
+#ifdef OST_TRACE_COMPILER_IN_USE
+#include "Stub3CCTraces.h"
+#endif
+
// Panic category
@@ -162,20 +166,22 @@
*/
void CUsbstub3ClassController::Start(TRequestStatus& aStatus)
{
- LOG_FUNC
-
+ OstTraceFunctionEntry0( CUSBSTUB3CLASSCONTROLLER_START_ENTRY );
+
aStatus = KRequestPending;
iReportStatus = &aStatus;
//If we are already started then just complete the request.
if (iState == EUsbServiceStarted)
{
User::RequestComplete(iReportStatus, KErrNone);
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_START_EXIT );
return;
}
if (iFailToStart)
{
User::RequestComplete(iReportStatus, KErrGeneral);
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_START_EXIT_DUP1 );
return;
}
@@ -183,14 +189,18 @@
#ifndef __WINS__
TInt ret = iLdd.Open(0);
- LOGTEXT2(_L8("Open LDD, ret=%d"), ret);
+ OstTrace1( TRACE_NORMAL, CUSBSTUB3CLASSCONTROLLER_START, _L8("Open LDD, ret=%d"), ret );
+
ret = SetUpInterface();
- LOGTEXT2(_L8("SetUpInterface(), ret=%d"), ret);
+ OstTrace1( TRACE_NORMAL, CUSBSTUB3CLASSCONTROLLER_START_DUP1,
+ _L8("SetUpInterface(), ret=%d"), ret );
+
#endif
iTimer.After(iStatus, iStartDelay*1000); //convert from usec to msec
SetActive();
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_START_EXIT_DUP2 );
}
/**
@@ -200,20 +210,22 @@
*/
void CUsbstub3ClassController::Stop(TRequestStatus& aStatus)
{
- LOG_FUNC
-
+ OstTraceFunctionEntry0( CUSBSTUB3CLASSCONTROLLER_STOP_ENTRY );
+
aStatus = KRequestPending;
iReportStatus = &aStatus;
//If we are already idle then just complete the request.
if (iState == EUsbServiceIdle)
{
User::RequestComplete(iReportStatus, KErrNone);
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_STOP_EXIT );
return;
}
if (iFailToStop)
{
User::RequestComplete(iReportStatus, KErrGeneral);
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_STOP_EXIT_DUP1 );
return;
}
@@ -226,6 +238,7 @@
iTimer.After(iStatus, iStopDelay*1000); //convert from usec to msec
SetActive();
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_STOP_EXIT_DUP2 );
}
/**
@@ -235,10 +248,12 @@
*/
void CUsbstub3ClassController::GetDescriptorInfo(TUsbDescriptor& aDescriptorInfo) const
{
- LOG_FUNC
-
+ OstTraceFunctionEntry0( CUSBSTUB3CLASSCONTROLLER_GETDESCRIPTORINFO_ENTRY );
+
aDescriptorInfo.iLength = Kstub3DescriptorLength;
aDescriptorInfo.iNumInterfaces = Kstub3NumberOfInterfacesPerstub3Function;
+
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_GETDESCRIPTORINFO_EXIT );
}
@@ -247,9 +262,14 @@
*/
void CUsbstub3ClassController::RunL()
{
- LOG_FUNC
-
- __ASSERT_DEBUG( iStatus == KErrNone, _USB_PANIC(Kstub3CcPanicCategory, EPanicUnexpectedStatus) );
+ OstTraceFunctionEntry0( CUSBSTUB3CLASSCONTROLLER_RUNL_ENTRY );
+
+ if(iStatus != KErrNone)
+ {
+ OstTrace1( TRACE_FATAL, CUSBSTUB3CLASSCONTROLLER_RUNL,
+ "CUsbstub3ClassController::RunL panic with error %d", EPanicUnexpectedStatus );
+ __ASSERT_DEBUG(EFalse,User::Panic(Kstub3CcPanicCategory,EPanicUnexpectedStatus));
+ }
switch (iState)
{
case EUsbServiceStarting:
@@ -259,10 +279,13 @@
iState = EUsbServiceIdle;
break;
default:
- _USB_PANIC(Kstub3CcPanicCategory, EPanicUnexpectedState);
+ OstTrace1( TRACE_FATAL, CUSBSTUB3CLASSCONTROLLER_RUNL_DUP1,
+ "CUsbstub3ClassController::RunL panic with error %d", EPanicUnexpectedStatus );
+ User::Panic(Kstub3CcPanicCategory,EPanicUnexpectedStatus);
}
*iReportStatus = KErrNone;
User::RequestComplete(iReportStatus, iStatus.Int());
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_RUNL_EXIT );
}
/**
@@ -284,8 +307,10 @@
case EUsbServiceStopping:
iState = EUsbServiceStarted;
break;
- default:
- _USB_PANIC(Kstub3CcPanicCategory, EPanicUnexpectedState);
+ default:
+ OstTrace1( TRACE_FATAL, CUSBSTUB3CLASSCONTROLLER_DOCANCEL,
+ "CUsbstub3ClassController::DoCancel panic with error %d", EPanicUnexpectedStatus );
+ User::Panic(Kstub3CcPanicCategory,EPanicUnexpectedStatus);
}
*iReportStatus = KErrNone;
User::RequestComplete(iReportStatus, KErrCancel);
@@ -300,7 +325,9 @@
*/
TInt CUsbstub3ClassController::RunError(TInt /*aError*/)
{
- __ASSERT_DEBUG( EFalse, _USB_PANIC(Kstub3CcPanicCategory, EUnusedFunction) );
+ OstTrace1( TRACE_FATAL, CUSBSTUB3CLASSCONTROLLER_RUNERROR,
+ "CUsbstub3ClassController::RunError panic with error %d", EUnusedFunction );
+ __ASSERT_DEBUG(EFalse,User::Panic(Kstub3CcPanicCategory,EUnusedFunction));
return KErrNone;
}
@@ -310,25 +337,34 @@
* endpoint and, if found, configuring the interface.
*/
{
- LOG_FUNC
-
+ OstTraceFunctionEntry0( CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_ENTRY );
+
TUsbDeviceCaps dCaps;
TInt ret = iLdd.DeviceCaps(dCaps);
- LOGTEXT(_L8("\tchecking result of DeviceCaps"));
+ OstTrace0( TRACE_NORMAL, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE,
+ _L8("\tchecking result of DeviceCaps"));
+
if ( ret )
{
- LOGTEXT2(_L8("<<CCdcControlInterface::SetUpInterface ret=%d"), ret);
+ OstTrace1( TRACE_ERROR, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP1,
+ _L8("<<CCdcControlInterface::SetUpInterface ret=%d"), ret );
+
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_EXIT );
return ret;
}
const TUint KRequiredNumberOfEndpoints = 1; // in addition to endpoint 0.
const TUint totalEndpoints = static_cast<TUint>(dCaps().iTotalEndpoints);
- LOGTEXT2(_L8("\tiTotalEndpoints = %d"), totalEndpoints);
+ OstTrace1( TRACE_NORMAL, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP2,
+ _L8("\tiTotalEndpoints = %d"), totalEndpoints );
+
if ( totalEndpoints < KRequiredNumberOfEndpoints )
{
- LOGTEXT2(_L8("<<CCdcControlInterface::SetUpInterface ret=%d"),
- KErrGeneral);
+ OstTrace1( TRACE_ERROR, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP3,
+ _L8("<<CCdcControlInterface::SetUpInterface ret=%d"), KErrGeneral );
+
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_EXIT_DUP1 );
return KErrGeneral;
}
@@ -336,10 +372,15 @@
TUsbcEndpointData data[KUsbcMaxEndpoints];
TPtr8 dataptr(reinterpret_cast<TUint8*>(data), sizeof(data), sizeof(data));
ret = iLdd.EndpointCaps(dataptr);
- LOGTEXT(_L8("\tchecking result of EndpointCaps"));
+ OstTrace0( TRACE_NORMAL, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP4,
+ _L8("\tchecking result of EndpointCaps"));
+
if ( ret )
{
- LOGTEXT2(_L8("<<CCdcControlInterface::SetUpInterface ret=%d"), ret);
+ OstTrace1( TRACE_ERROR, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP5,
+ _L8("<<CCdcControlInterface::SetUpInterface ret=%d"), ret );
+
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_EXIT_DUP2 );
return ret;
}
@@ -349,7 +390,13 @@
for ( TUint i = 0 ; i < totalEndpoints ; i++ )
{
const TUsbcEndpointCaps* caps = &data[i].iCaps;
- __ASSERT_DEBUG(caps,_USB_PANIC(Kstub3CcPanicCategory, EPanicUnexpectedStatus));
+ if(!caps)
+ {
+ OstTrace1( TRACE_FATAL, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP6,
+ "CUsbstub3ClassController::SetUpInterface panic with error %d",
+ EPanicUnexpectedStatus );
+ __ASSERT_DEBUG(EFalse,User::Panic(Kstub3CcPanicCategory,EPanicUnexpectedStatus));
+ }
if (data[i].iInUse)
{
@@ -375,11 +422,14 @@
break;
}
}
- LOGTEXT(_L8("\tchecking epFound"));
+ OstTrace0( TRACE_NORMAL, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP7, _L8("\tchecking epFound"));
+
if ( !epFound )
{
- LOGTEXT2(_L8("<<CCdcControlInterface::SetUpInterface ret=%d"),
- KErrGeneral);
+ OstTrace1( TRACE_ERROR, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP8,
+ _L8("<<CCdcControlInterface::SetUpInterface ret=%d"), KErrGeneral);
+
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_EXIT_DUP3 );
return KErrGeneral;
}
@@ -391,11 +441,16 @@
ifc().iClass.iSubClassNum = 0x02; // Table 16- Abstract Control Model
ifc().iClass.iProtocolNum = 0x01; // Table 17- Hayes compatible
- LOGTEXT(_L8("\tabout to call SetInterface"));
+ OstTrace0( TRACE_NORMAL, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP9,
+ _L8("\tabout to call SetInterface"));
+
// Zero effectively indicates that alternate interfaces are not used.
ret = iLdd.SetInterface(0, ifc);
- LOGTEXT2(_L8("<<CCdcControlInterface::SetUpInterface ret=%d"), ret);
+ OstTrace1( TRACE_NORMAL, CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_DUP10,
+ _L8("<<CCdcControlInterface::SetUpInterface ret=%d"), ret );
+
+ OstTraceFunctionExit0( CUSBSTUB3CLASSCONTROLLER_SETUPINTERFACE_EXIT_DUP4 );
return ret;
}