Currently DPM is listening for all netlink events
and there is battery drain because of that. Since,
DPM is interested only in BIND and LISTEN events,
changes are to process only BIND and LSTEN events
and skip the rest for the sockev client.
CRs-Fixed: 945890
Change-Id: Iae11027945b981538f9c16ae9d5cd1ecf88a3743
Signed-off-by: Susheel Yadagiri <syadagir@codeaurora.org>
Signed-off-by: Ravinder Konka <rkonka@codeaurora.org>
MSM_SYSMON_GLINK_COMM is currently enabled for all targets which breaks
few existing targets, which do not support GLINK protocol and therefore
are still dependent on MSM_SYSMON_COMM support. Enable back sysmon
communication over SMD interface to preserve backward compatibility
until all targets support GLINK.
Change-Id: I06b6c8cd9c079494998463bbf30e27ad224d7124
Signed-off-by: Avaneesh Kumar Dwivedi <akdwived@codeaurora.org>
Add support to enable/disable fpsimd_settings at runtime through
kernel command line.
fpsimd.fpsimd_settings=0 should disable it.
Change-Id: I14bf5f20e36ec4547aad45382cfffc91666a2ff4
Signed-off-by: Sanrio Alvares <salvares@codeaurora.org>
Currently, while de-initializing dci clients, there is
a possibility to access stale entries. This patch fixes
this issue by adding proper protection mechanism.
CRs-Fixed: 960958 968445
Change-Id: I829c9497eeb356662a6531592c66108e615ce6e4
Signed-off-by: Mohit Aggarwal <maggarwa@codeaurora.org>
There is a chance of zbud handle being used after a free.
Unable to handle kernel paging request at virtual address ffffffc05be72040
PC is at zcache_store_page+0x59c/0x618
LR is at zcache_store_page+0x59c/0x618
[<ffffffc00019c99c>] zcache_store_page+0x59c/0x618
[<ffffffc0001a70c4>] __cleancache_put_page+0x94/0xcc
[<ffffffc00015da4c>] __delete_from_page_cache+0xc0/0x2cc
[<ffffffc00016d230>] __remove_mapping+0xe4/0x128
[<ffffffc00016e750>] shrink_page_list+0x634/0x95c
[<ffffffc00016f32c>] shrink_inactive_list+0x41c/0x67c
[<ffffffc00016fc14>] shrink_lruvec+0x364/0x510
[<ffffffc00016fe10>] shrink_zone+0x50/0x12c
[<ffffffc000170278>] try_to_free_pages+0x38c/0x56c
[<ffffffc000164e4c>] __alloc_pages_nodemask+0x5e0/0x994
[<ffffffc000165214>] __get_free_pages+0x14/0x60
CRs-Fixed: 968859
Change-Id: I24f6cf8ccbac956d4c3114e70a9f94f5e3bfa1c8
Signed-off-by: Vinayak Menon <vinmenon@codeaurora.org>
Adapt JPEG DMA driver to SOC layer by replacing the msm specific
routines with SOC API which eases the portability of driver
on to non-msm platforms.
CRs-Fixed: 968483
Change-Id: I2738062ef11b64acb0501694ce4e35c26766d8a1
Signed-off-by: Lakshmi Narayana Kalavala <lkalaval@codeaurora.org>
Adapt JPEG driver to SOC layer by replacing the msm specific
routines with SOC API which eases the portability of jpeg driver
on to non-msm platforms.
CRs-Fixed: 968483
Change-Id: I8436604e8fd0984f1cd31ce99035dd55a33e89f3
Signed-off-by: Lakshmi Narayana Kalavala <lkalaval@codeaurora.org>
Adapt fd driver to SOC layer by replacing the msm specific
routines with SOC API which eases the portability of fd driver
on to non-msm platforms.
CRs-Fixed: 968483
Change-Id: Iadce8d1b62b81b000957bc482dd60547937cf5c1
Signed-off-by: Lakshmi Narayana Kalavala <lkalaval@codeaurora.org>
The bus vectors are statically setup in jpeg and fd driver which is not
portable on non-msm platforms. Hence add the vectors in dtsi node
where the vectors can be setup with the bus routine
msm_bus_cl_get_pdata dynamically.
CRs-Fixed: 968483
Change-Id: I50c81a6c5daafe02645b074525eb3c026b85d3bb
Signed-off-by: Lakshmi Narayana Kalavala <lkalaval@codeaurora.org>
Capacity learning is already enabled for PMI8994/8996 to run
the battery aging algorithm. Learning feedback is needed to
run that algorithm with providing feedback across multiple
resets. Enable it.
CRs-Fixed: 968756
Change-Id: Id6596a94f125f113c919c0bea107582165da2f90
Signed-off-by: Subbaraman Narayanamurthy <subbaram@codeaurora.org>
Add settings to switch to always-on clock source during certain
LPM mode exit paths on msm8996 Pro.
CRs-Fixed: 968587
Change-Id: I6138681e2a85b7d1ad11350718544de6abe38131
Signed-off-by: Deepak Katragadda <dkatraga@codeaurora.org>
Some MSM8996 boards will employ the PM8994 PMIC alone. Introduce
msm8996-pm8994.dtsi which allows only the use of peripherals
available under PM8994.
CRs-Fixed: 968575
Change-Id: I4ce7c5ade7f81f49cecb2cd458c9044489ea1132
Signed-off-by: Subbaraman Narayanamurthy <subbaram@codeaurora.org>
ARM driver supports only one bus for all context banks.
In some cases, hypervisor may not be available and GPU SMMU
uses ARM driver. This will make all context banks are on
non secure bus and kgsl_mmu_bus_secured() returns -EPERM.
Make platform_bus_type as secure for ARM driver.
Change-Id: I11a637ca2b1ef29cc42c9811cad009312a2879cd
Signed-off-by: Rajesh Kemisetti <rajeshk@codeaurora.org>
Add support to receive a static ratio from userspace to
divide the swap pages between ZRAM and disk based swap
devices. The existing infrastructure allows to keep
same priority for multiple swap devices, which results
in round robin distribution of pages. With this patch,
the ratio can be defined.
CRs-fixed: 968416
Change-Id: I54f54489db84cabb206569dd62d61a8a7a898991
Signed-off-by: Vinayak Menon <vinmenon@codeaurora.org>
App region notification has to be sent only once, if appsbl
has already sent then kernel should not send.
Change-Id: Ie1c7bb78be30c723fef6d9b89d488fdac36bc07f
Signed-off-by: Mallikarjuna Reddy Amireddy <mamire@codeaurora.org>
The service info structure is allocated with uninitialized memory for the
max number of services and returns the complete structure to the usersapce
resulting in the information leak if lookup operation finds less number of
services than the requested number.
Check the minimum of requested and available services and copy the minimum
information to the user-space.
CRs-Fixed: 965934
Change-Id: Ic97f875855fdc6440c1db1d8d0338ee8b03a9d0a
Signed-off-by: Arun Kumar Neelakantam <aneela@codeaurora.org>
cpu3 stuck in printk more time in spin lock low_water_lock cause cpu0
get spin lock fail and system crashed.
CRs-Fixed: 521570
Change-Id: I75356a4b4171ae2888ce6cce792f569b5ca8cdcf
Signed-off-by: Tingting Yang <tingting@codeaurora.org>
Signed-off-by: Prasad Sodagudi <psodagud@codeaurora.org>
Currently, on receiving the write completion
notification from USB, there is possibility of
double freeing the buffer. This patch takes care
of this issue.
CRs-Fixed: 967601
Change-Id: I1207e0c3f103fa9dec6d766621aec8ccea62be69
Signed-off-by: Mohit Aggarwal <maggarwa@codeaurora.org>
The hdcp 2.2 module may or may not be loaded based on source
and sink capabilities. Some of the common functionalities with
hdcp 1.4 may be accessed by different modules depending on which
hdcp version is being used. Check for validity of internal data
structures and variables to guard hdcp functionalities against
illicit calls.
Change-Id: I1383ae77f0b904589bcab571c902b5e3eeef6efa
Signed-off-by: Ajay Singh Parmar <aparmar@codeaurora.org>
During hotplug if an RPS CPU goes offline,
then there is a possibility that the IPI
delivery to the RPS core might fail, this
happens in the cases when unruly drivers
use netif_rx API in the wrong context.
This happens due to two reasons
a) Firstly using netif_rx API in non preemptive
context leads to enough latencies that the IPI
delivery might fail to an RPS core. This is because
the softIRQ trigger will become unpredictable.
b) by using netif_rx it becomes an architectural
issue where we are trying to do two things in two
different contexts. We set the NAPI bit in context
and sent the IPI in other context. Now since the
context switch is allowed, the remote CPU is allowed
to go finish its hotplug.
If there was no context switch in the first place,
which typically happens by either using the correct
version of netif_rx or switching to NAPI framework,
then the remote CPU is not allowed to go to CPU DOWN
state. This is by design since hotplug framework causes
the remote dying CPU to wait until atleast one context
switch happens on all other CPUS. If preemption is
disabled then the dying CPU has to wait until preemption
is enabled and a context switch happens.
This patch catches these unruly drivers and handles
IPI misses by clearing NAPI sate on remote RPS CPUs
Please refere here for more documentation on hotplug
and preemption cases https://lwn.net/Articles/569686/
CRs-Fixed: 966095
Change-Id: I072f91bdb4d7e444e3624e8e010ef1b66a67b1ed
Acked-by: Abhishek Chauhan <abchauha@qti.qualcomm.com>
Signed-off-by: Ravinder Konka <rkonka@codeaurora.org>
Remove improper deletion of list head and protect list
deletion with spinlock. Spinlock protects the list
from access in ADSP callback at the same time the
list is being deallocated.
Change-Id: I1b102cd99e7c80e1dfd4e2054b9ba7e0ca26d6a7
Signed-off-by: Ben Romberger <bromberg@codeaurora.org>
Change the HW delay debug print so it does not
seem like an error. Adds comments in the code
describing the HW delay feature.
Change-Id: I0955623a7bdb0f4180c908ba079cf6f900e496a5
Signed-off-by: Ben Romberger <bromberg@codeaurora.org>
A null pointer dereference was possible because a check on
the return value from kmalloc was not in place. This change
adds a null pointer check. This change also changes an
existing null check to check the correct pointer.
CRs-Fixed: 964856
Change-Id: I899b908b84d5d3b8e39b1b30b942e9eb7ed7f7da
Signed-off-by: Chris Lew <clew@codeaurora.org>
Ensure CPP RX fifo is empty while writing firmware and during
duplication command. This avoids any firmware load failures
due to possible overwrite of payload.
CRs-Fixed: 967466
Change-Id: Ie964614ac9893750e5c8ca08362e877352d6ebe2
Signed-off-by: Krishnankutty Kolathappilly <kkolatha@codeaurora.org>
In batch mode with duplication use case, video buffer needs to be
primary buffer and preview buffer the duplicate buffer. If preview
buffer is the primary buffer batch video output is not generated
resulting in green frames. Swap the preview and video identity if
the batch buffers are sent on preview identity and duplication is
enabled.
CRs-Fixed: 952085
Change-Id: I46f63189e784ec388969a756b8e32c1180640478
Signed-off-by: Krishnankutty Kolathappilly <kkolatha@codeaurora.org>
The meta data offsets is not set to zero causing invalid buffers
being send out from cpp causing green frames in video during
batch mode for NV12 formats. If format is not ubwc, reset meta
data offset to zero in the set buffer group payload.
CRs-Fixed: 952085
Change-Id: I912b28d9ed7f02e5e87513d92d088adcbebe1d1b
Signed-off-by: Krishnankutty Kolathappilly <kkolatha@codeaurora.org>
Currently class-H controller is not enabled when lineout is enabled
first and then headphone is enabled. Because of this mute is observed
on headphone during lineout and headphone concurrent scenarios.
Fix mute on headphone by enabling class H block during concurrency.
Change-Id: Ibed962eaacf87c3eb55c569d2ae9c6363a63cace
Signed-off-by: Vidyakumar Athota <vathota@codeaurora.org>
CRs-Fixed: 957405
Remove dump of SAW2 registers in case of wcnss abort to
avoid kernel panic.
CRs-Fixed: 967730
Change-Id: I08f9ca6268575b2d6d1e266353a98714fbeeee10
Signed-off-by: Sunkad, Anand Ningappa <asunka@codeaurora.org>
This patch prevents deadlock between userspace and kernel space uhid
driver by releasing the held mutex(s) before waiting for response for
set/get report. Without this there was potential deadlock in code
that lead to unnecessary wait for 5 seconds for response for set report
to be received from user space which could not come, as mutex was held
leading to blockage of read and further write.
Change-Id: I3f003f33bdf2a0aeb4821598c667cbf61db83b9b
CRs-Fixed: 963213
Signed-off-by: Hemant Gupta <hemantg@codeaurora.org>
Device crash is observed when the critical_irq is triggered because of
accessing uninitialized completion variable.
Update condition in critical_irq such that the completion variable will be
accessed only when initialized.
Change-Id: I0cba089882024bbe36ac4798866af4e33ee02fa2
Signed-off-by: Rama Krishna Phani A <rphani@codeaurora.org>
Enable support to model the multimedia clocks instead of
using dummy clocks. We are still using dummy clocks on
RUMI until the FSM dependencies can be met.
Change-Id: I1827188e554fab50db531091dde7ff99a366d823
Signed-off-by: Deepak Katragadda <dkatraga@codeaurora.org>
Add support to model the multimedia clocks on MSMCOBALT.
Change-Id: Iec33fa93e745a65205cf4206759289d7e842fe36
Signed-off-by: Deepak Katragadda <dkatraga@codeaurora.org>
DDC and CEC GPIOs need to be configured as pull up for both
active and suspend state as the companion chip's enable gpio
controls these lines and expects pull up configurations.
Correct the same to avoid any power leakage during suspend state.
Change-Id: Ia40d3e173a8215d0db48ee846f4f45938279f58f
Signed-off-by: Ajay Singh Parmar <aparmar@codeaurora.org>
Update with the correct maximum and minimum voltage levels for
the gfx supply on MSM8996-Pro.
Change-Id: Ie6a90488124bb0c47fd09948a5c202ffa10aee27
Signed-off-by: David Collins <collinsd@codeaurora.org>
Update the cpufreq device with the latest frequency list for
msm8996pro. Also update the cpufreq to devfreq device frequency
mapping accordingly.
Change-Id: Ice9a8d782ca3619ec93e02ec88a10ecd88667c36
Signed-off-by: David Collins <collinsd@codeaurora.org>
Update the frequency plan for the MSM8996pro GPU clock.
Change-Id: I05228a3fc5da4cf24452ada3ee4363fe6e92e042
Signed-off-by: David Collins <collinsd@codeaurora.org>
Signed-off-by: Deepak Katragadda <dkatraga@codeaurora.org>
Currently, the min_supported_rate is only used for fabia PLLs.
This is a generic PLL property which needs to be checked for
all PLLs.
Change-Id: Iaec2ee84468bb40a68ded9355aa7587eabbe86b9
Signed-off-by: Deepak Katragadda <dkatraga@codeaurora.org>
Update the frequency plan for the MSM8996pro CPU clocks.
Change-Id: Ib395678da8be33ff30a3630837008ee911bc5616
Signed-off-by: David Collins <collinsd@codeaurora.org>
Specify the MSM8996-Pro VDD_GFX CPR configurations. This
includes a new corner mapping for the additional GPU frequencies
found on MSM8996-Pro as well as the voltage adjustments
appropriate for all of the frequencies.
Change-Id: Ic704b0e886de2aaffa4afb044704c21b6e8d24da
Signed-off-by: David Collins <collinsd@codeaurora.org>
There can only be one module_init() function per module. Move all
three driver register calls into the same initialization function. The
ordering should still work correctly.
Change-Id: Ic0dedbadf7c69221a836ba3bbba362d0660f1f0f
Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org>
Add a helper macro to convert an adreno_device pointer to a
struct kgsl_device pointer. This is mostly syntatic sugar
but it makes the code a bit cleaner and it abstracts a bit of
the ugliness away.
Change-Id: Ic0dedbadd97bda3316a58514a5a64757bd4154c7
Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org>
The ringbuffer structures are static members of struct adreno_device
which means that they are permanently associated with a specific
adreno device and by extension a struct kgsl_device too. The upshot
is that we can use macro math to derive the adreno device from
a ringbuffer pointer and get rid of the device shortcut in the
ringbuffer struct. This also gives us a chance to clean up
how functions use the ringbuffer and adreno_device structs
to limit unnessesary dereferencing.
Change-Id: Ic0dedbad909ef71e99cd3319713cee38fb1700f0
Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org>
when mediaserver crashes we are getting error that
__msm_destroy_session_streams: Error: Destroyed
list is not empty.
The order of stream_id for normal camera close is
1,2,3,4,0.When mediaserver crashes stream id order
is 0,1,2,3,4.stream id 4 is coming in the last and
when pvdev->opened is 0 we are not deleting the
stream.Added stream_delete when pvdev->opened is 0.
Change-Id: I22eaa8bd485e19605b9b69e86644f68ce209e808
Signed-off-by: Manish Poddar <mpoddar@codeaurora.org>
RUMI platform uses QRBTCv2 phy that has 2 separate sets of registers.
One is the phy register set, and the other is the U11 user registers.
Mapping both sets will require to map a larger range of memory that
is overlapping with the ICE registers address.
Separate into 2 independent memory mappings to avoid the conflict.
Change-Id: Ifdb426cdd7139e918c5c3747f5529b047f4fc1e5
Signed-off-by: Gilad Broner <gbroner@codeaurora.org>
ADC driver prints error log whenever vadc read fails. Update the
condition such that log gets printed for error case and not for probe
deferal.
Change-Id: Ieff7f82fbe508a8422f666635a489f45f2deb41f
Signed-off-by: Rama Krishna Phani A <rphani@codeaurora.org>
Add support for deferring touch resume to a workqueue on
msm8996 DTP.
Change-Id: Ic66648a6cbb36d884f315eadaa0f4c48ced04c96
Signed-off-by: zhaoyuan <yzhao@codeaurora.org>
Currently VBIF XIN register offset is been overwritten by
the AXI offset. This will cause VBIF XIN halt time out in
VBIF clear transaction path. Fix this by using the proper
VBIF XIN offset for A3xx targets.
Change-Id: Iac20528cb105904e46e012d67287dd736fa11f70
Signed-off-by: Hareesh Gundu <hareeshg@codeaurora.org>
Add NULL check for the pointer to make sure memory is
allocated. Add array boundary check for the index
before using the same array.
Change-Id: Ifd35ca0c0a1cbeee18d7797ea6b43ea31be0eeba
Signed-off-by: Viraja Kommaraju <virajak@codeaurora.org>
Add change to include correct array length
for buffer.
Change-Id: I9fd89b38b795401e7b6fdf848cb34fa20814a319
Signed-off-by: Viraja Kommaraju <virajak@codeaurora.org>
