nvidia-tegra-modules.git/drivers/i2c, branch gpu-paging NVIDIA's kernel modules to support tegra chips (used in Jetson boards) i2c-ivc: Check return value. 2021-08-12T19:24:45+00:00 Achal Verma achalv@nvidia.com 2021-05-11T12:08:57+00:00 603a8031494660c40271862bd303a388d80378b0 This fixes the coverity issue: 3008845 Bug 200741055 Change-Id: I57112e37a16aec516a21363038c1e6dc511a7f9c Signed-off-by: Achal Verma <achalv@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2527774 GVS: Gerrit_Virtual_Submit Reviewed-by: Dmitry Pervushin <dpervushin@nvidia.com> Reviewed-by: Rahul Jain (SW-TEGRA) <rahuljain@nvidia.com> (cherry picked from commit efb34ee4c7f2f48ac7910a6936f5e2c1fc8b42af) Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2574544 Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com> Reviewed-by: Phoenix Jung <pjung@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
This fixes the coverity issue: 3008845

Bug 200741055

Change-Id: I57112e37a16aec516a21363038c1e6dc511a7f9c
Signed-off-by: Achal Verma <achalv@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2527774
GVS: Gerrit_Virtual_Submit
Reviewed-by: Dmitry Pervushin <dpervushin@nvidia.com>
Reviewed-by: Rahul Jain (SW-TEGRA) <rahuljain@nvidia.com>
(cherry picked from commit efb34ee4c7f2f48ac7910a6936f5e2c1fc8b42af)
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2574544
Reviewed-by: svc-mobile-coverity <svc-mobile-coverity@nvidia.com>
Reviewed-by: Phoenix Jung <pjung@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
drivers: i2c: Suppress NACK prints from bpmp i2c 2020-11-13T20:39:42+00:00 Mantravadi Karthik mkarthik@nvidia.com 2020-11-06T11:29:52+00:00 529dcdf0813bf2145f6d53b747f45b5f2dbd652b Use debug prints in case bpmp transaction fails. Only print debug msgs in case of NACK. Bug 3128306 Change-Id: Ifc08e4b139ffb9c9b3ebd860de75590004c38278 Signed-off-by: Mantravadi Karthik <mkarthik@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2442417 Reviewed-by: automaticguardword <automaticguardword@nvidia.com> Reviewed-by: Timo Alho <talho@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> GVS: Gerrit_Virtual_Submit 
Use debug prints in case bpmp transaction fails.
Only print debug msgs in case of NACK.

Bug 3128306

Change-Id: Ifc08e4b139ffb9c9b3ebd860de75590004c38278
Signed-off-by: Mantravadi Karthik <mkarthik@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2442417
Reviewed-by: automaticguardword <automaticguardword@nvidia.com>
Reviewed-by: Timo Alho <talho@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
GVS: Gerrit_Virtual_Submit
drivers: i2c-bpmp: Do not print error info for NACK 2020-10-13T10:09:50+00:00 Karthik Mantravadi mkarthik@nvidia.com 2020-09-25T15:58:53+00:00 a8fa9d9483f360e39cd1d8abb1d12cc65e509d53 In case of i2c transaction failures due to NACK on bpmp i2c, print only the return value from BPMP and not the debug error information. This avoids UART being flooded with prints in case of NACK from slave. Bug 3128306 Change-Id: I0869179b48f110a2b1bce7696966ebcae3f8b35b Signed-off-by: Karthik Mantravadi <mkarthik@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2419843 Reviewed-by: automaticguardword <automaticguardword@nvidia.com> Reviewed-by: Timo Alho <talho@nvidia.com> Reviewed-by: Bibek Basu <bbasu@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> GVS: Gerrit_Virtual_Submit 
In case of i2c transaction failures due to NACK on bpmp i2c,
print only the return value from BPMP and not the debug
error information. This avoids UART being flooded with
prints in case of NACK from slave.

Bug 3128306

