2016-08-01 19:50 GMT+03:00 Viresh Kumar <viresh.kumar@xxxxxxxxxx>: > On 31-07-16, 23:31, Matwey V. Kornilov wrote: >> Hello, >> >> I've also just found that the same commit breaks cpufreq on BeagleBone Black :) >> >> So, probably without HCD_BH flag musb works correctly only at 1Ghz CPU >> frequency, which is unlisted and being set to 720Mhz by cpufreq driver >> (as it did whet there was cpufreq driver). >> >> 2016-07-29 21:01 GMT+03:00 Matwey V. Kornilov <matwey@xxxxxxxxxx>: >> > Hello, >> > >> > I've found that the following commit fixes the issue: >> > >> > commit 7694ca6e1d6f01122f05039b81f70f64b1ec4063 >> > Author: Viresh Kumar <viresh.kumar@xxxxxxxxxx> >> > Date: Fri Apr 22 16:58:42 2016 +0530 >> > >> > cpufreq: omap: Use generic platdev driver >> > >> > The cpufreq-dt-platdev driver supports creation of cpufreq-dt platform >> > device now, reuse that and remove similar code from platform code. > > Sorry for this commit and the man hours wasted to get to this :( > > I am trying to figure out why things break though, as this patch shouldn't have > had any functional impacts. So, some of the assumptions I had are surely > incorrect.. Actually, nothing to sorry about. I suppose, that with this patch my BeagleBone run at 1Ghz after boot, because usually cpufreq limits it to 720Mhz saying [ 14.255646] cpu cpu0: dev_pm_opp_set_rate: failed to find current OPP for freq 1000000000 (-34) And actually musb is still broken after f551e13529833e052f75ec628a8af7 (" Revert "usb: musb: musb_host") > > The defconfig linked in the original thread [1] has this: > > CONFIG_CPUFREQ_DT=m > > So, the cpufreq-dt module needs to get inserted to make it work. > It has been inserted automatically by alias `platform:cpufreq-dt' Issue here that 4.7 stopped to provide `platfrom:cpufreq-dt' on BeagleBone. Have you received my patch fixing it yet? > Can someone provide the output of: > > cat /sys/devices/system/cpu/cpu0/cpufreq/scaling_driver > > with and without this patch ? With this patch, there is no cpufreq directory here. Without this patch, the output is the following: nohostname:~ # uname -a Linux nohostname 4.6.4-3.gecd9058-default #1 SMP PREEMPT Fri Jul 15 08:08:50 UTC 2016 (ecd9058) armv7l armv7l armv7l GNU/Linux nohostname:~ # cat /sys/devices/system/cpu/cpu0/cpufreq/scaling_driver cpufreq-dt > > It looks like we wanted to select "omap-cpufreq" for beaglebone and selected > "cpufreq-dt" by mistake. No, I am not sure, cpufreq-dt worked well until v4.7 where it is disappeared. > > [Note]: I am not subscribed to USB lists and so please include me for any emails > you want my response on. > > -- > viresh > > [1] http://www.spinics.net/lists/linux-usb/msg143956.html > > -- > viresh > -- With best regards, Matwey V. Kornilov. Sternberg Astronomical Institute, Lomonosov Moscow State University, Russia 119991, Moscow, Universitetsky pr-k 13, +7 (495) 9392382 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html