[OpenBIOS] [PATCH 7/9] SPARC64: set up small iomem region for memory-mapped devices

[Mark Cave-Ayland]

On 24/10/16 12:55, BALATON Zoltan wrote:

> On Sun, 23 Oct 2016, Mark Cave-Ayland wrote:
>> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland at ilande.co.uk>
>> ---
>> arch/sparc64/ldscript        |    2 +-
>> arch/sparc64/ofmem_sparc64.c |    4 ++--
>> 2 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/sparc64/ldscript b/arch/sparc64/ldscript
>> index c5cc6a5..a55e02b 100644
>> --- a/arch/sparc64/ldscript
>> +++ b/arch/sparc64/ldscript
>> @@ -61,7 +61,7 @@ SECTIONS
>>     _estack = .;
>>     }
>>
>> -    . = ALIGN(4096);
>> +    . = ALIGN(8192);
>>     _end = .;
>>     _iomem = _end + IOMEM_SIZE;
>>
>> diff --git a/arch/sparc64/ofmem_sparc64.c b/arch/sparc64/ofmem_sparc64.c
>> index 1711c7e..ba4a053 100644
>> --- a/arch/sparc64/ofmem_sparc64.c
>> +++ b/arch/sparc64/ofmem_sparc64.c
>> @@ -68,13 +68,13 @@ ucell ofmem_arch_get_virt_top(void)
>> ucell ofmem_arch_get_iomem_base(void)
>> {
>>     /* Currently unused */
>> -    return 0;
>> +    return (ucell)&_iomem;
>> }
>>
>> ucell ofmem_arch_get_iomem_top(void)
>> {
>>     /* Currently unused */
>> -    return 0;
>> +    return (ucell)&_iomem + 0x8000;
>> }
> 
> Doesn't this make use of these functions now so the "Currently unused"
> comment is also invalidated?

Ooops, yes indeed. I'll update this in my local repository and push to
github later assuming I receive no further comments.


ATB,

Mark.