Re: sensor-hub: Move the memset to sensor_hub_get_feature()

[Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>]

On Fri, 2016-12-30 at 16:17 +0530, Ritesh Raj Sarraf wrote:
> Hello Srini,
> 
> On Thu, 2016-12-29 at 23:01 +0530, Ritesh Raj Sarraf wrote:
> > 
> > On Thu, 2016-12-29 at 09:22 -0800, Srinivas Pandruvada wrote:
> > > 
> > > > 
> > > > If you need more information, please tell me.
> > > I tried on couple of systems with the patch. It seems to fix the
> > > issue.
> > > It is possible some low level power related changes have impacted
> > > this.
> > > Can you do bisect? I suggest just try 4.9-rc1. I think it will
> > > not work
> > > there probably.
> > Okay. Let me do a bisect on it. I'll ping back on this thread soon.
> Since yesterday's attempts, I haven't been able to reproduce the bug
> again, so
> far. Apart from re-building the kernel, I have also blacklisted the
> particular
> USB device from power savings (based on your hint). I don't know if
> disabling
> the power savings for that USB device is the cause for not seeing the
> issue, but
> so far, the device seems to report data.
> 
> Do you think power savings mode can impact these devices ? I know of
> USB plugged
> keyboard/mice to generally break, when enabling power savings mode.
> But I had
> hoped that in-built USB devices were immune to it.
> 
This is possible. If your remove from blacklist does it still work? May
be rebuild of kernel fixed this:-) 

Thanks,
Srinivas

> Thanks,
> Ritesh
>