[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/4] sdcard: Update the SDState documentation
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/4] sdcard: Update the SDState documentation |
Date: |
Wed, 9 May 2018 08:42:15 -0700 |
On Tue, May 8, 2018 at 11:01 PM, Philippe Mathieu-Daudé <address@hidden> wrote:
> Add more descriptive comments to keep a clear separation
> between static property vs runtime changeable.
>
> Suggested-by: Peter Maydell <address@hidden>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/sd/sd.c | 14 +++++++++-----
> 1 file changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/hw/sd/sd.c b/hw/sd/sd.c
> index 235e0518d6..5fb4787671 100644
> --- a/hw/sd/sd.c
> +++ b/hw/sd/sd.c
> @@ -90,12 +90,15 @@ struct SDState {
> uint32_t card_status;
> uint8_t sd_status[64];
>
> - /* Configurable properties */
> + /* Static properties */
> +
> BlockBackend *blk;
> bool spi;
>
> - uint32_t mode; /* current card mode, one of SDCardModes */
> - int32_t state; /* current card state, one of SDCardStates */
> + /* Runtime changeables */
> +
> + uint32_t mode; /** current card mode, one of #SDCardModes */
> + int32_t state; /** current card state, one of #SDCardStates */
> uint32_t vhs;
> bool wp_switch;
> unsigned long *wp_groups;
> @@ -109,8 +112,9 @@ struct SDState {
> uint32_t pwd_len;
> uint8_t function_group[6];
> uint8_t current_cmd;
> - /* True if we will handle the next command as an ACMD. Note that this
> does
> - * *not* track the APP_CMD status bit!
> + /**
> + * #True if we will handle the next command as an ACMD.
Why do we need a # here?
Otherwise:
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> + * Note that this does *not* track the APP_CMD status bit!
> */
> bool expecting_acmd;
> uint32_t blk_written;
> --
> 2.17.0
>
>
[Qemu-devel] [PATCH v2 3/4] sdcard: Implement the UHS-I SWITCH_FUNCTION entries (Spec v3), Philippe Mathieu-Daudé, 2018/05/09
[Qemu-devel] [PATCH v2 4/4] sdcard: Add a 'uhs' property, update the OCR register ACCEPT_SWITCH_1V8 bit, Philippe Mathieu-Daudé, 2018/05/09