rpmsg: glink_native: fix leak and cleanup

[PATCH 0/2] rpmsg: glink_native: fix leak and cleanup

Posted by srinivas.kandagatla@oss.qualcomm.com 5 months, 2 weeks ago

From: Srinivas Kandagatla <srinivas.kandagatla@oss.qualcomm.com>

This series fixes a rpmsg_device leak in glink_native, and also adds
helper function to remove some code duplication.

Am not 100% sure if this behaviour was intentional and not allow
rpmsg-char interface to work from glink_native, but by the looks
of the code it looks buggy, which is why am sending this series for
discussion.

Srinivas Kandagatla (2):
  rpmsg: glink_native: fix rpmsg device leak
  rpmsg: glink_native: remove duplicate code for rpmsg device remove

 drivers/rpmsg/qcom_glink_native.c | 35 ++++++++++++++-----------------
 1 file changed, 16 insertions(+), 19 deletions(-)

-- 
2.50.0

Re: [PATCH 0/2] rpmsg: glink_native: fix leak and cleanup

Posted by Bjorn Andersson 2 months, 2 weeks ago

On Fri, 22 Aug 2025 11:00:41 +0100, srinivas.kandagatla@oss.qualcomm.com wrote:
> From: Srinivas Kandagatla <srinivas.kandagatla@oss.qualcomm.com>
> 
> This series fixes a rpmsg_device leak in glink_native, and also adds
> helper function to remove some code duplication.
> 
> Am not 100% sure if this behaviour was intentional and not allow
> rpmsg-char interface to work from glink_native, but by the looks
> of the code it looks buggy, which is why am sending this series for
> discussion.
> 
> [...]

Applied, thanks!

[1/2] rpmsg: glink_native: fix rpmsg device leak
      commit: a53e356df548f6b0e82529ef3cc6070f42622189
[2/2] rpmsg: glink_native: remove duplicate code for rpmsg device remove
      commit: 112766cdf2e5ea0a0f72b0304d57a6f74c066670

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>