linux-stable.git/drivers/reset, branch linux-4.9.y

reset: ti-syscon: fix to_ti_syscon_reset_data macro

2021-07-28T07:14:24+00:00

[ Upstream commit 05cf8fffcdeb47aef1203c08cbec5224fd3a0e1c ]

The to_ti_syscon_reset_data macro currently only works if the
parameter passed into it is called 'rcdev'.

Fixes a checkpatch --strict issue:

  CHECK: Macro argument reuse 'rcdev' - possible side-effects?
  #53: FILE: drivers/reset/reset-ti-syscon.c:53:
  +#define to_ti_syscon_reset_data(rcdev)	\
  +	container_of(rcdev, struct ti_syscon_reset_data, rcdev)

Signed-off-by: Philipp Zabel 
Signed-off-by: Sasha Levin

reset: bail if try_module_get() fails

2021-07-20T14:21:15+00:00

[ Upstream commit 4fb26fb83f0def3d39c14e268bcd4003aae8fade ]

Abort instead of returning a new reset control for a reset controller
device that is going to have its module unloaded.

Reported-by: Uwe Kleine-König 
Fixes: 61fc41317666 ("reset: Add reset controller API")
Acked-by: Uwe Kleine-König 
Link: https://lore.kernel.org/r/20210607082615.15160-1-p.zabel@pengutronix.de
Signed-off-by: Philipp Zabel 
Signed-off-by: Sasha Levin

reset: Fix potential use-after-free in __of_reset_control_get()

2019-11-25T08:53:42+00:00

[ Upstream commit b790c8ea5593d6dc3580adfad8e117eeb56af874 ]

Calling of_node_put() decreases the reference count of a device tree
object, and may free some data.

However, the of_phandle_args structure embedding it is passed to
reset_controller_dev.of_xlate() after that, so it may still be accessed.

Move the call to of_node_put() down to fix this.

Signed-off-by: Geert Uytterhoeven 
[p.zabel@pengutronix.de: moved of_node_put after mutex_unlock]
Signed-off-by: Philipp Zabel 
Signed-off-by: Sasha Levin

reset: make device_reset_optional() really optional

2018-12-08T12:05:08+00:00

commit 1554bbd4ad401b7f0f916c0891874111c10befe5 upstream.

Commit bb475230b8e5 ("reset: make optional functions really optional")
converted *_get_optional* functions, but device_reset_optional() was
left behind.  Convert it in the same way.

Signed-off-by: Masahiro Yamada 
Signed-off-by: Philipp Zabel 
Cc: Dinh Nguyen 
Signed-off-by: Greg Kroah-Hartman

reset: add exported __reset_control_get, return NULL if optional

2018-12-08T12:05:08+00:00

commit 62e24c5775ecb387a3eb33701378ccfa6dbc98ee upstream.

Rename the internal __reset_control_get/put functions to
__reset_control_get/put_internal and add an exported
__reset_control_get equivalent to __of_reset_control_get
that takes a struct device parameter.
This avoids the confusing call to __of_reset_control_get in
the non-DT case and fixes the devm_reset_control_get_optional
function to return NULL if RESET_CONTROLLER is enabled but
dev->of_node == NULL.

Fixes: bb475230b8e5 ("reset: make optional functions really optional")
Reported-by: Andy Shevchenko 
Tested-by: Andy Shevchenko 
Cc: Ramiro Oliveira 
Signed-off-by: Philipp Zabel 
Cc: Dinh Nguyen 
Signed-off-by: Greg Kroah-Hartman

reset: core: fix reset_control_put

2018-12-08T12:05:08+00:00

commit 4891486fb2c80eaf3bb0f9eb065d15ecd357702f upstream.

Commit "reset: make optional functions really optional" missed to
adjust one check in reset_control_put, causing a NULL pointer
access for optional resets.

Fixes: bb475230b8e5 "reset: make optional functions really optional"
Signed-off-by: Heiner Kallweit 
Signed-off-by: Philipp Zabel 
Cc: Dinh Nguyen 
Signed-off-by: Greg Kroah-Hartman

reset: make optional functions really optional

2018-12-08T12:05:08+00:00

commit bb475230b8e59a547ab66ac3b02572df21a580e9 upstream.

The *_get_optional_* functions weren't really optional so this patch
makes them really optional.

These *_get_optional_* functions will now return NULL instead of an error
if no matching reset phandle is found in the DT, and all the
reset_control_* functions now accept NULL rstc pointers.

Signed-off-by: Ramiro Oliveira 
Signed-off-by: Philipp Zabel 
Cc: Dinh Nguyen 
Signed-off-by: Greg Kroah-Hartman

reset: hi6220: Set module license so that it can be loaded

2018-11-10T15:42:47+00:00

[ Upstream commit 4497a224f759cd8350e07382307b55f870ef0df2 ]

The hi6220_reset driver can be built as a standalone module
yet it cannot be loaded because it depends on GPL exported symbols.

Lets set the module license so that the module loads, and things like
the on-board kirin drm starts working.

Signed-off-by: Jeremy Linton 
Reviewed-by: Xinliang Liu 
Signed-off-by: Philipp Zabel 
Signed-off-by: Sasha Levin

reset: ti_syscon: fix a ti_syscon_reset_status issue

2017-10-08T08:26:03+00:00

[ Upstream commit 5987b4bf512101137fa60c5c0ccac3db51541221 ]

If STATUS_SET was not set, ti_syscon_reset_status would always return 0
no matter whether the status_bit was set or not.

Signed-off-by: Jiancheng Xue 
Fixes: cc7c2bb1493c ("reset: add TI SYSCON based reset driver")
Signed-off-by: Philipp Zabel 
Signed-off-by: Sasha Levin 
Signed-off-by: Greg Kroah-Hartman

reset: uniphier: rename MIO reset to SD reset for Pro5, PXs2, LD20 SoCs

2016-10-22T09:31:42+00:00

I made a mistake as for naming for this block.  The MIO block is not
implemented for these 3 SoCs in the first place.  The current naming
will be a trouble if an SoC with both MIO and SD-ctrl blocks appear
in the future.

This driver has just been merged in the previous merge window.
Rename it before the release.

Signed-off-by: Masahiro Yamada 
Acked-by: Philipp Zabel