Re: Patch for BeagleBone Black boot arguments

[Matt Thomas <matt%3am-software.com@localhost>]

On Jul 30, 2013, at 9:18 PM, Dennis Ferguson 
<dennis.c.ferguson%gmail.com@localhost> wrote:

> 
> On 30 Jul, 2013, at 22:24 , Matt Thomas <matt%3am-software.com@localhost> 
> wrote:
>> I thought about it for a while and realized that wasn't right either.
>> 
>> @@ -562,10 +562,12 @@ initarm(void *arg)
>>      arm32_kernel_vm_init(KERNEL_VM_BASE, ARM_VECTORS_LOW, 0, devmap,
>>          mapallmem_p);
>> 
>> +#ifdef __HAVE_MM_MD_DIRECT_MAPPED_PHYS
>>      /* "bootargs" env variable is passed as 4th argument to kernel */
>> -    if ((uboot_args[3] & 0xe0000000) == 0x80000000) {
>> +    if (uboot_args[3] - 0x80000000 < ram_size) {
>>              strlcpy(bootargs, (char *)uboot_args[3], sizeof(bootargs));
>>      }
>> +#endif
>>      boot_args = bootargs;
>>      parse_mi_bootargs(boot_args);
> 
> That works fine for me, I didn't know how to avoid someone having to chase 
> that
> line down again should the next card with the same processor have more memory
> still but that seems like it should do it.

I have "beagle class" cards with 1GB (pandaboard) and 2GB (omap5432_evm) so
I wanted a better fix. :)