[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/10] hw/sd: Add QOM bus which SD cards plug in
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 04/10] hw/sd: Add QOM bus which SD cards plug in to |
Date: |
Sun, 20 Dec 2015 23:18:19 +0000 |
On 20 December 2015 at 20:51, Peter Crosthwaite
<address@hidden> wrote:
> On Sun, Dec 20, 2015 at 9:10 AM, Peter Maydell <address@hidden> wrote:
>> On 19 December 2015 at 21:38, Peter Crosthwaite
>> <address@hidden> wrote:
>>> On Fri, Dec 11, 2015 at 04:37:05PM +0000, Peter Maydell wrote:
>>>> +bool sdbus_get_inserted(SDBus *sdbus)
>>>> +{
>>>> + SDState *card = get_card(sdbus);
>>>> +
>>>> + if (card) {
>>>> + SDClass *sc = SD_GET_CLASS(card);
>>>> +
>>>> + return sc->get_inserted(card);
>>>> + }
>>>> +
>>>> + return false;
>>>> +}
>>>
>>> This I am not sure about. Realistically, the card has no self
>>> awareness of its ejection state so it can't be polled for "are
>>> you there". The card insertion switch is implemented as a
>>> physical switch on the slot itself and a property of the bus.
>>>
>>> The absence on presence of a device should determine this, making me
>>> think this should return !!card.
>>>
>>> Unfortunately, we have the case of the block UI being able to trigger a
>>> card ejection from underneath the bus level. But since the SD card is
>>> already
>>> using qdev_get_parent_bus() the removal from the bus can be managed at the
>>> card level.
>>
>> For user-level back compat I think we need to retain "might have
>> an sdcard object with no block backend, and that means
>> 'no-card-present'". This is both for the user facing
>> monitor commands to manipulate the sd card, and also
>
> What are the user-facing monitor commands? I tried using "change" and
> "eject", but they don't seem to work for SD, due to the tray being
> closed. Has this ever worked in a way that is user manipulatable for
> SD or is it just to handle the case of unconditional SD card creation
> (with the card never hotplugging over the system lifetime)?
I admit to just assuming that this stuff worked rather than
testing it :-)
thanks
-- PMM
- Re: [Qemu-devel] [PATCH 02/10] hw/sd/sd.c: QOMify, (continued)
[Qemu-devel] [PATCH 06/10] sdhci_sysbus: Create SD card device in users, not the device itself, Peter Maydell, 2015/12/11
[Qemu-devel] [PATCH 04/10] hw/sd: Add QOM bus which SD cards plug in to, Peter Maydell, 2015/12/11
[Qemu-devel] [PATCH 08/10] hw/sd/pxa2xx_mmci: Update to use new SDBus APIs, Peter Maydell, 2015/12/11
[Qemu-devel] [PATCH 03/10] hw/sd/sd.c: Convert sd_reset() function into Device reset method, Peter Maydell, 2015/12/11
[Qemu-devel] [PATCH 05/10] hw/sd/sdhci.c: Update to use SDBus APIs, Peter Maydell, 2015/12/11