[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [Qemu-devel] [PATCH 01/10] hw/sd/sdhci.c: Remove x-drive
From: |
Alistair Francis |
Subject: |
Re: [Qemu-arm] [Qemu-devel] [PATCH 01/10] hw/sd/sdhci.c: Remove x-drive property |
Date: |
Thu, 17 Dec 2015 11:28:18 -0800 |
On Fri, Dec 11, 2015 at 8:37 AM, Peter Maydell <address@hidden> wrote:
> The following commits will remove support for the old sdhci-pci
> command line syntax using the x-drive property:
> -device sdhci-pci,x-drive=mydrive -drive id=mydrive,[...]
> and replace it with an explicit sd device:
> -device sdhci-pci -drive id=mydrive,[...] -device sd,drive=mydrive
>
> (This is OK because x-drive is experimental.)
>
> This commit removes the x-drive property so that old style
> command lines will fail with a reasonable error message:
> -device sdhci-pci,x-drive=mydrive: Property '.x-drive' not found
>
> Signed-off-by: Peter Maydell <address@hidden>
I agree with the way you have split up the patches. I don't think it
is worth maintaining compatibility over the series.
Reviewed-by: Alistair Francis <address@hidden>
Thanks,
Alistair
> ---
> hw/sd/sdhci.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/hw/sd/sdhci.c b/hw/sd/sdhci.c
> index 8612760..991c9b5 100644
> --- a/hw/sd/sdhci.c
> +++ b/hw/sd/sdhci.c
> @@ -1221,12 +1221,6 @@ const VMStateDescription sdhci_vmstate = {
> /* Capabilities registers provide information on supported features of this
> * specific host controller implementation */
> static Property sdhci_pci_properties[] = {
> - /*
> - * We currently fuse controller and card into a single device
> - * model, but we intend to separate them. For that purpose, the
> - * properties that belong to the card are marked as experimental.
> - */
> - DEFINE_PROP_DRIVE("x-drive", SDHCIState, blk),
> DEFINE_PROP_UINT32("capareg", SDHCIState, capareg,
> SDHC_CAPAB_REG_DEFAULT),
> DEFINE_PROP_UINT32("maxcurr", SDHCIState, maxcurr, 0),
> --
> 1.9.1
>
>
[Qemu-arm] [PATCH 08/10] hw/sd/pxa2xx_mmci: Update to use new SDBus APIs, Peter Maydell, 2015/12/11