Change-Id: I0869179b48f110a2b1bce7696966ebcae3f8b35b
Signed-off-by: Karthik Mantravadi <mkarthik@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2419843
Reviewed-by: automaticguardword <automaticguardword@nvidia.com>
Reviewed-by: Timo Alho <talho@nvidia.com>
Reviewed-by: Bibek Basu <bbasu@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
GVS: Gerrit_Virtual_Submit
i2c: tegra: disable class based instantiation 2020-03-19T09:55:16+00:00 Ken Chang kenc@nvidia.com 2018-12-17T06:14:12+00:00 e955195aa7a40d2d5fdaa568cca59d0c222b9cfe Set adapter->class as I2C_CLASS_DEPRECATED in bpmp i2c driver to disable the class based instantiation. This will avoid error return from tegra_bpmp_i2c_req() when lm sensor drivers are enabled and the i2c auto detect mechanism gets involved: [ 5.082367] i2c i2c-4: tegra_bpmp_i2c_req ret -5 [ 5.087011] i2c i2c-4: — message dump for debugging — [ 5.092431] i2c i2c-4: addr 0x48 flags 0x0 len 0 data: Bug 2463644 Change-Id: I45bec244e0e86757b3c2e115a6dedf6fab5283e1 Signed-off-by: Ken Chang <kenc@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/1974251 Reviewed-by: Shardar Mohammed <smohammed@nvidia.com> (cherry picked from commit c7c5830a2613dc9c3e5d9216d3978a87f9817865) Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2311499 Reviewed-by: automaticguardword <automaticguardword@nvidia.com> Reviewed-by: Bitan Biswas <bbiswas@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> GVS: Gerrit_Virtual_Submit Tested-by: Ahung Cheng <ahcheng@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
Set adapter->class as I2C_CLASS_DEPRECATED in bpmp i2c driver
to disable the class based instantiation. This will avoid error
return from tegra_bpmp_i2c_req() when lm sensor drivers are
enabled and the i2c auto detect mechanism gets involved:

[ 5.082367] i2c i2c-4: tegra_bpmp_i2c_req ret -5
[ 5.087011] i2c i2c-4: — message dump for debugging —
[ 5.092431] i2c i2c-4: addr 0x48 flags 0x0 len 0 data:

Bug 2463644

Change-Id: I45bec244e0e86757b3c2e115a6dedf6fab5283e1
Signed-off-by: Ken Chang <kenc@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1974251
Reviewed-by: Shardar Mohammed <smohammed@nvidia.com>
(cherry picked from commit c7c5830a2613dc9c3e5d9216d3978a87f9817865)
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2311499
Reviewed-by: automaticguardword <automaticguardword@nvidia.com>
Reviewed-by: Bitan Biswas <bbiswas@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
GVS: Gerrit_Virtual_Submit
Tested-by: Ahung Cheng <ahcheng@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
i2c-ivc: fix memory leak. 2020-03-19T05:00:11+00:00 Achal Verma achalv@nvidia.com 2020-03-16T16:34:55+00:00 7b209c53778ed9f6ad912f1669c12ef371085a29 allocate object when condion for its use meet. This fix coverity issue: 2877309 Bug 2028892 Change-Id: Ib87bc5f9192bd8f7428fff43384e2465da4434d2 Signed-off-by: Achal Verma <achalv@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2313490 Reviewed-by: automaticguardword <automaticguardword@nvidia.com> Reviewed-by: Automatic_Commit_Validation_User Reviewed-by: Dmitry Pervushin <dpervushin@nvidia.com> Reviewed-by: Phoenix Jung <pjung@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> GVS: Gerrit_Virtual_Submit Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
allocate object when condion for
its use meet.

This fix coverity issue: 2877309

Bug 2028892 

Change-Id: Ib87bc5f9192bd8f7428fff43384e2465da4434d2
Signed-off-by: Achal Verma <achalv@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2313490
Reviewed-by: automaticguardword <automaticguardword@nvidia.com>
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Dmitry Pervushin <dpervushin@nvidia.com>
Reviewed-by: Phoenix Jung <pjung@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
GVS: Gerrit_Virtual_Submit
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Avoid deserialize in bpmp_i2c. 2019-06-20T20:01:08+00:00 Achal Verma achalv@nvidia.com 2019-06-17T09:04:55+00:00 cd91d2f2e6f4d18d3e53db133ceeea341819bab2 Dont deserialize in case response/ack is not expected. Bug 2566502 Bug 2522971 Change-Id: Iac0d295c4138ddc715dbb2dac84aab656543529e Signed-off-by: Achal Verma <achalv@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/2137893 Reviewed-by: Automatic_Commit_Validation_User GVS: Gerrit_Virtual_Submit Reviewed-by: Dmitry Pervushin <dpervushin@nvidia.com> Reviewed-by: Rahul Jain (SW-TEGRA) <rahuljain@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
Dont deserialize in case response/ack is not
expected.

Bug 2566502
Bug 2522971

