hrev48125 adds 2 changesets to branch 'master' old head: 8c668f744f00c46b791e83efea7873e74a549b7c new head: 438d564c08e497713fe5c396b920c341ef7e82d2 overview: http://cgit.haiku-os.org/haiku/log/?qt=range&q=438d564+%5E8c668f7 ---------------------------------------------------------------------------- 6879e9d: Tarfs: fix traces 438d564: Firewire: fix more warnings. [ PulkoMandy <pulkomandy@xxxxxxxxxxxxx> ] ---------------------------------------------------------------------------- 4 files changed, 53 insertions(+), 52 deletions(-) .../kernel/bus_managers/firewire/firewire.cpp | 2 +- .../kernel/bus_managers/firewire/fwohcireg.h | 84 ++++++++++---------- src/add-ons/kernel/bus_managers/firewire/util.c | 12 +-- .../boot/loader/file_systems/tarfs/tarfs.cpp | 7 +- ############################################################################ Commit: 6879e9df771aedcbb13b61e542076777fe1edc3f URL: http://cgit.haiku-os.org/haiku/commit/?id=6879e9d Author: PulkoMandy <pulkomandy@xxxxxxxxxxxxx> Date: Tue Oct 28 07:49:05 2014 UTC Tarfs: fix traces ---------------------------------------------------------------------------- diff --git a/src/system/boot/loader/file_systems/tarfs/tarfs.cpp b/src/system/boot/loader/file_systems/tarfs/tarfs.cpp index 26af67a..f5c19f7 100644 --- a/src/system/boot/loader/file_systems/tarfs/tarfs.cpp +++ b/src/system/boot/loader/file_systems/tarfs/tarfs.cpp @@ -16,6 +16,7 @@ #include <AutoDeleter.h> #include <OS.h> +#include <SupportDefs.h> #include <zlib.h> @@ -272,8 +273,8 @@ TarFS::File::~File() ssize_t TarFS::File::ReadAt(void* cookie, off_t pos, void* buffer, size_t bufferSize) { - TRACE(("tarfs: read at %Ld, %lu bytes, fSize = %Ld\n", pos, bufferSize, - fSize)); + TRACE(("tarfs: read at %" B_PRIdOFF ", %" B_PRIuSIZE " bytes, fSize = %" + B_PRIdOFF "\n", pos, bufferSize, fSize)); if (pos < 0 || !buffer) return B_BAD_VALUE; @@ -703,7 +704,7 @@ TarFS::Volume::Init(boot::Partition* partition) off_t size = strtol(header->size, NULL, 8); - TRACE(("tarfs: \"%s\", %Ld bytes\n", header->name, size)); + TRACE(("tarfs: \"%s\", %" B_PRIdOFF " bytes\n", header->name, size)); // TODO: this is old-style GNU tar which probably won't work with newer // ones... ############################################################################ Revision: hrev48125 Commit: 438d564c08e497713fe5c396b920c341ef7e82d2 URL: http://cgit.haiku-os.org/haiku/commit/?id=438d564 Author: PulkoMandy <pulkomandy@xxxxxxxxxxxxx> Date: Tue Oct 28 07:49:18 2014 UTC Firewire: fix more warnings. ---------------------------------------------------------------------------- diff --git a/src/add-ons/kernel/bus_managers/firewire/firewire.cpp b/src/add-ons/kernel/bus_managers/firewire/firewire.cpp index 8b96cf5..727f456 100644 --- a/src/add-ons/kernel/bus_managers/firewire/firewire.cpp +++ b/src/add-ons/kernel/bus_managers/firewire/firewire.cpp @@ -2102,7 +2102,7 @@ fw_rcv(struct fw_rcv_buf *rb) if(bind == NULL){ device_printf(rb->fc->bdev, "%s: " "Unknown service addr 0x%04x:0x%08x %s(%x)" - " src=0x%x data=%lx\n", + " src=0x%x data=%" B_PRIu32 "\n", __func__, fp->mode.wreqq.dest_hi, fp->mode.wreqq.dest_lo, diff --git a/src/add-ons/kernel/bus_managers/firewire/fwohcireg.h b/src/add-ons/kernel/bus_managers/firewire/fwohcireg.h index bcc40a7..f522c37 100644 --- a/src/add-ons/kernel/bus_managers/firewire/fwohcireg.h +++ b/src/add-ons/kernel/bus_managers/firewire/fwohcireg.h @@ -36,48 +36,48 @@ */ #define PCI_CBMEM PCIR_BAR(0) -#define FW_VENDORID_NATSEMI 0x100B -#define FW_VENDORID_NEC 0x1033 -#define FW_VENDORID_SIS 0x1039 -#define FW_VENDORID_TI 0x104c -#define FW_VENDORID_SONY 0x104d -#define FW_VENDORID_VIA 0x1106 -#define FW_VENDORID_RICOH 0x1180 -#define FW_VENDORID_APPLE 0x106b -#define FW_VENDORID_LUCENT 0x11c1 -#define FW_VENDORID_INTEL 0x8086 -#define FW_VENDORID_ADAPTEC 0x9004 -#define FW_VENDORID_SUN 0x108e - -#define FW_DEVICE_CS4210 (0x000f << 16) -#define FW_DEVICE_UPD861 (0x0063 << 16) -#define FW_DEVICE_UPD871 (0x00ce << 16) -#define FW_DEVICE_UPD72870 (0x00cd << 16) -#define FW_DEVICE_UPD72873 (0x00e7 << 16) -#define FW_DEVICE_UPD72874 (0x00f2 << 16) -#define FW_DEVICE_TITSB22 (0x8009U << 16) -#define FW_DEVICE_TITSB23 (0x8019U << 16) -#define FW_DEVICE_TITSB26 (0x8020U << 16) -#define FW_DEVICE_TITSB43 (0x8021U << 16) -#define FW_DEVICE_TITSB43A (0x8023U << 16) -#define FW_DEVICE_TITSB43AB23 (0x8024U << 16) -#define FW_DEVICE_TITSB82AA2 (0x8025U << 16) -#define FW_DEVICE_TITSB43AB21 (0x8026U << 16) -#define FW_DEVICE_TIPCI4410A (0x8017U << 16) -#define FW_DEVICE_TIPCI4450 (0x8011U << 16) -#define FW_DEVICE_TIPCI4451 (0x8027U << 16) -#define FW_DEVICE_CXD1947 (0x8009U << 16) -#define FW_DEVICE_CXD3222 (0x8039U << 16) -#define FW_DEVICE_VT6306 (0x3044 << 16) -#define FW_DEVICE_R5C551 (0x0551 << 16) -#define FW_DEVICE_R5C552 (0x0552 << 16) -#define FW_DEVICE_PANGEA (0x0030 << 16) -#define FW_DEVICE_UNINORTH (0x0031 << 16) -#define FW_DEVICE_AIC5800 (0x5800 << 16) -#define FW_DEVICE_FW322 (0x5811 << 16) -#define FW_DEVICE_7007 (0x7007 << 16) -#define FW_DEVICE_82372FB (0x7605 << 16) -#define FW_DEVICE_PCIO2FW (0x1102 << 16) +#define FW_VENDORID_NATSEMI 0x100Bu +#define FW_VENDORID_NEC 0x1033u +#define FW_VENDORID_SIS 0x1039u +#define FW_VENDORID_TI 0x104cu +#define FW_VENDORID_SONY 0x104du +#define FW_VENDORID_VIA 0x1106u +#define FW_VENDORID_RICOH 0x1180u +#define FW_VENDORID_APPLE 0x106bu +#define FW_VENDORID_LUCENT 0x11c1u +#define FW_VENDORID_INTEL 0x8086u +#define FW_VENDORID_ADAPTEC 0x9004u +#define FW_VENDORID_SUN 0x108eu + +#define FW_DEVICE_CS4210 (0x000fu << 16) +#define FW_DEVICE_UPD861 (0x0063u << 16) +#define FW_DEVICE_UPD871 (0x00ceu << 16) +#define FW_DEVICE_UPD72870 (0x00cdu << 16) +#define FW_DEVICE_UPD72873 (0x00e7u << 16) +#define FW_DEVICE_UPD72874 (0x00f2u << 16) +#define FW_DEVICE_TITSB22 (0x8009u << 16) +#define FW_DEVICE_TITSB23 (0x8019u << 16) +#define FW_DEVICE_TITSB26 (0x8020u << 16) +#define FW_DEVICE_TITSB43 (0x8021u << 16) +#define FW_DEVICE_TITSB43A (0x8023u << 16) +#define FW_DEVICE_TITSB43AB23 (0x8024u << 16) +#define FW_DEVICE_TITSB82AA2 (0x8025u << 16) +#define FW_DEVICE_TITSB43AB21 (0x8026u << 16) +#define FW_DEVICE_TIPCI4410A (0x8017u << 16) +#define FW_DEVICE_TIPCI4450 (0x8011u << 16) +#define FW_DEVICE_TIPCI4451 (0x8027u << 16) +#define FW_DEVICE_CXD1947 (0x8009u << 16) +#define FW_DEVICE_CXD3222 (0x8039u << 16) +#define FW_DEVICE_VT6306 (0x3044u << 16) +#define FW_DEVICE_R5C551 (0x0551u << 16) +#define FW_DEVICE_R5C552 (0x0552u << 16) +#define FW_DEVICE_PANGEA (0x0030u << 16) +#define FW_DEVICE_UNINORTH (0x0031u << 16) +#define FW_DEVICE_AIC5800 (0x5800u << 16) +#define FW_DEVICE_FW322 (0x5811u << 16) +#define FW_DEVICE_7007 (0x7007u << 16) +#define FW_DEVICE_82372FB (0x7605u << 16) +#define FW_DEVICE_PCIO2FW (0x1102u << 16) #define PCI_INTERFACE_OHCI 0x10 diff --git a/src/add-ons/kernel/bus_managers/firewire/util.c b/src/add-ons/kernel/bus_managers/firewire/util.c index 34fd895..66e3d19 100644 --- a/src/add-ons/kernel/bus_managers/firewire/util.c +++ b/src/add-ons/kernel/bus_managers/firewire/util.c @@ -65,7 +65,7 @@ alloc_mem(void **virt, void **phy, size_t size, uint32 protection, *virt = virtadr; if (phy) *phy = (void*)(addr_t)pe.address; - TRACE("area = %ld, size = %ld, virt = %p, phy = %" B_PRIxPHYSADDR "\n", area, size, virtadr, + TRACE("area = %" B_PRId32 ", size = %ld, virt = %p, phy = %" B_PRIxPHYSADDR "\n", area, size, virtadr, pe.address); return area; } @@ -83,22 +83,22 @@ map_mem(void **virt, void *phy, size_t size, uint32 protection, TRACE("mapping physical address %p with %ld bytes for %s\n", phy, size, name); - offset = (uint32)phy & (B_PAGE_SIZE - 1); + offset = (intptr_t)phy & (B_PAGE_SIZE - 1); phyadr = (char *)phy - offset; size = round_to_pagesize(size + offset); area = map_physical_memory(name, (addr_t)phyadr, size, B_ANY_KERNEL_BLOCK_ADDRESS, protection, &mapadr); if (area < B_OK) { - ERROR("mapping '%s' failed, error 0x%lx (%s)\n", name, area, + ERROR("mapping '%s' failed, error 0x%" B_PRIx32 " (%s)\n", name, area, strerror(area)); return area; } *virt = (char *)mapadr + offset; - TRACE("physical = %p, virtual = %p, offset = %ld, phyadr = %p, mapadr = " - "%p, size = %ld, area = 0x%08lx\n", phy, *virt, offset, phyadr, mapadr, - size, area); + TRACE("physical = %p, virtual = %p, offset = %" B_PRId32 ", phyadr = %p, " + "mapadr = %p, size = %ld, area = 0x%08" B_PRIx32 "\n", phy, *virt, + offset, phyadr, mapadr, size, area); return area; }