[haiku-commits] haiku: hrev48125 - in src: add-ons/kernel/bus_managers/firewire system/boot/loader/file_systems/tarfs

  • From: pulkomandy@xxxxxxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 28 Oct 2014 08:51:12 +0100 (CET)

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;
 }


Other related posts:

  • » [haiku-commits] haiku: hrev48125 - in src: add-ons/kernel/bus_managers/firewire system/boot/loader/file_systems/tarfs - pulkomandy