Change-Id: Iac0d295c4138ddc715dbb2dac84aab656543529e
Signed-off-by: Achal Verma <achalv@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/2137893
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Dmitry Pervushin <dpervushin@nvidia.com>
Reviewed-by: Rahul Jain (SW-TEGRA) <rahuljain@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
driver: vii2c: Fix unbalanced pm_runtime_enable 2019-04-11T09:43:37+00:00 Frank Chen frankc@nvidia.com 2019-04-09T18:26:07+00:00 41595d875403a2bf79433ed7697d87c4db012e1b Removing extra pm_runtime_enable call in probe() to fix Unbalanced pm_runtime_enable kernel warnings. Bug 200490028 Change-Id: I295fa2cc2eb5ebd4656e93b5418d1f181ad6c437 Signed-off-by: Frank Chen <frankc@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/2093570 GVS: Gerrit_Virtual_Submit Reviewed-by: Automatic_Commit_Validation_User Reviewed-by: Bhushan Rayrikar <brayrikar@nvidia.com> Reviewed-by: Vincent Chung <vincentc@nvidia.com> Reviewed-by: Ian Kaszubski <ikaszubski@nvidia.com> Reviewed-by: Joshua Widen <jwiden@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
Removing extra pm_runtime_enable call in probe()
to fix Unbalanced pm_runtime_enable kernel warnings.

Bug 200490028

Change-Id: I295fa2cc2eb5ebd4656e93b5418d1f181ad6c437
Signed-off-by: Frank Chen <frankc@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/2093570
GVS: Gerrit_Virtual_Submit
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Bhushan Rayrikar <brayrikar@nvidia.com>
Reviewed-by: Vincent Chung <vincentc@nvidia.com>
Reviewed-by: Ian Kaszubski <ikaszubski@nvidia.com>
Reviewed-by: Joshua Widen <jwiden@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
i2c: tegra-slave: add support for failed transactions 2018-09-10T22:23:48+00:00 mkarthik mkarthik@nvidia.com 2018-09-03T06:03:06+00:00 18b214b4f6940ad18df5d550e6b788131cb713bb In case of failed transactions i.e. tx_in_progress is true, the status received case in isr resets the controller. Move the timeout scheduler to status received, in order to sync the timer to the start of transaction. Bug 200376118 Change-Id: Ic690f35f62784eff3bbc1ffcdb2600dc6a9a0e16 Signed-off-by: mkarthik <mkarthik@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/1811818 Reviewed-by: Shardar Mohammed <smohammed@nvidia.com> Reviewed-by: Bibek Basu <bbasu@nvidia.com> GVS: Gerrit_Virtual_Submit Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
In case of failed transactions i.e. tx_in_progress is true,
the status received case in isr resets the controller.

Move the timeout scheduler to status received, in order to
sync the timer to the start of transaction.

Bug 200376118

Change-Id: Ic690f35f62784eff3bbc1ffcdb2600dc6a9a0e16
Signed-off-by: mkarthik <mkarthik@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1811818
Reviewed-by: Shardar Mohammed <smohammed@nvidia.com>
Reviewed-by: Bibek Basu <bbasu@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
i2c: tegra: use compatible base types 2018-09-05T05:16:18+00:00 kartik kkartik@nvidia.com 2018-09-03T09:34:02+00:00 5f341d3864979e8694b88234f9c43b881484b5d9 __le32 is an unsigned int with bitwise attribute, which is used by sparse utility to make sure that the variable is converted to local processor type. That's why typecasting from unsigned int to __le32 or vice versa is throwing a warning. Bug 200434802 Change-Id: I91f5e2734666c358b1ecb58e4ca99003e15b84bd Signed-off-by: kartik <kkartik@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/1811817 Reviewed-by: Automatic_Commit_Validation_User GVS: Gerrit_Virtual_Submit Reviewed-by: Sachin Nikam <snikam@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
__le32 is an unsigned int with bitwise attribute, which is used by
sparse utility to make sure that the variable is converted to local
processor type. That's why typecasting from unsigned int to __le32
or vice versa is throwing a warning.

Bug 200434802

Change-Id: I91f5e2734666c358b1ecb58e4ca99003e15b84bd
Signed-off-by: kartik <kkartik@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1811817
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Sachin Nikam <snikam@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
i2c: tegra-slave: Add support for timeout during read 2018-08-27T13:22:31+00:00 mkarthik mkarthik@nvidia.com 2018-08-16T09:13:16+00:00 5276e80d92d168243b7c348fa921e7c8d68eccda Add the support for timeout in slave driver during read from slave to master Bug 200376118 Change-Id: I87f93890d4b121b4d966bc20e4fea5dfef84362c Signed-off-by: mkarthik <mkarthik@nvidia.com> Reviewed-on: https://git-master.nvidia.com/r/1800968 Reviewed-by: Shardar Mohammed <smohammed@nvidia.com> GVS: Gerrit_Virtual_Submit Reviewed-by: Bibek Basu <bbasu@nvidia.com> Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com> Tested-by: mobile promotions <svcmobile_promotions@nvidia.com> 
Add the support for timeout in slave driver during read
from slave to master

Bug 200376118

Change-Id: I87f93890d4b121b4d966bc20e4fea5dfef84362c
Signed-off-by: mkarthik <mkarthik@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1800968
Reviewed-by: Shardar Mohammed <smohammed@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bibek Basu <bbasu@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>