From 222b10cbc0981904272b180609c74df8510b5c21 Mon Sep 17 00:00:00 2001 From: Siarzhuk Zharski Date: Mon, 19 Aug 2013 21:18:59 +0200 Subject: [PATCH] USB audio: Fix building on x86_64 * fix x86_64 build errors and warnings pointed out by Diver; * fix warning building with gcc2. --- src/add-ons/kernel/drivers/audio/usb/Driver.cpp | 2 +- src/add-ons/kernel/drivers/audio/usb/Stream.cpp | 11 ++++++----- src/add-ons/kernel/drivers/audio/usb/Stream.h | 4 ++-- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/src/add-ons/kernel/drivers/audio/usb/Driver.cpp b/src/add-ons/kernel/drivers/audio/usb/Driver.cpp index d636ac909e..6ea128af1a 100644 --- a/src/add-ons/kernel/drivers/audio/usb/Driver.cpp +++ b/src/add-ons/kernel/drivers/audio/usb/Driver.cpp @@ -254,7 +254,7 @@ publish_devices() MutexLocker driverLock; int32 deviceCount = 0; - for (int32 i = 0; i < MAX_DEVICES; i++) { + for (size_t i = 0; i < MAX_DEVICES; i++) { if (gDevices[i] == NULL) continue; diff --git a/src/add-ons/kernel/drivers/audio/usb/Stream.cpp b/src/add-ons/kernel/drivers/audio/usb/Stream.cpp index 0e5f685383..74933694c3 100644 --- a/src/add-ons/kernel/drivers/audio/usb/Stream.cpp +++ b/src/add-ons/kernel/drivers/audio/usb/Stream.cpp @@ -117,7 +117,7 @@ Stream::_ChooseAlternate() == USB_ENDPOINT_ADDR_DIR_IN; if (fIsInput) - fCurrentBuffer = -1; + fCurrentBuffer = (size_t)-1; TRACE(INF, "Alternate %d EP:%x selected for %s!\n", fActiveAlternate, endpoint->fEndpointAddress, @@ -306,8 +306,8 @@ Stream::_QueueNextTransfer(size_t queuedBuffer, bool start) void -Stream::_TransferCallback(void* cookie, int32 status, void* data, - uint32 actualLength) +Stream::_TransferCallback(void* cookie, status_t status, void* data, + size_t actualLength) { Stream* stream = (Stream*)cookie; atomic_add(&stream->fInsideNotify, 1); @@ -371,10 +371,11 @@ Stream::SetEnabledChannels(uint32& offset, multi_channel_enable* Enable) if (cluster == 0) return B_ERROR; - for (size_t i = 0; i < cluster->ChannelsCount(); i++) + for (size_t i = 0; i < cluster->ChannelsCount(); i++, offset++) { TRACE(INF, "%s channel %d.\n", - (B_TEST_CHANNEL(Enable->enable_bits, offset++) + (B_TEST_CHANNEL(Enable->enable_bits, offset) ? "Enable" : "Disable"), offset + 1); + } return B_OK; } diff --git a/src/add-ons/kernel/drivers/audio/usb/Stream.h b/src/add-ons/kernel/drivers/audio/usb/Stream.h index 0635f70950..b525acbb39 100644 --- a/src/add-ons/kernel/drivers/audio/usb/Stream.h +++ b/src/add-ons/kernel/drivers/audio/usb/Stream.h @@ -64,8 +64,8 @@ private: status_t _ChooseAlternate(); status_t _SetupBuffers(); status_t _QueueNextTransfer(size_t buffer, bool start); - static void _TransferCallback(void* cookie, int32 status, - void* data, uint32 actualLength); + static void _TransferCallback(void* cookie, status_t status, + void* data, size_t actualLength); void _DumpDescriptors(); };