[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 04/11] kinetis_k64_sim.h has been added
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v3 04/11] kinetis_k64_sim.h has been added |
Date: |
Mon, 23 Oct 2017 18:42:33 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 10/23/2017 05:36 PM, Gabriel Costa wrote:
> The name SIM came from kinetis datasheet, it is the name of the peripheral.
> I believe others families of kinetis microcontrollers uses the same name.
SIM stands for "System Integration Module"; naming it
"kinetis_k64_sim.c" does not sound very helpful.
This is why I suggested "kinetis_k64_system.c" which better matches the
system controller of the other SoCs.
> On Fri, Oct 20, 2017 at 6:17 PM, Philippe Mathieu-Daudé <address@hidden>
> wrote:
>> I'd rather name it kinetis_k64_system.h or kinetis_k64_sysctl.h