remove sdhc/mmc-after-suspend from whitelist

Bug #1087173 reported by TienFu Chen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Won't Fix
Low
Unassigned

Bug Description

I'm not sure we should test the card reader for all 3 cards(mmc,sd,sdhc) after suspend.
I think test sd or sdhc after suspend is enough.
My idea is that we just need to make sure the card reader is still alive after suspend.

TienFu Chen (ctf)
description: updated
description: updated
Revision history for this message
Jeff Lane  (bladernr) wrote : Re: [Bug 1087173] [NEW] remove sdhc/mmc-after-suspend from whitelist

I'd +1 this. At the least, I cant imagine how if all three work before,
only one would stop working after suspend. I agree with Tim's assessment.

The before tests test that the device is working and the device
recognizes common card types.
The after test should just test that the device is alive after
suspend/resume, so just testing one (I'd suggest sdhc as it's the most
common) should be enough.

It would be worth checking with some of the CE kernel or other guys to
see if they can think of a situation where one type would work before
but not after, while the others work both before and after suspend.

--
Jeff Lane - Hardware Certification Engineer and Test Tools Developer
Ubuntu Ham: W4KDH
Freenode IRC: bladernr or bladernr_
gpg: 1024D/3A14B2DD 8C88 B076 0DD7 B404 1417 C466 4ABD 3635 3A14 B2DD

Zygmunt Krynicki (zyga)
affects: checkbox → plainbox-provider-checkbox
Zygmunt Krynicki (zyga)
Changed in plainbox-provider-checkbox:
status: New → Triaged
importance: Undecided → Low
Revision history for this message
Jerry Kao (jerry.kao) wrote :

+1

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

I'm marking this as WONT FIX but please reopen if this is actively hampering anyone's work. My goal is to limit the number of open bugs to get a better idea as to what is really important.

Remember that you can always escalate bugs by contacting us in #checkbox on freenode (or #cert-infra in the internal IRC) or by responding in bugs directly.

Having said that, if you want it changed, just patch the whitelist and propose a merge request please.

Changed in plainbox-provider-checkbox:
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.