[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 1/7] include/hw/southbridge/piix: Aggregate all PIIX sough
From: |
Bernhard Beschow |
Subject: |
Re: [PATCH v3 1/7] include/hw/southbridge/piix: Aggregate all PIIX soughbridge type names |
Date: |
Wed, 01 Jun 2022 21:34:34 +0000 |
Am 30. Mai 2022 13:19:33 UTC schrieb "Philippe Mathieu-Daudé" <f4bug@amsat.org>:
>On 29/5/22 20:09, Bernhard Beschow wrote:
>> On Sun, May 29, 2022 at 11:05 AM Mark Cave-Ayland
>> <mark.cave-ayland@ilande.co.uk <mailto:mark.cave-ayland@ilande.co.uk>> wrote:
>>
>> On 28/05/2022 20:20, Bernhard Beschow wrote:
>>
>> > TYPE_PIIX3_PCI_DEVICE resides there as already, so add the remaining
>> > ones, too.
>> >
>> > Signed-off-by: Bernhard Beschow <shentey@gmail.com
>> <mailto:shentey@gmail.com>>
>> > Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk
>> <mailto:mark.cave-ayland@ilande.co.uk>>
>> > Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org
>> <mailto:f4bug@amsat.org>>
>> > ---
>> > hw/isa/piix3.c | 3 ---
>> > include/hw/isa/isa.h | 2 --
>> > include/hw/southbridge/piix.h | 4 ++++
>> > 3 files changed, 4 insertions(+), 5 deletions(-)
>
>> One tiny nit here: there's a typo in the subject line which I missed
>> when reviewing v2.
>>
>> s/soughbridge/southbridge/
>>
>>
>> Ack. Will fix in v3.
>
>Can do. Also, "include/" in subject is not helpful.
I'll take care of this in v4.
Thanks,
Bernhard
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH v3 1/7] include/hw/southbridge/piix: Aggregate all PIIX soughbridge type names,
Bernhard Beschow <=