Messages in this thread | | | Date | Mon, 8 Dec 2014 18:14:19 +0200 | Subject | Re: [PATCH] mfd: dln2: add suspend/resume functionality | From | Octavian Purdila <> |
| |
On Mon, Dec 8, 2014 at 12:57 PM, Johan Hovold <johan@kernel.org> wrote: > On Fri, Dec 05, 2014 at 03:05:45PM +0200, Octavian Purdila wrote: >> On Fri, Dec 5, 2014 at 2:06 PM, Johan Hovold <johan@kernel.org> wrote: >> > On Fri, Dec 05, 2014 at 01:51:17PM +0200, Octavian Purdila wrote: >> >> On Fri, Dec 5, 2014 at 12:17 PM, Johan Hovold <johan@kernel.org> wrote: > >> >> > And surely you need to resubmit the read urbs in resume, or you will >> >> > never receive any more data. >> >> > >> >> > How did you test this patch? >> >> > >> >> >> >> The resume cb is not called in my setup (kvm), only reset_resume. >> > >> > Please make sure to test your patches on proper hardware. >> >> It was tested with the proper hardware, I was just testing it in the >> setup we use, which is kvm with USB pass through. > > I was referring to your kvm setup. >
I see the same behavior (reset_resume called) when testing on bare bone hardware. BTW, KVM allows me to test more thoroughly, as I can use custom SSDT tables required by i2c devices to find their IRQs.
| |