On 12/15/2017 07:43 AM, Alexey Kardashevskiy wrote:
> On 15/12/17 17:17, Cédric Le Goater wrote:
>> Signed-off-by: Cédric Le Goater <c...@kaod.org>
>> ---
>> hw/ppc/pnv.c | 84
>> ++++++++++++++++++++++++++--------------------------
>> hw/ppc/pnv_core.c | 8 ++---
>> hw/ppc/pnv_lpc.c | 2 +-
>> include/hw/ppc/pnv.h | 8 ++---
>> 4 files changed, 51 insertions(+), 51 deletions(-)
>>
>> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
>> index c35c439d816b..4bc5f61d0da7 100644
>> --- a/hw/ppc/pnv.c
>> +++ b/hw/ppc/pnv.c
>> @@ -77,7 +77,7 @@ static const char *pnv_chip_core_typename(const PnvChip *o)
>> * that has a different "affinity". In practice, it means one range
>> * per chip.
>> */
>> -static void powernv_populate_memory_node(void *fdt, int chip_id, hwaddr
>> start,
>> +static void pnv_populate_memory_node(void *fdt, int chip_id, hwaddr start,
>> hwaddr size)
>
> You may also want to fix indentations like the line above.
yes. I missed that. I should run indent after using sed.
While I am at it, I can also change the name of the routines
populating the device tree.
David,
are you following more or less this pattern :
<machine>_dt_<device>
Thanks,
C.
