[PATCH qemu 03/38] aspeed: attach the second SPI controller object to the SoC

Joel Stanley joel at jms.id.au
Mon Nov 21 22:42:49 AEDT 2016


On Mon, Nov 21, 2016 at 9:33 PM, Andrew Jeffery <andrew at aj.id.au> wrote:
> On Fri, 2016-11-18 at 15:21 +0100, Cédric Le Goater wrote:
>> Signed-off-by: Cédric Le Goater <clg at kaod.org>
>
> Reviewed-by: Andrew Jeffery <andrew at aj.id.au>
>
>> ---
>>  hw/arm/aspeed_soc.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/hw/arm/aspeed_soc.c b/hw/arm/aspeed_soc.c
>> index db145e201451..bbbbfd13862c 100644
>> --- a/hw/arm/aspeed_soc.c
>> +++ b/hw/arm/aspeed_soc.c
>> @@ -124,7 +124,7 @@ static void aspeed_soc_init(Object *obj)
>>      for (i = 0; i < sc->info->spis_num; i++) {
>>          object_initialize(&s->spi[i], sizeof(s->spi[i]),
>>                            sc->info->spi_typename[i]);
>> -        object_property_add_child(obj, "spi", OBJECT(&s->spi[i]), NULL);
>> +        object_property_add_child(obj, "spi[*]", OBJECT(&s->spi[i]), NULL);
>
> Such a curious API. I guess it's effective.

I didn't review this one because wtf is with that *?

Cheers,

Joel


More information about the openbmc mailing list