[PATCH qemu 03/38] aspeed: attach the second SPI controller object to the SoC
Cédric Le Goater
clg at kaod.org
Tue Nov 22 00:14:01 AEDT 2016
On 11/21/2016 12:57 PM, Andrew Jeffery wrote:
> On Mon, 2016-11-21 at 22:12 +1030, Joel Stanley wrote:
>>> 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 *?
>
> According the code it means "keep incrementing a counter until you
> generate a unique object name and successfully register the object".
> AKA the 'i' we already use here, just the API doesn't provide an
> argument to pass it.
yes. see :
https://github.com/openbmc/qemu/blob/master/qom/object.c#L910
C.
More information about the openbmc
mailing list