lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 9 Nov 2017 16:12:59 -0800 From: Bjorn Andersson <bjorn.andersson@...aro.org> To: MyungJoo Ham <myungjoo.ham@...sung.com> Cc: Chanwoo Choi <cw00.choi@...sung.com>, Kyungmin Park <kyungmin.park@...sung.com>, "rafael.j.wysocki@...el.com" <rafael.j.wysocki@...el.com>, "chanwoo@...nel.org" <chanwoo@...nel.org>, Inki Dae <inki.dae@...sung.com>, lkml <linux-kernel@...r.kernel.org>, Linux PM list <linux-pm@...r.kernel.org>, Vinayak Holikatti <vinholikatti@...il.com>, Vivek Gautam <vivek.gautam@...eaurora.org> Subject: Re: [PATCH v5 0/7] PM / devfreq: Use OPP interface to handle the frequency On Thu 09 Nov 15:56 PST 2017, MyungJoo Ham wrote: > > On Sun, Oct 22, 2017 at 6:32 PM, Chanwoo Choi <cw00.choi@...sung.com> wrote: > > > These patches makes the devfreq to use the OPP interface and clean-up codes. > > > > > > > Hi Chanwoo, > > > > This patch series breaks UFS support on modern Qualcomm platforms > > (e.g. MSM8996). > > Could you please show us the code location of it? > The driver is drivers/scsi/ufs/ufshcd.c function ufshcd_probe_hba() calls devm_devfreq_add_device() without there being an opp table associated with hba->dev. This used to return fine and ufshcd_devfreq_target() was called with *freq = UINT_MAX. Unfortunately the driver seems to be designed in such a way that without a call to ufshcd_devfreq_target() with *freq 0 or UINT_MAX some clock is left off, causing my board to reboot; so there's unfortunately a few levels of issues to fix here. Regards, Bjorn
Powered by blists - more mailing lists