Memory allocated with devm_kzalloc() is automatically freed if
the probe function fails and returns an error code. So there
is no need to free cpu_grp explicitly for the failure cases
that might come up after it is allocated. Additionally calls to
devm_kzalloc() must be accompanied by devm_kfree() if memory
deallocation is necessary.
For these reasons remove the kfree(cpu_grp) from the probe
function.
Change-Id: Ic4838fd58d40d283ac301facc64b06813eb3bd7d
Signed-off-by: Rohit Gupta <rohgup@codeaurora.org>
This change adjusts the ot limits in the mdp vbif
for msmcobalt by removing the xin ot limits and
adding the axi ot limits to the mdp vbif.
CRs-Fixed: 1040746
Change-Id: I365976021dcf02fbc3687091371d7d72cb711605
Signed-off-by: Ingrid Gallardo <ingridg@codeaurora.org>
Fix to GSI reset stats logic to clear the correct stats per channel
Change-Id: I94ae8c769ec2868a291e75a2f47c85bd8a449937
CRs-Fixed: 1046484
Acked-by: Ady Abraham <adya@qti.qualcomm.com>
Signed-off-by: Skylar Chang <chiaweic@codeaurora.org>
IPA driver uses 8k buffers for RX packets as those are aggregated.
If these packets deaggregated in IPA driver, cloned and provided
to clients, later calls to skb_copy() results in creating an 8k skb
which may lead to high memory utilization.
This change avoid the use of skb clones and allocates new skbs
for LAN clients and copy the packet content to it.
CRs-Fixed: 1050670
Change-Id: Iba47db8936b49309360791c1c6b482dcfcffd4d7
Acked by: Ady Abraham <adya@qti.qualcomm.com>
Signed-off-by: Skylar Chang <chiaweic@codeaurora.org>
Use of cpu_suspend is deprecated. Instead use arm_cpuidle_suspend for
32bit systems.
Change-Id: I5c61c970e17cba44c555e60418a9ece80c61488f
Signed-off-by: Raju P.L.S.S.S.N <rplsssn@codeaurora.org>
Synchronization is required because bind/unbind can happen in two
different contexts. For example, function unbind due to
unregister_gadget_item can race with bind which is initiated from
sysfs.
Change-Id: Iec93a9d4cb2a10b833cbba0d5256b64161147de2
Signed-off-by: Hemant Kumar <hemantk@codeaurora.org>
Release the lock before unregistering gadget item.
Unregistered gadget item will result in unbind call back
to the driver which may use the same lock and resulting
in deadlock condition.
Change-Id: I2f8fba73a831d3bb711fe9310c5758221359819e
Signed-off-by: Hemant Kumar <hemantk@codeaurora.org>
GDSC regulators in case of 'qcom,skip-logic-collapse', would require block
assert/deassert of the clocks. Add support for the same to use the reset
controller APIs.
Change-Id: I83651b3b3515e0923d7431cfe8b70e6059b51067
Signed-off-by: Taniya Das <tdas@codeaurora.org>
Add the block reset clocks which will be used by clients to
assert/deassert these clocks using the reset controller framework.
Change-Id: I3e9f7f85bf1faf0e1bb501196ba9d7e197111a03
Signed-off-by: Taniya Das <tdas@codeaurora.org>
A reset signal is associated with each clock branch but sometimes a reset
signal is associated with a handful of clocks. Either way the register
interface is the same; set a bit to assert a reset and clear a bit to
deassert a reset. Add support for these types of resets signals.
Change-Id: Ic9d00c0a03507a55ca6c96f977a6ddf55b4b5db7
Signed-off-by: Taniya Das <tdas@codeaurora.org>
Change BCR value for IPAv3.1 according to updated value from HPG.
Add IPAv3.5 to receive BCR initialization according to HPG.
CRs-Fixed: 1051371
Change-Id: If2a7eaf11ad05ced3c52ff87be934745b7dc713b
Signed-off-by: Amir Levy <alevy@codeaurora.org>
Check input parameter range before using it to access
array elements and check pointer value before
dereferencing it.
Change-Id: I702b4ff11582047b00e3f0d5433d29c84f34e913
CRs-Fixed: 1048706
Signed-off-by: Ghanim Fodi <gfodi@codeaurora.org>
Hardware characterization has shown that some parts require
higher voltages in order to operate consistently. Increase
the ceiling voltage for all corners so that they are able to
operate at higher voltages. This applies to all CPR revisions
except 0.
Change-Id: Ie9d4e825e5c6040036642cdaf22d1f67b6129685
CRs-Fixed: 1050071
Signed-off-by: David Collins <collinsd@codeaurora.org>
Add header file of irq-helper to export black_list on/off api to
other module.
CRs-Fixed: 1051104
Change-Id: I4737975961b18f1095e3cd78d07a2bca22931291
Signed-off-by: Runmin Wang <runminw@codeaurora.org>
Enable irq-helper on msmcobalt to export apis for user space.
CRs-Fixed: 1051104
Change-Id: I7a623463a142a4db1db7247cf7c5dfeb5b99283c
Signed-off-by: Runmin Wang <runminw@codeaurora.org>
Add support for the MSMCOBALT force highest corner fuse. This
fuse is set on parts which can only operate consistently when
the highest corner's voltage is applied to all of the lower
corners.
Change-Id: Ibeef761044deea375dc7684c0a160609b610b8f6
CRs-Fixed: 1050071
Signed-off-by: David Collins <collinsd@codeaurora.org>
Log function entry and exit and dump relevant values
into ipc log buffer. This allows to debug various
race conditions and stability issues.
Change-Id: I15011d79fc2f054e64f8bbd1f8f5db8944b46ada
Signed-off-by: Hemant Kumar <hemantk@codeaurora.org>
Driver is using "qcom,emu-init-seq" for finding length of property
but using "qcom,qemu-init-seq" instead of "qcom,emu-init-seq" to
get passed values from device tree. With this, length of property
comes as expected but values are not received. This results into
programming zeros as values with offset as zero instead of expected
<value, offset> passed through devicetree.
CRs-Fixed: 1051115
Change-Id: I1809fcef844d275175814e636591b87e91432609
Signed-off-by: Mayank Rana <mrana@codeaurora.org>
Add the QDSS STM property to the device tree so
that data can be logged from the GPU to the debug
bus when available.
CRs-Fixed: 1031648
Change-Id: I754c85a2ed02d0b1e40fd1e27b10ff84c463ac83
Signed-off-by: Jonathan Wicks <jwicks@codeaurora.org>
Add support for FB notifications to trigger suspend/resume
based on FB blank/unblank events.
Change-Id: I79abc586ad8c14a25afd56559c579a1415df9f0e
Signed-off-by: Mohan Pallaka <mpallaka@codeaurora.org>
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
Correct function and variable names in ITE tech driver from upper case
to lower case, and use underscore instead of camel-case in their names.
This change makes the driver as per kernel coding standards and similar
to other touch drivers.
This is a propagation patch from 3.18 kernel 'commit c1fe9ac38d93 ("input:
touchscreen: correct function and variable names in ITE tech driver")'
CRs-Fixed: 982219
Change-Id: Ic8b66317f414fe3bfcd0f9c5b03369cc304edb55
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
Correct return values of the functions in ITE tech driver.
Use valid integer error and success values instead of boolean
values.
This is a propagated patch from 3.18 kernel 'commit a68ff7e23d20
("input: touchscreen: correct return values in ITE tech driver")'
CRs-Fixed: 982219
Change-Id: Iba07517fd32dbeb3079df9c0346ea0c044b88f63
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
Remove the use of global variable from ITE tech touch driver,
and use the internal driver structure passed as function argument.
This change makes the driver complied to kernel coding standards
and similar to other touch driver.
This is a propagated patch from 3.18 patch 'commit 041f90ddeb5e ("input:
touchscreen: remove global variable from ITE tech touch driver")'
CRs-Fixed: 982219
Change-Id: Ib609d6e76ea70cd3b49c4ea6f09c75bf52521aa8
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
Identify the ITE tech controller until it is ready to do any
i2c transaction. This is needed for dynamic detection and to check
if ITE tech controller is connected or not.
This is a propagated patch from 3.18 kernel 'commit eb37d1174948
("input: touchscreen: identify ITE tech controller until ready")'
CRs-Fixed: 980427
Change-Id: I9f69eb541e31de6a04db7468ca12a04a837d2b40
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
Correct the return value check of kstrtoint function.
Also correct the platform_data parsing check in ITE tech
touch driver.
This is a propagated patch from 3.18 kernel
'commit 9a1496537aeb ("input: touchscreen: correct condition checks
in ITE tech touch driver")'
Change-Id: I792c043ca4d85e236c209605e80392a05d26b983
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
Don't allow ITE tech driver to check the status of the controller forever
before any i2c read/write, instead make it to 10 times only.
In case of firmware flashing and external calibration features, this is
made to 500 times.
This is a propagated patch from 3.18 kernel
'commit 5e8c98c6705b ("input: touchscreen: don't allow ITE tech driver
to do infinite i2c checks")'
Change-Id: I88e024700de43cc9f712fd7fa509fcb387f9cd21
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
Add device tree support for focaltech controller driver.
The data is represented in device tree format and parsed
by the driver using of framework apis.
This patch is a propagated patch from 3.18 kernel
'commit 83310a9a151e ("input: ft5x06_ts: Add DT support")'
Change-Id: Ibcdd8c3579f401ac0632d670e1b7c674aa67ba8d
Signed-off-by: Mohan Pallaka <mpallaka@codeaurora.org>
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
[abinayap@codeaurora.org: Fix checkpatch errors for 4.4 kernel
struct of_device_id should normally be const]
Signed-off-by: Abinaya P <abinayap@codeaurora.org>
Update TLMM base address to match new address.
Change-Id: Ie0c46cbbcbc3edabd5e0f867b4393a27e55db9f5
Signed-off-by: Neeraj Upadhyay <neeraju@codeaurora.org>
Update TLMM configuration for msmfalcon, to match new
HW definitions.
Change-Id: I61d99454dab2fa90b4ed6577d1a58685088eb968
Signed-off-by: Neeraj Upadhyay <neeraju@codeaurora.org>
Add SDC1 related pin configuration and update addresses for
SDC2 pins, to match TLMM addresses in new HW definitions.
Change-Id: If39ad62cc591565793cd1d8a0f2a454e0b7d5c58
Signed-off-by: Neeraj Upadhyay <neeraju@codeaurora.org>
Before doing any i2c read/write, driver needs to check the value
of QUERY buffer to know if the controller is in ready or busy state.
Sometime, it takes more number of checks to know the status.
So this change increases the number of checks to know the ITE tech
touch controller status.
This is a propagated patch from 3.18 kernel 'commit 60589278b8e7 ("input:
touchscreen: increase number of checks for touch controller status")
Change-Id: I30dd0cf0b15538e67f50725754d5059e2fe721d0
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
Signed-off-by: Abinaya P <abinayap@codeaurora.org>
Mstar reference driver is being cleaned up to be checkpatch
compliant. Add Kconfig and Makefile changes to enable Mstar
driver's compilation. And following new features are added to
the driver:
1. Pinctrl support
2. Threaded irq support
3. Release all touches in suspend
4. Protocol B compliance
5. Explicit suspend/resume function
6. Configure gpios in suspend/resume function
7. Add device tree parser function
8. Add dtsi support for gpio, regulator, I2C address, display coords
This patch is propagated from 3.18 kernel
'commit 091a01709e12 ("input: msg21xx_ts: update mstar
driver to enable msg2138A")'
Change-Id: Ic49a18de64ec210a0636405394ba7a8f52f336a9
Signed-off-by: Mao Li <maol@codeaurora.org>
Signed-off-by: Shantanu Jain <shjain@codeaurora.org>
Signed-off-by: Sudhakar Manapati <smanap@codeaurora.org>
[abinayap@codeaurora.org: Fix checkpatch errors for 4.4 kernel
- Block comments use a trailing */ on a separate line
- Comparisons should place the constant on the right side of the test
- struct of_device_id should normally be const
- DT compatible string vendor "mstar" appears un-documented]
Signed-off-by: Abinaya P <abinayap@codeaurora.org>
Modified condition to ensure data is read
only after interrupt has occurred.
Spurious interrupt handling is done in nfc_read
instead of irq handler.
Change-Id: Ie2362610fe922e792e6358b4386c828fdd754fa8
Signed-off-by: Gaurav Singhal <gsinghal@codeaurora.org>
