| Commit message (Collapse) | Author | Age |
| ... | |
| |\ \ \ \ \ \ \ |
|
| | | |/ / / / /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Recovery can be initiated any time and can happen during driver
register also. With current implementation, if recovery starts
just before driver register then probe call back will not be
called as FW is not ready yet but probe call back will not be
called even after FW ready indication. Fix this case by calling
probe call back after FW ready if driver is registered but probe
call back is not called during driver register.
CRs-fixed: 2002797
Change-Id: Ia1ea812a1bac3204ad95a62b7c6e55511753f2db
Signed-off-by: Prashanth Bhatta <bhattap@codeaurora.org>
|
| |\ \ \ \ \ \ \
| |/ / / / / /
|/| | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Correct warnings flagged by checkpatch. In particular, modify
the following:
- Add 'const' to the type of a struct of_device_id variable.
- Use octal file permissions instead of symbolic.
- Restructure conditionals to avoid else after return.
- Add a blank line after variable definitions.
- Correct the format of block comments.
- Replace sscanf() for a single integer with kstrtoint().
- Update the QPNP_HAPTIC Kconfig entry to use the name
'Qualcomm Technologies, Inc.'
- Expand the description of the QPNP_HAPTIC config option.
Change-Id: Ifc9ccbb83e2ca59328d6dcb5915b80c578d38fa1
Signed-off-by: David Collins <collinsd@codeaurora.org>
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Locking ION buffer should not fail, print error message if it happens.
Signed-off-by: Amir Samuelov <amirs@codeaurora.org>
Change-Id: Ic67f69daa646712dd814f29d5aa0fc68386177e1
|
| |\ \ \ \ \ \ \ |
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Add error message for GFP_ATOMIC allocation failure. Keep current
design to drop packet if allocation fails. This print will help debug
issues where a system critical client fails because of a dropped GLINK
packet.
CRs-Fixed: 1112151
Change-Id: I6a69cbf1f88295009284d726a06fa5affd4cc591
Signed-off-by: Chris Lew <clew@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ |
|
| | |/ / / / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
During ramdump collection we assign memory to HLOS from subsystem for
non-secure pil. Whether ramdump collection is successful or not, we
should assign memory back to subsystem. This is to avoid access
violations in powerup path which happens after ramdump.
CRs-Fixed: 2002073
Change-Id: I7f1d42aebb44464fe077ca544ce91c2d7a8eefbb
Signed-off-by: Puja Gupta <pujag@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ |
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
It is not always guaranteed that glink will ack all
tx buffers intime and hence apr might run out of tx
buffers for apr message transactions. Create TX buffers
dynamically instead of preallocating fixed number of
buffers.
Change-Id: I53c9529d1e84a88cc2e0b161a8a5047d5a475620
Signed-off-by: Shiv Maliyappanahalli <smaliyap@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ \
| |_|/ / / / / / /
|/| | | | | | | | |
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
In function close_ch_worker variable ss_info->cb_data is not set to
NULL after free, this results in crash due to use after free in
notify_for_subsystem function.
This variable is set to NULL to avoid any crash due to use after free.
CRs-Fixed: 2000384
Change-Id: Ia0c7b52e4c8f5d0215fd0ebfbca5e823530f205c
Signed-off-by: Dhoat Harpal <hdhoat@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ \ |
|
| | | |_|/ / / / / /
| |/| | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Enable FW rejuvenate feature so that WLAN FW
can rejuvenate by default without changing the
configuration from debugfs.
CRs-Fixed: 2000561
Change-Id: Ic9b0524311c08bf3b99f43881054c23967734e5c
Signed-off-by: Yuanyuan Liu <yuanliu@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ \
| |_|/ / / / / / /
|/| | | | | | | | |
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
enable_fw_log is renamed to fw_log_mode and changed
from boolean type to uint8_t, so that it can support
multiple log mode.
CRs-Fixed: 1116983
Change-Id: I9d4af7cd2c857b62f434d968f51af76121006907
Signed-off-by: Yuanyuan Liu <yuanliu@codeaurora.org>
|
| | |/ / / / / / /
|/| | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Add config option to enable/disable l2 indirect access debug capability.
The driver exposes l2 indirect access debugfs interface to get/set data,
address, and target cpus so keep it disabled by default.
Change-Id: I22f84d16a3bf12a78295f2d052bb50e90d6f2a8b
Signed-off-by: Kaushal Kumar <kaushalk@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ |
|
| | | |_|_|/ / / /
| |/| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
This patch defines condition for collecting ramdump
only when ramdump collection is enabled, modem is crashed
and memshare is notified of SUBSYS_RAMDUMP_NOTIFICATION.
CRs-Fixed: 1114015
Change-Id: I7e3ca0794980fd21cc0a5d890df4aeadfd5fbbbb
Signed-off-by: Manoj Prabhu B <bmanoj@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ |
|
| | | |_|/ / / / /
| |/| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
While posting PD_SERVICE_DOWN indication to event queue, wrong
variable used in sizeof to allocate memory for event_data. Fix by
using event_data in sizeof to get the right memory size to be
allocated.
CRs-fixed: 1113172
Change-Id: I44c04cf7edd1f09e2275ec040ab99dc34a197cf8
Signed-off-by: Prashanth Bhatta <bhattap@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ |
|
| | | |_|_|_|_|_|/
| |/| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Add checks to make sure userspace buffer is valid before
it is used. Add upper limit for glink channels and number
of intents allowed.
Change-Id: I9b8f2e47aada7922ba22cbb010176bf36a9d6288
Signed-off-by: Vidyakumar Athota <vathota@codeaurora.org>
|
| |\ \ \ \ \ \ \ \
| | |_|_|/ / / /
| |/| | | | | | |
|
| | | |_|/ / / /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Don't check return value of sending dynamic feature mask in cold
boot initialization sequence as it is an optional message.
CRs-Fixed: 1115626
Change-Id: Ieb2f86f3a5c2c368b2699755a716a3e385562f48
Signed-off-by: Yuanyuan Liu <yuanliu@codeaurora.org>
|
| |\ \ \ \ \ \ \ |
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Add CDSP loader driver to bring compute DSP out of reset
during boot.
Change-Id: I2571e2158fcf7706efc5a5922da24f22755eeed7
Acked-by: Himateja Reddy <hmreddy@qti.qualcomm.com>
Signed-off-by: Sathish Ambley <sathishambley@codeaurora.org>
|
| |\ \ \ \ \ \ \ \
| |_|/ / / / / /
|/| | | | | | | |
|
| | | |/ / / / /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
G-Link commands are deferred until all the incoming data traffic is
handled. If the incoming data traffic is overwhelming, then this causes
the G-Link commands to pile up and be deferred forever.
Set a threshold to the number of deferred commands. Once that threshold
is reached, handle the deferred commands before handling further data.
Change-Id: I53dc3a7feb40a869030ae64e69a5359c6537e589
Signed-off-by: Karthikeyan Ramasubramanian <kramasub@codeaurora.org>
|
| |\ \ \ \ \ \ \ |
|
| | | |/ / / / /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Service UP/DOWN indication logs are critical during boot or PD restart
to understand the communication between application processor and remote
processor.
Change-Id: Ie372f00ae032e9a5697c2eb6ef6ed65de9849933
Signed-off-by: Satya Durga Srinivasu Prabhala <satyap@codeaurora.org>
|
| |\ \ \ \ \ \ \
| |/ / / / / /
|/| | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Usage of uninitialized variable remote_host happens if common partition
header hosts value dont match with TOC values.
Add common partition check and error log if hosts mismatch with TOC.
CRs-Fixed: 1113351
Change-Id: I15f8823751abccf87c55e570a8a762bfb2cfb2df
Signed-off-by: Dhoat Harpal <hdhoat@codeaurora.org>
Signed-off-by: Arun Kumar Neelakantam <aneela@codeaurora.org>
|
| |\ \ \ \ \ \ \
| |_|_|/ / / /
|/| | | | | | |
|
| | |/ / / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Conversion of a 64 bit int variable to void pointer is illegal
operation when compiling for 32 bit architecture.
Added explicit conversion to uintprt_t, so that 32 bit conversion
can be done.
CRs-Fixed: 1107920
Change-Id: I3854fcb460308c9a77a17198dcb604dd4da135ac
Signed-off-by: Dhoat Harpal <hdhoat@codeaurora.org>
|
| |\ \ \ \ \ \ |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Replace WARN with pr_warn so that stack trace is not printed with the
message.
CRs-Fixed: 1111653
Change-Id: Id1c7bbe8a528199261455ba7901e9df81913aef8
Signed-off-by: Puja Gupta <pujag@codeaurora.org>
|
| |\ \ \ \ \ \ \
| |_|/ / / / /
|/| | | | | | |
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Free the sg_table properly.
Change-Id: Idae945d17803e10756bf3a20426e57837d185067
Signed-off-by: Patrick Daly <pdaly@codeaurora.org>
|
| |\ \ \ \ \ \ \ |
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Below object corruption observed from service-notifier while executing
WLAN PDR tests. While executing WLAN PDR tests, Modem crashed, which
resulted into use after free condition. Add mutex lock to fix the issue.
===============================================
BUG kmalloc-1024 (Tainted: G W O ): \
Poison overwritten
-----------------------------------------------
INFO: 0xffffffc0ba5c4d30-0xffffffc0ba5c4d3a. \
First byte 0xff instead of 0x6b
INFO: Allocated in qmi_handle_create+0x4c/0x320 \
age=80203 cpu=0 pid=27134
alloc_debug_processing+0x124/0x178
___slab_alloc.constprop.62+0x4ec/0x5d0
__slab_alloc.isra.59.constprop.61+0x48/0x74
kmem_cache_alloc_trace+0xd4/0x25c
qmi_handle_create+0x4c/0x320
root_service_service_arrive+0x3c/0x1e8
process_one_work+0x234/0x40c
worker_thread+0x2e0/0x410
kthread+0xf4/0xfc
ret_from_fork+0x10/0x40
INFO: Freed in qmi_handle_destroy+0x154/0x16c \
age=50 cpu=1 pid=22765
free_debug_processing+0x27c/0x354
__slab_free+0x74/0x3ac
kfree+0x21c/0x290
qmi_handle_destroy+0x154/0x16c
root_service_service_exit+0xf0/0x110
ssr_event_notify+0x74/0x84
notifier_call_chain+0x58/0x8c
__srcu_notifier_call_chain+0x9c/0xc0
srcu_notifier_call_chain+0x38/0x44
subsys_notif_queue_notification+0x40/0x54
notify_each_subsys_device.isra.9+0x18c/0x1b0
subsystem_restart_wq_func+0x2f4/0x5a8
process_one_work+0x234/0x40c
worker_thread+0x2e0/0x410
kthread+0xf4/0xfc
ret_from_fork+0x10/0x40
Change-Id: Iebc0871220547c2d6f026db1dffafe30b854b199
Signed-off-by: Satya Durga Srinivasu Prabhala <satyap@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ |
|
| | | |_|/ / / / /
| |/| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Propagate error code upwards when piltz_resc_init fails. Previous return
prevented probe from retrying when piltz_resc_init returns -EPROBE_DEFER.
Change-Id: I217f13eb4bff321ae1b7583f5d2614b6c9589987
Signed-off-by: Kyle Yan <kyan@codeaurora.org>
|
| |\ \ \ \ \ \ \ \ |
|
| | |/ / / / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
There are cases where IPC router read worker not getting scheduled
because of other high priority tasks. And when this happens there could
be a delay in receiving messages from WLAN firmware. So increase the
timeout delay to 10 seconds.
CRs-Fixed: 1110268
Change-Id: I4b3420a91108eca844206789d8f1e478526afd62
Signed-off-by: Sameer Thalappil <sameert@codeaurora.org>
|
| |\ \ \ \ \ \ \ \
| |_|_|_|/ / / /
|/| | | | | | | |
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
In function glink_core_remote_close_common, notify_state callback
is called before clearing wait queue. This leads to deadlock if client
want to synchronize tx and state notify function.
Complete_all is called before notify client about state change,
so that all pending requests from client will be cleared.
CRs-Fixed: 1107652
Change-Id: Ia6c4a305eb42c014a928bad36491e6e5f6eac9d5
Signed-off-by: Dhoat Harpal <hdhoat@codeaurora.org>
|
| | |/ / / / / /
|/| | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Select which SPSS firmware to load based on 2 fuses.
Change-Id: I7080cc0b58c38965ed7e58808431cf487008de9e
Signed-off-by: Amir Samuelov <amirs@codeaurora.org>
|
| |\ \ \ \ \ \ \ |
|