Loading...
--- xnu/xnu-1228/iokit/Kernel/IODeviceTreeSupport.cpp
+++ xnu/xnu-792.17.14/iokit/Kernel/IODeviceTreeSupport.cpp
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 1998-2006 Apple Computer, Inc. All rights reserved.
+ * Copyright (c) 1998-2000 Apple Computer, Inc. All rights reserved.
*
* @APPLE_OSREFERENCE_LICENSE_HEADER_START@
*
@@ -94,7 +94,7 @@
OSObject * obj;
OSDictionary * allInts;
vm_offset_t * dtMap;
- unsigned int propSize;
+ int propSize;
bool intMap;
bool freeDT;
@@ -144,7 +144,7 @@
freeDT = (kSuccess == DTLookupEntry( 0, "/chosen/memory-map", &mapEntry ))
&& (kSuccess == DTGetProperty( mapEntry,
"DeviceTree", (void **) &dtMap, &propSize ))
- && ((2 * sizeof(vm_offset_t)) == propSize);
+ && ((2 * sizeof( vm_offset_t)) == propSize);
parent = MakeReferenceTable( (DTEntry)dtTop, freeDT );
@@ -222,7 +222,7 @@
if( (obj = child->getProperty( "driver,AAPL,MacOS,PowerPC"))) {
if( (0 == (prop = (OSData *)child->getProperty( gIODTTypeKey )))
- || (strncmp("display", (char *)prop->getBytesNoCopy(), sizeof("display"))) ) {
+ || (strcmp( "display", (char *) prop->getBytesNoCopy())) ) {
child->removeProperty( "driver,AAPL,MacOS,PowerPC");
}
}
@@ -308,11 +308,7 @@
{
vm_offset_t virt;
-#if defined (__i386__)
- virt = ml_boot_ptovirt( range[0] );
-#else
virt = ml_static_ptovirt( range[0] );
-#endif
if( virt) {
ml_static_mfree( virt, range[1] );
}
@@ -328,7 +324,7 @@
const OSSymbol *sym;
DTPropertyIterator dtIter;
void *prop;
- unsigned int propSize;
+ int propSize;
char *name;
char location[ 32 ];
bool noLocation = true;
@@ -373,17 +369,17 @@
} else if( nameKey == gIODTUnitKey ) {
// all OF strings are null terminated... except this one
- if( propSize >= (int) sizeof(location))
- propSize = sizeof(location) - 1;
+ if( propSize >= (int) sizeof( location))
+ propSize = sizeof( location) - 1;
strncpy( location, (const char *) prop, propSize );
location[ propSize ] = 0;
regEntry->setLocation( location );
propTable->removeObject( gIODTUnitKey );
noLocation = false;
- } else if(noLocation && (!strncmp(name, "reg", sizeof("reg")))) {
+ } else if( noLocation && (0 == strcmp( name, "reg"))) {
// default location - override later
- snprintf(location, sizeof(location), "%lX", *((UInt32 *) prop));
+ sprintf( location, "%lX", *((UInt32 *) prop) );
regEntry->setLocation( location );
}
}
@@ -462,7 +458,7 @@
assert( ok );
if( ok) {
- snprintf(buf, sizeof(buf), "IOInterruptController%08lX", phandle);
+ sprintf( buf, "IOInterruptController%08lX", phandle);
sym = OSSymbol::withCString( buf );
} else
sym = 0;
@@ -499,7 +495,7 @@
addrCmp = 0;
if( acells) {
data = OSDynamicCast( OSData, regEntry->getProperty( "reg" ));
- if( data && (data->getLength() >= (acells * sizeof(UInt32))))
+ if( data && (data->getLength() >= (acells * sizeof( UInt32))))
addrCmp = (UInt32 *) data->getBytesNoCopy();
}
original_icells = icells;
@@ -520,7 +516,7 @@
// found a controller - don't want to follow cascaded controllers
parent = 0;
*spec = OSData::withBytesNoCopy( (void *) intSpec,
- icells * sizeof(UInt32));
+ icells * sizeof( UInt32));
*controller = IODTInterruptControllerName( regEntry );
ok = (*spec && *controller);
} else if( parent && (data = OSDynamicCast( OSData,
@@ -529,7 +525,7 @@
map = (UInt32 *) data->getBytesNoCopy();
endMap = map + (data->getLength() / sizeof(UInt32));
data = OSDynamicCast( OSData, regEntry->getProperty( "interrupt-map-mask" ));
- if( data && (data->getLength() >= ((acells + icells) * sizeof(UInt32))))
+ if( data && (data->getLength() >= ((acells + icells) * sizeof( UInt32))))
maskCmp = (UInt32 *) data->getBytesNoCopy();
else
maskCmp = 0;
@@ -665,7 +661,7 @@
}
localBits = (UInt32 *) local->getBytesNoCopy();
- localEnd = localBits + (local->getLength() / sizeof(UInt32));
+ localEnd = localBits + (local->getLength() / sizeof( UInt32));
mapped = OSArray::withCapacity( 1 );
controllers = OSArray::withCapacity( 1 );
@@ -680,8 +676,8 @@
break;
}
} else {
- map = OSData::withData( local, mapped->getCount() * sizeof(UInt32),
- sizeof(UInt32));
+ map = OSData::withData( local, mapped->getCount() * sizeof( UInt32),
+ sizeof( UInt32));
controller = gIODTDefaultInterruptController;
controller->retain();
}
@@ -907,7 +903,7 @@
persist.compareFunc = compareFunc;
persist.locationFunc = locationFunc;
- prop = OSData::withBytes( &persist, sizeof(persist));
+ prop = OSData::withBytes( &persist, sizeof( persist));
if( !prop)
return;
@@ -956,7 +952,7 @@
UInt32 *startRange;
UInt32 *endRanges;
bool ok = true;
- SInt32 diff, diff2, endDiff;
+ SInt32 diff, endDiff;
IODTPersistent *persist;
IODTCompareAddressCellFunc compare;
@@ -964,7 +960,7 @@
IODTGetCellCounts( regEntry, &childSizeCells, &childAddressCells );
childCells = childAddressCells + childSizeCells;
- bcopy( cellsIn, cell, sizeof(UInt32) * childCells );
+ bcopy( cellsIn, cell, 4 * childCells );
if( childSizeCells > 1)
*len = IOPhysical32( cellsIn[ childAddressCells ],
cellsIn[ childAddressCells + 1 ] );
@@ -987,7 +983,7 @@
// search
startRange = (UInt32 *) prop->getBytesNoCopy();
range = startRange;
- endRanges = range + (length / sizeof(UInt32));
+ endRanges = range + (length / 4);
prop = (OSData *) regEntry->getProperty( gIODTPersistKey );
if( prop) {
@@ -1000,14 +996,9 @@
range < endRanges;
range += (childCells + addressCells) ) {
- // is cell start within range?
+ // is cell start >= range start?
diff = (*compare)( childAddressCells, cell, range );
-
- bcopy(range, endCell, childAddressCells * sizeof(UInt32));
- endCell[childAddressCells - 1] += range[childCells + addressCells - 1];
- diff2 = (*compare)( childAddressCells, cell, endCell );
-
- if ((diff < 0) || (diff2 >= 0))
+ if( diff < 0)
continue;
ok = (0 == cell[childCells - 1]);
@@ -1040,8 +1031,8 @@
}
// Get the physical start of the range from our parent
- bcopy( range + childAddressCells, cell, sizeof(UInt32) * addressCells );
- bzero( cell + addressCells, sizeof(UInt32) * sizeCells );
+ bcopy( range + childAddressCells, cell, 4 * addressCells );
+ bzero( cell + addressCells, 4 * sizeCells );
} /* else zero length range => pass thru to parent */