[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 07/14] hw/arm/bcm2835_peripherals: implement
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [PATCH v8 07/14] hw/arm/bcm2835_peripherals: implement SDHCI Spec. v3 |
Date: |
Tue, 23 Jan 2018 14:45:34 -0800 |
On Mon, Jan 22, 2018 at 7:06 PM, Philippe Mathieu-Daudé <address@hidden> wrote:
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/arm/bcm2835_peripherals.c | 21 +++++++++++++--------
> 1 file changed, 13 insertions(+), 8 deletions(-)
>
> diff --git a/hw/arm/bcm2835_peripherals.c b/hw/arm/bcm2835_peripherals.c
> index 12e0dd11af..0570cc4ad3 100644
> --- a/hw/arm/bcm2835_peripherals.c
> +++ b/hw/arm/bcm2835_peripherals.c
> @@ -254,14 +254,19 @@ static void bcm2835_peripherals_realize(DeviceState
> *dev, Error **errp)
> memory_region_add_subregion(&s->peri_mr, RNG_OFFSET,
> sysbus_mmio_get_region(SYS_BUS_DEVICE(&s->rng), 0));
>
> - /* Extended Mass Media Controller */
> - object_property_set_int(OBJECT(&s->sdhci), BCM2835_SDHC_CAPAREG,
> "capareg",
> - &err);
> - if (err) {
> - error_propagate(errp, err);
> - return;
> - }
> -
> + /* Extended Mass Media Controller
> + *
> + * Compatible with:
> + * - SD Host Controller Specification Version 3.0 Draft 1.0
> + * - SDIO Specification Version 3.0
> + * - MMC Specification Version 4.4
> + *
> + * For the exact details please refer to the Arasan documentation:
> + * SD3.0_Host_AHB_eMMC4.4_Usersguide_ver5.9_jan11_10.pdf ¯\_(ツ)_/¯
I don't think we need the shrugging face in the comments.
Otherwise:
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> + */
> + object_property_set_uint(OBJECT(&s->sdhci), 3, "sd-spec-version", &err);
> + object_property_set_uint(OBJECT(&s->sdhci), BCM2835_SDHC_CAPAREG,
> "capareg",
> + &err);
> object_property_set_bool(OBJECT(&s->sdhci), true, "pending-insert-quirk",
> &err);
> if (err) {
> --
> 2.15.1
>
>
- [Qemu-devel] [PATCH v8 02/14] sdhci: rename the hostctl1 register, (continued)
- [Qemu-devel] [PATCH v8 02/14] sdhci: rename the hostctl1 register, Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 03/14] sdhci: implement the Host Control 2 register (tuning sequence), Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 04/14] sdbus: add trace events, Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 05/14] sdhci: implement UHS-I voltage switch, Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 06/14] sdhci: implement CMD/DAT[] fields in the Present State register, Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 07/14] hw/arm/bcm2835_peripherals: implement SDHCI Spec. v3, Philippe Mathieu-Daudé, 2018/01/22
- Re: [Qemu-devel] [PATCH v8 07/14] hw/arm/bcm2835_peripherals: implement SDHCI Spec. v3,
Alistair Francis <=
- [Qemu-devel] [PATCH v8 08/14] hw/arm/bcm2835_peripherals: change maximum block size to 1kB, Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 09/14] hw/arm/fsl-imx6: implement SDHCI Spec. v3, Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 10/14] hw/arm/xilinx_zynqmp: fix the capabilities/spec version to match the datasheet, Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 11/14] hw/arm/xilinx_zynqmp: enable the UHS-I mode, Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 12/14] sdhci: check Spec v3 capabilities qtest, Philippe Mathieu-Daudé, 2018/01/22
- [Qemu-devel] [PATCH v8 13/14] sdhci: add a check_capab_v3() qtest, Philippe Mathieu-Daudé, 2018/01/22