[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machi
From: |
Andrew Jones |
Subject: |
Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types |
Date: |
Wed, 15 Jul 2020 15:05:30 +0200 |
On Wed, Jul 15, 2020 at 02:26:19PM +0200, Laszlo Ersek wrote:
> Hi Drew,
>
> On 07/15/20 08:36, Andrew Jones wrote:
>
> > So it sounds to me like we have some flexibility in our versioned machine
> > maintenance. We can choose to forgo the usual compat code when the risk is
> > deemed low enough. And, if somebody screams, we can always fix it later.
> > I can live with that. I'll go ahead and respin without the versioning.
>
> In that case, please don't simply remove the acpi_dsdt_add_flash() call
> from build_dsdt(), because then "git blame" won't be able to help later.
> Can you please replace the call with a comment instead, similar to the
> RTC comment from commit 67736a25f865 ("ARM: virt: Don't generate RTC
> ACPI device when using UEFI", 2016-01-15)?
>
In the end I won't be respinning, as this patch is already merged. And,
unless Igor twists my arm, then I don't plan to write another patch
that removes the compat code. If I did remove it, I'd put a comment
in there for git-blame to find. And, in the comment I'd write "Igor
said to remove this", because git-blame would otherwise blame me :-)
Thanks,
drew
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, (continued)
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, Igor Mammedov, 2020/07/13
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, Andrew Jones, 2020/07/14
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, Michael S. Tsirkin, 2020/07/14
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, Andrew Jones, 2020/07/14
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, Michael S. Tsirkin, 2020/07/14
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, Igor Mammedov, 2020/07/14
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, Andrew Jones, 2020/07/15
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, Andrew Jones, 2020/07/15
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types, Laszlo Ersek, 2020/07/15
- Re: [PATCH 3/4] hw/arm/virt-acpi-build: Only expose flash on older machine types,
Andrew Jones <=