From mboxrd@z Thu Jan  1 00:00:00 1970
From: Alexander Graf <agraf@suse.de>
Date: Fri, 11 Apr 2014 16:38:07 +0200
Subject: [U-Boot] [PATCH v4 6/6] PPC 85xx: Add qemu-ppce500 machine
In-Reply-To: <5329E6AE.3080107@freescale.com>
References: <1392900768-33843-1-git-send-email-agraf@suse.de>	
	<1392900768-33843-7-git-send-email-agraf@suse.de>
	<1392911955.6733.825.camel@snotra.buserror.net>
	<5329E6AE.3080107@freescale.com>
Message-ID: <5347FE4F.2060607@suse.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de


On 19.03.14 19:49, York Sun wrote:
> On 02/20/2014 07:59 AM, Scott Wood wrote:
>> On Thu, 2014-02-20 at 13:52 +0100, Alexander Graf wrote:
>>> diff --git a/arch/powerpc/cpu/mpc85xx/cpu.c b/arch/powerpc/cpu/mpc85xx/cpu.c
>>> index 3e99b07..51f2085 100644
>>> --- a/arch/powerpc/cpu/mpc85xx/cpu.c
>>> +++ b/arch/powerpc/cpu/mpc85xx/cpu.c
>>> @@ -272,6 +272,7 @@ int do_reset (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>>>   #ifndef CONFIG_SYS_FSL_TBCLK_DIV
>>>   #define CONFIG_SYS_FSL_TBCLK_DIV 8
>>>   #endif
>>> +__attribute__((weak))
>>>   unsigned long get_tbclk (void)
>>>   {
>>>   	unsigned long tbclk_div = CONFIG_SYS_FSL_TBCLK_DIV;
>> It's not a big deal, but we do have __weak available.
>>
>>> +static void map_fdt_as(int esel)
>>> +{
>>> +	u32 mas0, mas1, mas2, mas3, mas7;
>>> +	uint64_t fdt_phys = get_fdt_phys();
>>> +	unsigned long fdt_phys_tlb = fdt_phys & ~0xffffful;
>>> +	unsigned long fdt_virt_tlb = (ulong)get_fdt_virt() & ~0xffffful;
>>> +
>>> +	mas0 = MAS0_TLBSEL(1) | MAS0_ESEL(10);
>> Shouldn't you use the esel parameter here?
>>
> Are you going to address this comment with a new version?

Yes :).


Alex