On 12 March 2018 at 12:36, Philippe Mathieu-Daudé <f4...@amsat.org> wrote: > On 03/09/2018 06:03 PM, Peter Maydell wrote: >> Previously we were writing 0x00, 0x01 to the first 2 bytes of data; >> now we will write 0x01, 0x00. Are you sure that's right ? I guess >> it's the difference between claiming 1mA and 256mA. >> (I can't make any sense of the table in the spec so I have no idea.) > > Good catch. I'm not sure which default value we want here, I doubt 256 > mA matches the card used, but the hw tests pass so I'll keep it. > We might change it to a property later.
Do the tests fail if we report 1mA ? If not, then I'd prefer us to keep the current behaviour. (QEMU's implementation obviously has no current draw limits, so reporting the lowest possible value means we won't ever accidentally cause the guest to think it can't do something because the current requirement is too high.) thanks -- PMM