[U-Boot] [PATCH] arch: ifc: update the IFC IP input clock

Scott Wood scott.wood at nxp.com
Fri Sep 9 02:34:47 CEST 2016


On 09/08/2016 07:05 PM, Prabhakar Kushwaha wrote:
> 
>> -----Original Message-----
>> From: york sun
>> Sent: Thursday, September 08, 2016 9:22 PM
>> To: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; u-
>> boot at lists.denx.de; Scott Wood <scott.wood at nxp.com>
>> Subject: Re: [PATCH] arch: ifc: update the IFC IP input clock
>>
>> On 09/08/2016 02:33 AM, Prabhakar Kushwaha wrote:
>>
>> <snip>
>>
>>>>> So better to print IP clock to avoid any confusion.
>>>>> IFC output clock will be printed when it is actually being used during
>>>> synchronous NOR, syn NAND.
>>>>
>>>> I am not against changing it to internal clock. But what are you going
>>>> to print on the console? I think it is confusing to say IFC or local bus
>>>> internal clock speed. Please also check how this clock is used and make
>>>> sure arch.lbc_clk is still correct, after passing to Linux.
>>>>
>>> arch.lbc_clk is only being used for eLBC for device tree fixup.
>>> And I checked the Linux eLBC driver. Looks like it is not using used.
>>>
>>
>> If this clock is not used, can we drop it completely?
>>
> 
> From my point of view Yes. 
> 
> Scott, Please advice

Well, there is that patch from Matt Weber that is trying to guess the
IFC frequency in order to use NWAIT...  Not sure if we'll end up
actually using NWAIT (Prabhakar, can you answer my question of whether
there is a better opcode to use with RNDOUT?) or ever sending a real
RNDOUT, or if we'll ever care about these newer NAND chips on eLBC, but
if U-Boot is currently writing the clock frequency into the device tree
I don't see why we'd rip it out.

-Scott



More information about the U-Boot mailing list