Linux kernel stable tree (mirror) https://git.radix-linux.su/kernel/linux.git/atom?h=v6.6.131 2025-10-19T14:30:47+00:00 2025-10-19T14:30:47+00:00 Johan Hovold johan@kernel.org 2025-07-25T07:40:19+00:00 urn:sha1:dbcbac8c47cbb9afe9853c18cad8f9e6336c2548 commit 8ece3173f87df03935906d0c612c2aeda9db92ca upstream. Make sure to drop the reference to the secure monitor device taken by of_find_device_by_node() when looking up its driver data on behalf of other drivers (e.g. during probe). Note that holding a reference to the platform device does not prevent its driver data from going away so there is no point in keeping the reference after the helper returns. Fixes: 8cde3c2153e8 ("firmware: meson_sm: Rework driver as a proper platform driver") Cc: stable@vger.kernel.org # 5.5 Cc: Carlo Caione <ccaione@baylibre.com> Signed-off-by: Johan Hovold <johan@kernel.org> Acked-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> Link: https://lore.kernel.org/r/20250725074019.8765-1-johan@kernel.org Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 2025-10-15T09:57:52+00:00 Johan Hovold johan@kernel.org 2025-07-25T07:54:29+00:00 urn:sha1:aa30d3077298ff59de1d52913c7250ea20be9e72 [ Upstream commit 0454346d1c5f7fccb3ef6e3103985de8ab3469f3 ] Enabling compile testing should not enable every individual driver (we have "allyesconfig" for that). Fixes: 4a434abc40d2 ("firmware: meson-sm: enable build as module") Signed-off-by: Johan Hovold <johan@kernel.org> Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> Link: https://lore.kernel.org/r/20250725075429.10056-1-johan@kernel.org Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> Signed-off-by: Sasha Levin <sashal@kernel.org> 2023-08-04T13:19:01+00:00 Zhang Shurong zhang_shurong@foxmail.com 2023-07-15T14:13:38+00:00 urn:sha1:f2ed165619c16577c02b703a114a1f6b52026df4 of_match_device() may fail and returns a NULL pointer. Fix this by checking the return value of of_match_device. Fixes: 8cde3c2153e8 ("firmware: meson_sm: Rework driver as a proper platform driver") Signed-off-by: Zhang Shurong <zhang_shurong@foxmail.com> Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> Link: https://lore.kernel.org/r/tencent_AA08AAA6C4F34D53ADCE962E188A879B8206@qq.com Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> 2023-03-27T10:07:17+00:00 Dmitry Rokosov ddrokosov@sberdevices.ru 2023-03-24T14:55:57+00:00 urn:sha1:e45f243409db98d610248c843b25435e7fb0baf3 In some meson boards, secure monitor device has children, for example, power secure controller. By default, secure monitor isn't the bus in terms of device tree subsystem, so the of_platform initialization code doesn't populate its device tree data. As a result, secure monitor's children aren't probed at all. Run the 'of_platform_populate()' routine manually to resolve such issues. Signed-off-by: Dmitry Rokosov <ddrokosov@sberdevices.ru> Acked-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> Link: https://lore.kernel.org/r/20230324145557.27797-1-ddrokosov@sberdevices.ru Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> 2023-01-10T14:21:13+00:00 Miles Chen miles.chen@mediatek.com 2023-01-10T03:12:42+00:00 urn:sha1:fcdc53aceed413dce3ef1d0166a8e9348d5a9c94 Use NULL for NULL pointer to fix the following sparse warning: drivers/firmware/meson/meson_sm.c:85:24: sparse: warning: Using plain integer as NULL pointer Signed-off-by: Miles Chen <miles.chen@mediatek.com> Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org> Link: https://lore.kernel.org/r/20230110031242.4917-1-miles.chen@mediatek.com Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> 2020-10-26T20:36:05+00:00 Kevin Hilman khilman@baylibre.com 2020-09-21T22:11:19+00:00 urn:sha1:4a434abc40d2a0c15032f88e30aeb7ad271ba795 Enable secure module driver as module. Default remains built-in. Signed-off-by: Kevin Hilman <khilman@baylibre.com> 2020-02-14T19:37:34+00:00 Jianxin Pan jianxin.pan@amlogic.com 2020-01-15T11:30:28+00:00 urn:sha1:fe98d0ff5d5c43ee179e801275bb37641d398c6e The Amlogic Meson A1/C1 Secure Monitor implements calls to control power domain. Signed-off-by: Jianxin Pan <jianxin.pan@amlogic.com> Signed-off-by: Kevin Hilman <khilman@baylibre.com> Link: https://lore.kernel.org/r/1579087831-94965-2-git-send-email-jianxin.pan@amlogic.com 2019-10-03T15:43:33+00:00 Andy Shevchenko andriy.shevchenko@linux.intel.com 2019-09-04T17:48:35+00:00 urn:sha1:9be579f4c41f69c42278980af73e3b201f4159bb Use %*ph format to print small buffer as hex string. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Kevin Hilman <khilman@baylibre.com> 2019-10-03T15:43:33+00:00 Carlo Caione ccaione@baylibre.com 2019-07-31T08:23:39+00:00 urn:sha1:8cde3c2153e8f57be884c0e73f18bc4de150e870 The secure monitor driver is currently a frankenstein driver which is registered as a platform driver but its functionality goes through a global struct accessed by the consumer drivers using exported helper functions. Try to tidy up the driver moving the firmware struct into the driver data and make the consumer drivers referencing the secure-monitor using a new property in the DT. Currently only the nvmem driver is using this API so we can fix it in the same commit. Reviewed-by: Jerome Brunet <jbrunet@baylibre.com> Signed-off-by: Carlo Caione <ccaione@baylibre.com> Signed-off-by: Kevin Hilman <khilman@baylibre.com> 2019-10-03T15:43:33+00:00 Carlo Caione ccaione@baylibre.com 2019-07-31T08:23:36+00:00 urn:sha1:beb91681a20abd8db7cfc177cb621864ce4e7967 No need to be a global struct. Reviewed-by: Jerome Brunet <jbrunet@baylibre.com> Signed-off-by: Carlo Caione <ccaione@baylibre.com> Signed-off-by: Kevin Hilman <khilman@baylibre.com>