linux-toradex.git/drivers/slimbus, branch v5.9-rc5

slimbus: core: Fix mismatch in of_node_get/put

2020-05-15T14:21:08+00:00

Adding missing corresponding of_node_put

Fixes: 7588a511bdb4 ("slimbus: core: add support to device tree helper")
Signed-off-by: Saravana Kannan 
[Srini: added fixes tag, removed NULL check and updated log]
Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20200511151334.362-3-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman

slimbus: core: Set fwnode for a device when setting of_node

2020-05-15T14:21:07+00:00

When setting the of_node for a newly created device, also set the
fwnode. This allows fw_devlink feature to work for slimbus devices.

Also, remove some unnecessary NULL checks. The functions in question
already do NULL checks.

Signed-off-by: Saravana Kannan 
[Srini: removed unnecessary NULL check from other patch]
Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20200511151334.362-2-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman

slimbus: ngd: remove redundant assignment

2020-04-28T13:02:47+00:00

remove redundant assignment, as this is already done fewlines lines before.

Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20200417093914.22052-1-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman

slimbus: ngd: get drvdata from correct device

2020-04-28T13:02:47+00:00

Get drvdata directly from parent instead of ngd dev, as ngd
dev can probe defer and previously set drvdata will become null.

Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20200417093618.7929-1-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman

slimbus: ngd: add v2.1.0 compatible

2020-03-12T15:51:15+00:00

This patch adds compatible for SlimBus Controller on SDM845.

Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20200312152510.12224-1-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman

slimbus: qcom: add missed clk_disable_unprepare in remove

2020-01-14T20:46:48+00:00

The remove misses to disable and unprepare rclk and hclk.
Add calls to clk_disable_unprepare to fix it.

Signed-off-by: Chuhong Yuan 
Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20200109103148.5612-5-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman

slimbus: Use the correct style for SPDX License Identifier

2020-01-14T20:46:47+00:00

This patch corrects the SPDX License Identifier style in
header file related to SLIMbus driver.
For C header files Documentation/process/license-rules.rst
mandates C-like comments (opposed to C source files where
C++ style should be used).

Changes made by using a script provided by Joe Perches here:
https://lkml.org/lkml/2019/2/7/46.

Suggested-by: Joe Perches 
Signed-off-by: Nishad Kamdar 
Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20200109103148.5612-4-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman

slimbus: qcom-ngd-ctrl: Use dma_request_chan() instead dma_request_slave_channel()

2020-01-14T20:46:47+00:00

dma_request_slave_channel() is a wrapper on top of dma_request_chan()
eating up the error code.

By using dma_request_chan() directly the driver can support deferred
probing against DMA.

Signed-off-by: Peter Ujfalusi 
Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20200109103148.5612-3-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman

slimbus: fix slim_tid_txn()

2019-08-18T10:56:10+00:00

fix below issue reported by coccicheck
./drivers/slimbus/slimbus.h:440:3-46: duplicated argument to && or ||

Looks like this was a typo, SLIM_MSG_MC_REQUEST_CHANGE_VALUE is command
which requires transaction ID, so fix it, this also fix the warning.

Reported-by: Hariprasad Kelam 
Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20190818093902.29993-3-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman

slimbus: qcom-ngd-ctrl: Add of_node_put() before return

2019-08-18T10:56:10+00:00

Each iteration of for_each_available_child_of_node puts the previous
node, but in the case of a return from the middle of the loop, there is
no put, thus causing a memory leak. Hence add an of_node_put before the
return in two places.
Issue found with Coccinelle.

Signed-off-by: Nishka Dasgupta 
Reviewed-by: Bjorn Andersson 
Signed-off-by: Srinivas Kandagatla 
Link: https://lore.kernel.org/r/20190818093902.29993-2-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman