Re: [PATCH V6 3/5] target-i386: add rtc 0x70 port as coalesced_pio

[<peng.hao2@zte.com.cn>]

>On 18/10/2018 10:03, peng.hao2@zte.com.cn wrote:
>>>> +    /* register rtc 0x70 port as coalesced_pio */
>>>> +    memory_region_set_flush_coalesced(&s->io);
>>>> +    memory_region_init_io(&s->coalesced_io, OBJECT(s), &cmos_ops,
>>>> +                          s, "rtc1", 1);
>>>> +    isa_register_ioport(isadev, &s->coalesced_io, base);
>>> I think instead of isa_register_ioport you can use
>>> memory_region_add_subregion, so that s->coalesced_io is added below s->io?
>>>
>> isa_register_ioport also called memory_region_add_subregion.
>> modify code  like this:
>>   //isa_register_ioport(isadev, &s->coalesced_io, base);
>>     memory_region_add_subregion(get_system_io(),0x70,&s->coalesced_io);
>> The regional distribution before and after code modification is consistent.
>
>Right, but I'd rather add s->coalesced_io subregion as a subregion of
>s->io at offset 0.
Ok， I will modify it and resubmit the patch.
By the way , I find that  when I adjusted the format of the patch "PATCH V6 1/5", I made a mistake to modify a code character. I also resubmit the patch.
Thanks.
>
>Paolo
>
>>    0000000000000070-0000000000000070 (prio 0, i/o): rtc1
>>     0000000000000070-0000000000000071 (prio 0, i/o): rtc