Commit Graph

615642 Commits

Author SHA1 Message Date
Sandy Huang
4144e451b8 drm/rockchip: vop: add extern csc for rk3399 win lite
Change-Id: I7a92e64e2bcb9695b23895a6fd79ca5434852d55
Signed-off-by: Sandy Huang <hjc@rock-chips.com>
2019-04-04 15:40:04 +08:00
Finley Xiao
a23f7ce24f MALI: midgard_for_linux: RK: Add support to get soc infomation
Change-Id: I0fcc59bdb978ea85de3c1afb9ae6b368d1fd5e9c
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-04-03 18:29:32 +08:00
Finley Xiao
7c4fc2453a MALI: midgard: RK: Add support to get soc infomation
Change-Id: I9e5ea0458ec08ddfc10d9d4e258713e747946959
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-04-03 18:28:57 +08:00
Finley Xiao
a9f0970c09 ARM: dts: rockchip: rk3288: Add performance configuration for gpu
Change-Id: Iac51c59395c3111d267b50aea69a2704442def1b
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-04-03 18:28:03 +08:00
Finley Xiao
ff106652d9 clk: rockchip: rk3288: Add 420MHz for PLL
Change-Id: Ic722bdf5d467a64cdf093f8bdabb6dab533cd230
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-04-03 18:27:42 +08:00
Hans Yang
6b1d3f74d8 input/touchscreen/rockchip_gslX680: add revert_x setting
Change-Id: I2ce5da8730f70b5b62149c6e8c27ca58e20770e0
Signed-off-by: Hans Yang <yhx@rock-chips.com>
2019-04-03 17:35:52 +08:00
Hans Yang
02b3dddab9 ARM: rockchip_linux_defconfig: enable CONFIG_TOUCHSCREEN_GSLX680_D708
Change-Id: I1484182ca5bd841a557cfa3c3fea4009e5d5c945
Signed-off-by: Hans Yang <yhx@rock-chips.com>
2019-04-03 14:00:46 +08:00
Yifeng Zhao
c76939e892 drivers: rk_nand: fix ota update loader fail issue
BUG:
[   98.697677] Unable to handle kernel paging request at virtual address 0000f9ff
[   98.698359] pgd = ffffff800940e000
[   98.698799] [0000f9ff] *pgd=000000003effe003, *pud=000000003effe003, *pmd=0000000000000000
[   98.699604] Internal error: Oops: 96000005 [#1] PREEMPT SMP
[   98.700116] Modules linked in:
[   98.700430] CPU: 1 PID: 152 Comm: rknand Tainted: G        W       4.4.167 #285
[   98.701085] Hardware name: Rockchip rk3326 S863 7 avb board (DT)
[   98.701630] task: ffffffc03ca71b00 task.stack: ffffffc007e40000
[   98.702192] PC is at write_loader_lba+0x188/0x1e8
[   98.702627] LR is at FtlWrite+0x40/0x70
[   98.702991] pc : [<ffffff8008881770>] lr : [<ffffff8008881810>] pstate: 20400145
[   98.703656] sp : ffffffc007e43c10
[   98.703969] x29: ffffffc007e43c10 x28: 0000000000003000
[   98.704480] x27: ffffffc007c8e280 x26: ffffffbdc0000000
[   98.704990] x25: 0000000000000000 x24: 0000000000000000

Change-Id: I9ddca6e8122f90ccc8103829d7d2b8ef89327ee7
Signed-off-by: Yifeng Zhao <zyf@rock-chips.com>
2019-04-02 20:16:19 +08:00
Alex Zhao
fbc0c1ec5b net: wireless: rockchip_wlan: fix ap6255 tx control timeout issue
Change-Id: I4e4eb4666621286457dfa7be9786337ad22dde57
Signed-off-by: Alex Zhao <zzc@rock-chips.com>
2019-04-02 20:10:47 +08:00
Finley Xiao
756092be7a soc: rockchip: opp_select: Disable rate scaling when fail to get clk
Change-Id: I1d9f8fcb6cda53bca6d7f63603322a80fcc88e7d
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-04-02 20:07:11 +08:00
Zefa Chen
787fdee6e5 dt-bindings: Document add gc8034
Change-Id: Ie602b8317cf98bb26031daa3c4c369d3546682f7
Signed-off-by: Zefa Chen <zefa.chen@rock-chips.com>
2019-04-02 16:31:17 +08:00
Zefa Chen
1bf75bcd61 media: i2c: add gc8034 driver
Change-Id: I6380d1e2629efa60b5474198faf89e322f4aa2dc
Signed-off-by: Zefa Chen <zefa.chen@rock-chips.com>
2019-04-02 16:31:06 +08:00
Nickey Yang
292f718976 ARM: dts: rockchip: change suspend config for rk3288-evb-rk808-linux
Change-Id: Ifcd902841efd94b4a1ef0e6901ff25724729564c
Signed-off-by: Nickey Yang <nickey.yang@rock-chips.com>
2019-04-02 10:29:31 +08:00
Lin Huang
ca811d3159 arm64: dts: rockchip: rk1808-evb: set bt-wake-host default to pull none
gpio4c2 default status is pull up, when it connect to bt module,
that cause this pin default voltage level to 0.8V, it may trigger
irq. So set this pin default status to pull none.

Change-Id: I48f0e22198f200b05c71b4330602e8f638cf997a
Signed-off-by: Lin Huang <hl@rock-chips.com>
2019-04-02 09:20:30 +08:00
Zhen Chen
8237d4a56a Mali: midgard: fix Spectre v1 vulnerability
user_atom.atom_number can be indirectly controlled by user-space,
hence leading to a potential exploitation
of the Spectre variant 1 vulnerability.

This issue was detected with the help of Smatch:

drivers/gpu/arm/midgard/mali_kbase_jd.c:1397 kbase_jd_submit() warn:
potential spectre issue 'jctx->atoms' [r]
katom = &jctx->atoms[user_atom.atom_number];

Fix this by sanitizing user_atom.atom_number
before 'katom = &jctx->atoms[user_atom.atom_number];'.

Notice that given that speculation windows are large, the policy is
to kill the speculation on the first load and not worry if it can be
completed with a dependent load/store [1].

[1] https://marc.info/?l=linux-kernel&m=152449131114778&w=2

Fixes: 5cf27d0b60 ("Mali: midgard: changes to enlarge BASE_JD_ATOM_COUNT to 512, for defect 184210")
Change-Id: If52f30d29a80a06c6693ddadd5947ab9fe8fbc25
Signed-off-by: Zhen Chen <chenzhen@rock-chips.com>
2019-04-01 19:44:04 +08:00
Zefa Chen
f0630e6f87 dt-bindings: Document add gc5025
Change-Id: I66c091aea3c135cc2c03b867b78d22b3e699cefe
Signed-off-by: Zefa Chen <zefa.chen@rock-chips.com>
2019-04-01 17:54:53 +08:00
Zefa Chen
d774ae6da4 media: i2c: add gc5025 driver
Change-Id: I7565fbce1abb3f598c91eebb888eb383e27bd08e
Signed-off-by: Zefa Chen <zefa.chen@rock-chips.com>
2019-04-01 16:38:52 +08:00
Caesar Wang
304ba59de2 arm64: dts: rockchip: change the order for sound card on rk3399pro evb
Change the order in which the sound card is used. The rk809 sound card is
used by default for card0.

This will keep the status on original, and just add the hdmi sound card
for rk3399pro evb board.

Says the cards order as below.
[root@rk3399pro:/]# cat /proc/asound/cards
 0 [rockchiprk809co]: rockchip_rk809- - rockchip,rk809-codec
                      rockchip,rk809-codec
 1 [rockchiphdmi   ]: rockchip_hdmi - rockchip,hdmi
                      rockchip,hdmi

Fixes: 313c342e9a
("arm64: dts: rockchip: add hdmi-sound for rk3399pro evb board")

Change-Id: Ia14f5703f6b79795cd7504fa35a23a550178b82f
Signed-off-by: Caesar Wang <wxt@rock-chips.com>
2019-04-01 16:30:55 +08:00
William Wu
1b2e6aadd9 Revert "usb: dwc_otg_310: resize the depth of TxFIFO number 1 and 2"
This reverts commit 987105c1ff.

According to DWC2 Programmer's Guide, 2.1.1.2 Dedicated TxFIFO Operation,
more space allocated in the transmit IN Endpoint FIFO results in a better
performance on the USB and can hide latencies on the AHB.

So this patch sets the depth of TxFIFO number 1 to 1024 Bytes which is
usually used for primary USB function (e.g MTP).

Test on RK3288 EVB Android P:
Without this patch: MTP over eMMC - Read is only 9.48 MBps
With this patch:    MTP over eMMC - Read is 28.4 MB/s

Side effect of this patch, if the user uses UVC function, it need to
change the depth of TxFIFO number 1 to 512 Bytes (used for UVC control
ep), and change the depth of TxFIFO number 2 to 1024 Bytes (used for
UVC stream ep). The dwc_otg_310 driver has provided module_param to
set the depth of TxFIFO for each number. You can set the depth of TxFIFO
number 1 in cmdline like this:

dwc_otg.dev_tx_fifo_size_1=0x80,dwc_otg.dev_tx_fifo_size_2=0x100

Change-Id: I7cebf112731e43f89994b92729bf2a3f9e54e8dc
Signed-off-by: William Wu <william.wu@rock-chips.com>
2019-04-01 15:45:03 +08:00
Xing Zheng
f91e736a94 arm64: dts: rockchip: rk3308: add the pinctrl of ext_micbias
Change-Id: I7f93652e6cbda20a9c51503c47b9bf2a7354ef0d
Signed-off-by: Xing Zheng <zhengxing@rock-chips.com>
2019-04-01 10:05:29 +08:00
Xing Zheng
53c5b92aa2 dt-bindings: sound: rk3308_codec: add some descriptions for external micbias
Change-Id: If4ef8c5d149390adf207710246bf3c2289d8c95d
Signed-off-by: Xing Zheng <zhengxing@rock-chips.com>
2019-04-01 10:05:29 +08:00
Xing Zheng
14f9b005bc ASoC: rk3308_codec: add support checking external micbias
Change-Id: I644faff116184ee3fb927cff6531f8d4ac497edc
Signed-off-by: Xing Zheng <zhengxing@rock-chips.com>
2019-04-01 10:05:29 +08:00
Xing Zheng
984af25ece ASoC: rk3308_codec: add some delay to wait micbias is stable
To avoid bring some noise dues to unstable micbias.

Change-Id: I0127505f0adeacda5d852ae3cefae2fb5f4ee9d2
Signed-off-by: Xing Zheng <zhengxing@rock-chips.com>
2019-04-01 10:05:29 +08:00
Wyon Bi
4c520eca72 drm/rockchip: cdn-dp: Fix bus format setting
Change-Id: Id54d3dcf00dc1d535783674d3a57309469d279f3
Signed-off-by: Wyon Bi <bivvy.bi@rock-chips.com>
2019-03-29 14:17:53 +08:00
Xiaotan Luo
1d1098b342 arch: rockchip: dts: update PCM format from dsp_b to dsp_a
rockchip_i2s.c driver update the DSP_A & DSP_B config,
so we update the dts config.

Change-Id: Ida013540f263d082e20d65a43c3de932aca9f906
Signed-off-by: Xiaotan Luo <lxt@rock-chips.com>
2019-03-29 11:28:01 +08:00
Xiaotan Luo
acdf3e94ad ASoC: rockchip: i2s: fixed PCM DSP format
DSP_A: PCM delay 1 bit mode,L data MSB after FRM LRC
DSP_B: PCM no delay mode,L data MSB during FRM LRC

Change-Id: I198519c431815de3ca1fc154da78773ca705f0e0
Signed-off-by: Xiaotan Luo <lxt@rock-chips.com>
2019-03-29 11:26:36 +08:00
Elaine Zhang
534369dc33 clk: rockchip: rk3036: add cru regs dump for panic
Add cru regs dump when system panic.
It's just for debug.

Change-Id: Ic60f491d549e030490c14ea78f4857a8cead596d
Signed-off-by: Elaine Zhang <zhangqing@rock-chips.com>
2019-03-29 09:09:21 +08:00
Elaine Zhang
311c96cb28 clk: rockchip: rk3128: add cru regs dump for panic
Add cru regs dump when system panic.
It's just for debug.

Change-Id: I7f9bc78deef60b1fa48bada5b1a6203185ddce48
Signed-off-by: Elaine Zhang <zhangqing@rock-chips.com>
2019-03-29 09:09:21 +08:00
Elaine Zhang
cc07bb20b3 clk: rockchip: rv1108: add cru regs dump for panic
Add cru regs dump when system panic.
It's just for debug.

Change-Id: Ic4efc985892cbcc5e561203fe8e00dba116439e7
Signed-off-by: Elaine Zhang <zhangqing@rock-chips.com>
2019-03-29 09:09:21 +08:00
Elaine Zhang
03aca6529a clk: rockchip: rk3228: add cru regs dump for panic
Add cru regs dump when system panic.
It's just for debug.

Change-Id: Ide2c3e8add083934672f6d22d8182bcfde046783
Signed-off-by: Elaine Zhang <zhangqing@rock-chips.com>
2019-03-29 09:09:21 +08:00
Elaine Zhang
5210dded8f clk: rockchip: rk3288: add cru regs dump for panic
Add cru regs dump when system panic.
It's just for debug.

Change-Id: I86ff4f12ed932431d131d22a307360418e2e9f40
Signed-off-by: Elaine Zhang <zhangqing@rock-chips.com>
2019-03-29 09:09:21 +08:00
Elaine Zhang
ce80bc232d clk: rockchip: rk3368: add cru regs dump for panic
Add cru regs dump when system panic.
It's just for debug.

Change-Id: I3aeeeb7f7b9240c917c18bc2d36b082003dc6370
Signed-off-by: Elaine Zhang <zhangqing@rock-chips.com>
2019-03-29 09:09:21 +08:00
Frank Wang
07ebc201ba ARM: dts: amend rk322x-usb2phy to rk3228-usb2phy
Change-Id: If47746bf2bc1736c5c9e891829c1013c2449e1f4
Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
2019-03-28 19:06:34 +08:00
Frank Wang
35b8b481b8 dt-bindings: phy-rockchip-inno-usb2: amend rk322x to rk3228
Amend rk322x-usb2phy to rk3228-usb2phy, to keep consistent with
Upstream codes.

Change-Id: I8b97856c25ce61e9d8565f6c8653938e7465e4d4
Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
2019-03-28 19:06:34 +08:00
Frank Wang
9691e69344 phy: rockchip-inno-usb2: format rk3228 chip name
Format rk3228 chip name, to keep consistent with Upstream codes.

Change-Id: I3b49b18257485661570086be0898bdcbd016808a
Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
2019-03-28 19:06:34 +08:00
Frank Wang
4ca50547e8 UPSTREAM: phy: rockchip-inno-usb2: add support of usb2-phy for rv1108 SoCs
This adds support usb2-phy for rv1108 SoCs and amend phy Documentation.

Change-Id: Ib1598bf9ec25adefad798a9fe579d38007e91a39
Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
(cherry picked from commit fc938810d9)
2019-03-28 19:06:34 +08:00
Frank Wang
38e24d2123 UPSTREAM: dt-bindings: phy-rockchip-inno-usb2: add otg-mux interrupt
Add otg-mux property to support multiplexed interrupt in otg-port
on some Rockchip SoC (e.g RV1108).

Change-Id: I93a0d38aec9a13a7b5677d34fe87ea7330e0c2d9
Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
(cherry picked from commit 9c1712d5ce)
2019-03-28 19:06:34 +08:00
Frank Wang
cf46645948 UPSTREAM: phy: rockchip-inno-usb2: add support for otg-mux interrupt
The otg-id/otg-bvalid/linestate interrupts are multiplexed together
in otg-port on some Rockchip SoC (e.g RV1108), this patch add support
for it.

Change-Id: If1a29c950fe25a8ca4873fcd3d2d4fcbb504da95
Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
(cherry picked from commit 0983e2abc8)
2019-03-28 19:06:34 +08:00
Frank Wang
66817afa8a UPSTREAM: dt-bindings: phy-rockchip-inno-usb2: add rockchip,usbgrf property
Add rockchip,usbgrf property to support the registers of usb-phy
that are distributed in grf and usbgrf on some special Rockchip
SoCs (e.g RV1108).

Change-Id: Ifcec36474e6828ee90009a47dceca6174c76f45d
Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
(cherry picked from commit c7527e07f0)
2019-03-28 19:06:34 +08:00
Frank Wang
f646d95a5b UPSTREAM: phy: rockchip-inno-usb2: add support for rockchip,usbgrf property
The registers of usb-phy are distributed in grf and usbgrf on some
Rockchip SoCs (e.g RV1108), this patch add a new rockchip,usbgrf
property to support this companion grf design.

Change-Id: If66c03426d4ad63b285fa7132ae20ee10be1d627
Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
(cherry picked from commit 1543645c31)
2019-03-28 19:06:34 +08:00
Zhen Chen
5cf27d0b60 Mali: midgard: changes to enlarge BASE_JD_ATOM_COUNT to 512, for defect 184210
The source code of mali_so must be modified correspondingly.

Change-Id: I3f4bd03fa2d369d912e6bc05c53d2d3abefb92d3
Signed-off-by: Zhen Chen <chenzhen@rock-chips.com>
2019-03-28 18:03:39 +08:00
Hu Kejun
ba2207c20b media: i2c: add imx317 driver
Change-Id: Ic4d272ee4e34aad4fee1c6213ea50ed6ed0f6f7f
Signed-off-by: Hu Kejun <william.hu@rock-chips.com>
2019-03-28 15:06:50 +08:00
Lin Jianhua
5d6cdac108 arm: dts: rockchip: rk3308-dot-rk816: enable uboot charge mode
Change-Id: Ia2262a1855f5f97f224f22a0125adeee28568e09
Signed-off-by: Lin Jianhua <linjh@rock-chips.com>
2019-03-28 15:06:16 +08:00
Matthias Kaehlcke
7d6f699ff3 UPSTREAM: PM / devfreq: Don't adjust to user limits in governors
Several governors use the user space limits df->min/max_freq to adjust
the target frequency. This is not necessary, since update_devfreq()
already takes care of this. Instead the governor can request the available
min/max frequency by setting the target frequency to DEVFREQ_MIN/MAX_FREQ
and let update_devfreq() take care of any adjustments.

Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Brian Norris <briannorris@chromium.org>
Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
(cherry picked from commit 6ff66e2a00)

Conflicts:
	drivers/devfreq/governor.h

Change-Id: Ide444eaf2a0d7c5f20d8521786749e8dbb1b1632
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-03-28 14:53:57 +08:00
Matthias Kaehlcke
638f27a59e UPSTREAM: PM / devfreq: Fix handling of min/max_freq == 0
Commit ab8f58ad72 ("PM / devfreq: Set min/max_freq when adding the
devfreq device") initializes df->min/max_freq with the min/max OPP when
the device is added. Later commit f1d981eaec ("PM / devfreq: Use the
available min/max frequency") adds df->scaling_min/max_freq and the
following to the frequency adjustment code:

  max_freq = MIN(devfreq->scaling_max_freq, devfreq->max_freq);

With the current handling of min/max_freq this is incorrect:

Even though df->max_freq is now initialized to a value != 0 user space
can still set it to 0, in this case max_freq would be 0 instead of
df->scaling_max_freq as intended. In consequence the frequency adjustment
is not performed:

  if (max_freq && freq > max_freq) {
	freq = max_freq;

To fix this set df->min/max freq to the min/max OPP in max/max_freq_store,
when the user passes a value of 0. This also prevents df->max_freq from
being set below the min OPP when df->min_freq is 0, and similar for
min_freq. Since it is now guaranteed that df->min/max_freq can't be 0 the
checks for this case can be removed.

Fixes: f1d981eaec ("PM / devfreq: Use the available min/max frequency")
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Brian Norris <briannorris@chromium.org>
Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
(cherry picked from commit df5cf4a361)

Change-Id: Ib8e2def7d3bf605dba81cf75bab89a23a39ca6f3
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-03-28 14:53:57 +08:00
Bjorn Andersson
f1be83ba38 UPSTREAM: PM / devfreq: Drop custom MIN/MAX macros
Drop the custom MIN/MAX macros in favour of the standard min/max from
kernel.h

Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
(cherry picked from commit d0e464205b)

Change-Id: I15d97eb81276cef6a440449e7db62d866e5b7e26
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-03-28 14:53:57 +08:00
Matthias Kaehlcke
e4bc938072 UPSTREAM: PM / devfreq: Init user limits from OPP limits, not viceversa
Commit ab8f58ad72 ("PM / devfreq: Set min/max_freq when adding
the devfreq device") introduced the initialization of the user
limits min/max_freq from the lowest/highest available OPPs. Later
commit f1d981eaec ("PM / devfreq: Use the available min/max
frequency") added scaling_min/max_freq, which actually represent
the frequencies of the lowest/highest available OPP. scaling_min/
max_freq are initialized with the values from min/max_freq, which
is totally correct in the context, but a bit awkward to read.

Swap the initialization and assign scaling_min/max_freq with the
OPP freqs and then the user limts min/max_freq with scaling_min/
max_freq.

Needless to say that this change is a NOP, intended to improve
readability.

Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
Reviewed-by: Brian Norris <briannorris@chromium.org>
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
(cherry picked from commit 2c2cb1e6b0)

Change-Id: I002271f08a076d89c44305ef8c8d5d06cb9818cf
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-03-28 14:53:57 +08:00
Arvind Yadav
733ad67627 UPSTREAM: PM / devfreq: use put_device() instead of kfree()
Never directly free @dev after calling device_register() or
device_unregister(), even if device_register() returned an error.
Always use put_device() to give up the reference initialized.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
(cherry picked from commit 2d803dc8f7)

Change-Id: Ida1b076e7207854dc8567a3a635e1a638a5789c7
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-03-28 14:53:57 +08:00
Gustavo A. R. Silva
fb22d5a078 UPSTREAM: PM / devfreq: Fix potential NULL pointer dereference in governor_store
df->governor is being dereferenced before it is null checked,
hence there is a potential null pointer dereference.

Notice that df->governor is being null checked at line 1004:
if (df->governor) {, which implies it might be null.

Fix this by null checking df->governor before dereferencing it.

Addresses-Coverity-ID: 1401988 ("Dereference before null check")
Fixes: bcf23c79c4 ("PM / devfreq: Fix available_governor sysfs")
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
(cherry picked from commit 63f1e05f7f)

Change-Id: I9f30f3bb6e1f6b4342c63e505d48103c752b9b28
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-03-28 14:53:57 +08:00
Chanwoo Choi
b19a778913 UPSTREAM: PM / devfreq: Remove unneeded conditional statement
The freq_table array of each devfreq device is always not NULL.
In result, it is unneeded to check whether profile->freq_table
is NULL or not.

Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
(cherry picked from commit ccc4c3bcbb)

Change-Id: I1efaeec81f6b65c920b57e3d0e7051c5d6d7299b
Signed-off-by: Finley Xiao <finley.xiao@rock-chips.com>
2019-03-28 14:53:57 +08:00