* [PATCH] rpmsg: core: add support to power domains for devices @ 2018-06-15 9:59 Srinivas Kandagatla 2018-09-25 0:25 ` Suman Anna 0 siblings, 1 reply; 9+ messages in thread From: Srinivas Kandagatla @ 2018-06-15 9:59 UTC (permalink / raw) To: ohad, bjorn.andersson Cc: linux-remoteproc, linux-arm-msm, linux-kernel, Srinivas Kandagatla Some of the rpmsg devices need to switch on power domains to communicate with remote processor. For example on Qualcomm DB820c platform LPASS power domain needs to switched on for any kind of audio services. This patch adds the missing power domain support in rpmsg core. Without this patch attempting to play audio via QDSP on DB820c would reboot the system. Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> --- drivers/rpmsg/rpmsg_core.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/rpmsg/rpmsg_core.c b/drivers/rpmsg/rpmsg_core.c index b714a543a91d..8122807db380 100644 --- a/drivers/rpmsg/rpmsg_core.c +++ b/drivers/rpmsg/rpmsg_core.c @@ -15,6 +15,7 @@ #include <linux/module.h> #include <linux/rpmsg.h> #include <linux/of_device.h> +#include <linux/pm_domain.h> #include <linux/slab.h> #include "rpmsg_internal.h" @@ -449,6 +450,10 @@ static int rpmsg_dev_probe(struct device *dev) struct rpmsg_endpoint *ept = NULL; int err; + err = dev_pm_domain_attach(dev, true); + if (err) + goto out; + if (rpdrv->callback) { strncpy(chinfo.name, rpdev->id.name, RPMSG_NAME_SIZE); chinfo.src = rpdev->src; @@ -490,6 +495,8 @@ static int rpmsg_dev_remove(struct device *dev) rpdrv->remove(rpdev); + dev_pm_domain_detach(dev, true); + if (rpdev->ept) rpmsg_destroy_ept(rpdev->ept); -- 2.16.2 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] rpmsg: core: add support to power domains for devices 2018-06-15 9:59 [PATCH] rpmsg: core: add support to power domains for devices Srinivas Kandagatla @ 2018-09-25 0:25 ` Suman Anna 2018-09-25 8:29 ` Srinivas Kandagatla 2018-09-25 17:25 ` Arnaud Pouliquen 0 siblings, 2 replies; 9+ messages in thread From: Suman Anna @ 2018-09-25 0:25 UTC (permalink / raw) To: Srinivas Kandagatla, ohad, bjorn.andersson Cc: linux-remoteproc, linux-arm-msm, linux-kernel, Loic Pallardy, Arnaud POULIQUEN Hi Srinivas, On 06/15/2018 04:59 AM, Srinivas Kandagatla wrote: > Some of the rpmsg devices need to switch on power domains to communicate > with remote processor. For example on Qualcomm DB820c platform LPASS > power domain needs to switched on for any kind of audio services. > This patch adds the missing power domain support in rpmsg core. > > Without this patch attempting to play audio via QDSP on DB820c would > reboot the system. > > Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> > --- > drivers/rpmsg/rpmsg_core.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/rpmsg/rpmsg_core.c b/drivers/rpmsg/rpmsg_core.c > index b714a543a91d..8122807db380 100644 > --- a/drivers/rpmsg/rpmsg_core.c > +++ b/drivers/rpmsg/rpmsg_core.c > @@ -15,6 +15,7 @@ > #include <linux/module.h> > #include <linux/rpmsg.h> > #include <linux/of_device.h> > +#include <linux/pm_domain.h> > #include <linux/slab.h> > > #include "rpmsg_internal.h" > @@ -449,6 +450,10 @@ static int rpmsg_dev_probe(struct device *dev) > struct rpmsg_endpoint *ept = NULL; > int err; > > + err = dev_pm_domain_attach(dev, true); > + if (err) > + goto out; This patch has broken the virtio-rpmsg stack based rpmsg devices. These devices are non-DT and the rpmsg_dev_probe() is now failing with -19 (-ENODEV) error code. Loic, Arnaud, Can one of you double-confirm this behavior on ST platforms as well? The patch came through 4.14.71 stable release and broke our downstream kernels. regards Suman > + > if (rpdrv->callback) { > strncpy(chinfo.name, rpdev->id.name, RPMSG_NAME_SIZE); > chinfo.src = rpdev->src; > @@ -490,6 +495,8 @@ static int rpmsg_dev_remove(struct device *dev) > > rpdrv->remove(rpdev); > > + dev_pm_domain_detach(dev, true); > + > if (rpdev->ept) > rpmsg_destroy_ept(rpdev->ept); > > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] rpmsg: core: add support to power domains for devices 2018-09-25 0:25 ` Suman Anna @ 2018-09-25 8:29 ` Srinivas Kandagatla 2018-09-25 17:20 ` Suman Anna 2018-09-25 17:25 ` Arnaud Pouliquen 1 sibling, 1 reply; 9+ messages in thread From: Srinivas Kandagatla @ 2018-09-25 8:29 UTC (permalink / raw) To: Suman Anna, ohad, bjorn.andersson Cc: linux-remoteproc, linux-arm-msm, linux-kernel, Loic Pallardy, Arnaud POULIQUEN Hi Suman, On 25/09/18 01:25, Suman Anna wrote: > Hi Srinivas, > > On 06/15/2018 04:59 AM, Srinivas Kandagatla wrote: >> Some of the rpmsg devices need to switch on power domains to communicate >> with remote processor. For example on Qualcomm DB820c platform LPASS >> power domain needs to switched on for any kind of audio services. >> This patch adds the missing power domain support in rpmsg core. >> >> Without this patch attempting to play audio via QDSP on DB820c would >> reboot the system. >> >> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >> --- >> drivers/rpmsg/rpmsg_core.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/rpmsg/rpmsg_core.c b/drivers/rpmsg/rpmsg_core.c >> index b714a543a91d..8122807db380 100644 >> --- a/drivers/rpmsg/rpmsg_core.c >> +++ b/drivers/rpmsg/rpmsg_core.c >> @@ -15,6 +15,7 @@ >> #include <linux/module.h> >> #include <linux/rpmsg.h> >> #include <linux/of_device.h> >> +#include <linux/pm_domain.h> >> #include <linux/slab.h> >> >> #include "rpmsg_internal.h" >> @@ -449,6 +450,10 @@ static int rpmsg_dev_probe(struct device *dev) >> struct rpmsg_endpoint *ept = NULL; >> int err; >> >> + err = dev_pm_domain_attach(dev, true); >> + if (err) >> + goto out; > > This patch has broken the virtio-rpmsg stack based rpmsg devices. These > devices are non-DT and the rpmsg_dev_probe() is now failing with -19 > (-ENODEV) error code. > There seems to be some dependency on this patch ( https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v4.19-rc5&id=919b7308fcc452cd4e282bab389c33384a9f3790 ) which seems to rework the return values from acpi_dev_pm_attach() and genpd_dev_pm_attach() --srini > Loic, Arnaud, > Can one of you double-confirm this behavior on ST platforms as well? > The patch came through 4.14.71 stable release and broke our downstream > kernels. > > regards > Suman > >> + >> if (rpdrv->callback) { >> strncpy(chinfo.name, rpdev->id.name, RPMSG_NAME_SIZE); >> chinfo.src = rpdev->src; >> @@ -490,6 +495,8 @@ static int rpmsg_dev_remove(struct device *dev) >> >> rpdrv->remove(rpdev); >> >> + dev_pm_domain_detach(dev, true); >> + >> if (rpdev->ept) >> rpmsg_destroy_ept(rpdev->ept); >> >> > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] rpmsg: core: add support to power domains for devices 2018-09-25 8:29 ` Srinivas Kandagatla @ 2018-09-25 17:20 ` Suman Anna 0 siblings, 0 replies; 9+ messages in thread From: Suman Anna @ 2018-09-25 17:20 UTC (permalink / raw) To: Srinivas Kandagatla, ohad, bjorn.andersson Cc: linux-remoteproc, linux-arm-msm, linux-kernel, Loic Pallardy, Arnaud POULIQUEN, stable, Sasha Levin, Greg Kroah-Hartman Hi Srini, On 09/25/2018 03:29 AM, Srinivas Kandagatla wrote: > Hi Suman, > > On 25/09/18 01:25, Suman Anna wrote: >> Hi Srinivas, >> >> On 06/15/2018 04:59 AM, Srinivas Kandagatla wrote: >>> Some of the rpmsg devices need to switch on power domains to communicate >>> with remote processor. For example on Qualcomm DB820c platform LPASS >>> power domain needs to switched on for any kind of audio services. >>> This patch adds the missing power domain support in rpmsg core. >>> >>> Without this patch attempting to play audio via QDSP on DB820c would >>> reboot the system. >>> >>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >>> --- >>> drivers/rpmsg/rpmsg_core.c | 7 +++++++ >>> 1 file changed, 7 insertions(+) >>> >>> diff --git a/drivers/rpmsg/rpmsg_core.c b/drivers/rpmsg/rpmsg_core.c >>> index b714a543a91d..8122807db380 100644 >>> --- a/drivers/rpmsg/rpmsg_core.c >>> +++ b/drivers/rpmsg/rpmsg_core.c >>> @@ -15,6 +15,7 @@ >>> #include <linux/module.h> >>> #include <linux/rpmsg.h> >>> #include <linux/of_device.h> >>> +#include <linux/pm_domain.h> >>> #include <linux/slab.h> >>> #include "rpmsg_internal.h" >>> @@ -449,6 +450,10 @@ static int rpmsg_dev_probe(struct device *dev) >>> struct rpmsg_endpoint *ept = NULL; >>> int err; >>> + err = dev_pm_domain_attach(dev, true); >>> + if (err) >>> + goto out; >> >> This patch has broken the virtio-rpmsg stack based rpmsg devices. These >> devices are non-DT and the rpmsg_dev_probe() is now failing with -19 >> (-ENODEV) error code. >> > > There seems to be some dependency on this patch ( > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v4.19-rc5&id=919b7308fcc452cd4e282bab389c33384a9f3790 > ) which seems to rework the return values from acpi_dev_pm_attach() and > genpd_dev_pm_attach() > OK, I see that dependent commit is part of the 4.18 Linux kernel. So mainline kernel has no issues, but this patch was picked up in 4.14.71 (don't know why) as commit 1ed3a9307230, so 4.14 stable is broken. So, how do we proceed here - don't think we can pick all those dependent patches against stable. regards Suman > > > --srini > > >> Loic, Arnaud, >> Can one of you double-confirm this behavior on ST platforms as well? >> The patch came through 4.14.71 stable release and broke our downstream >> kernels. >> >> regards >> Suman >> >>> + >>> if (rpdrv->callback) { >>> strncpy(chinfo.name, rpdev->id.name, RPMSG_NAME_SIZE); >>> chinfo.src = rpdev->src; >>> @@ -490,6 +495,8 @@ static int rpmsg_dev_remove(struct device *dev) >>> rpdrv->remove(rpdev); >>> + dev_pm_domain_detach(dev, true); >>> + >>> if (rpdev->ept) >>> rpmsg_destroy_ept(rpdev->ept); >>> >> ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] rpmsg: core: add support to power domains for devices 2018-09-25 0:25 ` Suman Anna 2018-09-25 8:29 ` Srinivas Kandagatla @ 2018-09-25 17:25 ` Arnaud Pouliquen 2018-09-25 19:02 ` Suman Anna 1 sibling, 1 reply; 9+ messages in thread From: Arnaud Pouliquen @ 2018-09-25 17:25 UTC (permalink / raw) To: Suman Anna, Srinivas Kandagatla, ohad, bjorn.andersson Cc: linux-remoteproc, linux-arm-msm, linux-kernel, Loic Pallardy Hi Suman On 09/25/2018 02:25 AM, Suman Anna wrote: > Hi Srinivas, > > On 06/15/2018 04:59 AM, Srinivas Kandagatla wrote: >> Some of the rpmsg devices need to switch on power domains to communicate >> with remote processor. For example on Qualcomm DB820c platform LPASS >> power domain needs to switched on for any kind of audio services. >> This patch adds the missing power domain support in rpmsg core. >> >> Without this patch attempting to play audio via QDSP on DB820c would >> reboot the system. >> >> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >> --- >> drivers/rpmsg/rpmsg_core.c | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/drivers/rpmsg/rpmsg_core.c b/drivers/rpmsg/rpmsg_core.c >> index b714a543a91d..8122807db380 100644 >> --- a/drivers/rpmsg/rpmsg_core.c >> +++ b/drivers/rpmsg/rpmsg_core.c >> @@ -15,6 +15,7 @@ >> #include <linux/module.h> >> #include <linux/rpmsg.h> >> #include <linux/of_device.h> >> +#include <linux/pm_domain.h> >> #include <linux/slab.h> >> >> #include "rpmsg_internal.h" >> @@ -449,6 +450,10 @@ static int rpmsg_dev_probe(struct device *dev) >> struct rpmsg_endpoint *ept = NULL; >> int err; >> >> + err = dev_pm_domain_attach(dev, true); >> + if (err) >> + goto out; > > This patch has broken the virtio-rpmsg stack based rpmsg devices. These > devices are non-DT and the rpmsg_dev_probe() is now failing with -19 > (-ENODEV) error code. > > Loic, Arnaud, > Can one of you double-confirm this behavior on ST platforms as well? > The patch came through 4.14.71 stable release and broke our downstream > kernels. I do not reproduce issue on 4.14.0 + your patch, on my ST platform (not able to test on the LTS branch). By looking the source code, seems that your issue is related to genpd_dev_pm_attach that returns -ENODEV because you have no remoteproc device node... Do you have a rproc node defined in your DT? we have one. Regards Arnaud > > regards > Suman > >> + >> if (rpdrv->callback) { >> strncpy(chinfo.name, rpdev->id.name, RPMSG_NAME_SIZE); >> chinfo.src = rpdev->src; >> @@ -490,6 +495,8 @@ static int rpmsg_dev_remove(struct device *dev) >> >> rpdrv->remove(rpdev); >> >> + dev_pm_domain_detach(dev, true); >> + >> if (rpdev->ept) >> rpmsg_destroy_ept(rpdev->ept); >> >> > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] rpmsg: core: add support to power domains for devices 2018-09-25 17:25 ` Arnaud Pouliquen @ 2018-09-25 19:02 ` Suman Anna 2018-09-26 12:48 ` Arnaud Pouliquen 0 siblings, 1 reply; 9+ messages in thread From: Suman Anna @ 2018-09-25 19:02 UTC (permalink / raw) To: Arnaud Pouliquen, Srinivas Kandagatla, ohad, bjorn.andersson Cc: linux-remoteproc, linux-arm-msm, linux-kernel, Loic Pallardy Hi Arnaud, > On 09/25/2018 02:25 AM, Suman Anna wrote: >> Hi Srinivas, >> >> On 06/15/2018 04:59 AM, Srinivas Kandagatla wrote: >>> Some of the rpmsg devices need to switch on power domains to communicate >>> with remote processor. For example on Qualcomm DB820c platform LPASS >>> power domain needs to switched on for any kind of audio services. >>> This patch adds the missing power domain support in rpmsg core. >>> >>> Without this patch attempting to play audio via QDSP on DB820c would >>> reboot the system. >>> >>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >>> --- >>> drivers/rpmsg/rpmsg_core.c | 7 +++++++ >>> 1 file changed, 7 insertions(+) >>> >>> diff --git a/drivers/rpmsg/rpmsg_core.c b/drivers/rpmsg/rpmsg_core.c >>> index b714a543a91d..8122807db380 100644 >>> --- a/drivers/rpmsg/rpmsg_core.c >>> +++ b/drivers/rpmsg/rpmsg_core.c >>> @@ -15,6 +15,7 @@ >>> #include <linux/module.h> >>> #include <linux/rpmsg.h> >>> #include <linux/of_device.h> >>> +#include <linux/pm_domain.h> >>> #include <linux/slab.h> >>> >>> #include "rpmsg_internal.h" >>> @@ -449,6 +450,10 @@ static int rpmsg_dev_probe(struct device *dev) >>> struct rpmsg_endpoint *ept = NULL; >>> int err; >>> >>> + err = dev_pm_domain_attach(dev, true); >>> + if (err) >>> + goto out; >> >> This patch has broken the virtio-rpmsg stack based rpmsg devices. These >> devices are non-DT and the rpmsg_dev_probe() is now failing with -19 >> (-ENODEV) error code. >> >> Loic, Arnaud, >> Can one of you double-confirm this behavior on ST platforms as well? >> The patch came through 4.14.71 stable release and broke our downstream >> kernels. > I do not reproduce issue on 4.14.0 + your patch, on my ST platform (not > able to test on the LTS branch). > By looking the source code, seems that your issue is related to > genpd_dev_pm_attach that returns -ENODEV because you have > no remoteproc device node... > Do you have a rproc node defined in your DT? we have one. The dev here is not the remoteproc dev but the rpmsg device which does not have an OF node, and so I get the ENODEV due to the check on dev->of_node in genpd_dev_pm_attach. I do have DT remoteproc nodes. Are you publishing an rpmsg device from the remote side and are they being probed? Issue is seen only during the probe phase, you will not see any errors if the rpmsg devices are just being published and created. regards Suman > > Regards > Arnaud > >> >> regards >> Suman >> >>> + >>> if (rpdrv->callback) { >>> strncpy(chinfo.name, rpdev->id.name, RPMSG_NAME_SIZE); >>> chinfo.src = rpdev->src; >>> @@ -490,6 +495,8 @@ static int rpmsg_dev_remove(struct device *dev) >>> >>> rpdrv->remove(rpdev); >>> >>> + dev_pm_domain_detach(dev, true); >>> + >>> if (rpdev->ept) >>> rpmsg_destroy_ept(rpdev->ept); >>> >>> >> ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] rpmsg: core: add support to power domains for devices 2018-09-25 19:02 ` Suman Anna @ 2018-09-26 12:48 ` Arnaud Pouliquen 2018-09-26 13:06 ` Srinivas Kandagatla 0 siblings, 1 reply; 9+ messages in thread From: Arnaud Pouliquen @ 2018-09-26 12:48 UTC (permalink / raw) To: Suman Anna, Srinivas Kandagatla, ohad, bjorn.andersson Cc: linux-remoteproc, linux-arm-msm, linux-kernel, Loic Pallardy Hi Suman, On 09/25/2018 09:02 PM, Suman Anna wrote: > Hi Arnaud, > >> On 09/25/2018 02:25 AM, Suman Anna wrote: >>> Hi Srinivas, >>> >>> On 06/15/2018 04:59 AM, Srinivas Kandagatla wrote: >>>> Some of the rpmsg devices need to switch on power domains to communicate >>>> with remote processor. For example on Qualcomm DB820c platform LPASS >>>> power domain needs to switched on for any kind of audio services. >>>> This patch adds the missing power domain support in rpmsg core. >>>> >>>> Without this patch attempting to play audio via QDSP on DB820c would >>>> reboot the system. >>>> >>>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> >>>> --- >>>> drivers/rpmsg/rpmsg_core.c | 7 +++++++ >>>> 1 file changed, 7 insertions(+) >>>> >>>> diff --git a/drivers/rpmsg/rpmsg_core.c b/drivers/rpmsg/rpmsg_core.c >>>> index b714a543a91d..8122807db380 100644 >>>> --- a/drivers/rpmsg/rpmsg_core.c >>>> +++ b/drivers/rpmsg/rpmsg_core.c >>>> @@ -15,6 +15,7 @@ >>>> #include <linux/module.h> >>>> #include <linux/rpmsg.h> >>>> #include <linux/of_device.h> >>>> +#include <linux/pm_domain.h> >>>> #include <linux/slab.h> >>>> >>>> #include "rpmsg_internal.h" >>>> @@ -449,6 +450,10 @@ static int rpmsg_dev_probe(struct device *dev) >>>> struct rpmsg_endpoint *ept = NULL; >>>> int err; >>>> >>>> + err = dev_pm_domain_attach(dev, true); >>>> + if (err) >>>> + goto out; >>> >>> This patch has broken the virtio-rpmsg stack based rpmsg devices. These >>> devices are non-DT and the rpmsg_dev_probe() is now failing with -19 >>> (-ENODEV) error code. >>> >>> Loic, Arnaud, >>> Can one of you double-confirm this behavior on ST platforms as well? >>> The patch came through 4.14.71 stable release and broke our downstream >>> kernels. > >> I do not reproduce issue on 4.14.0 + your patch, on my ST platform (not >> able to test on the LTS branch). >> By looking the source code, seems that your issue is related to >> genpd_dev_pm_attach that returns -ENODEV because you have >> no remoteproc device node... >> Do you have a rproc node defined in your DT? we have one. > > The dev here is not the remoteproc dev but the rpmsg device which does > not have an OF node, and so I get the ENODEV due to the check on > dev->of_node in genpd_dev_pm_attach. I do have DT remoteproc nodes. > > Are you publishing an rpmsg device from the remote side and are they > being probed? Issue is seen only during the probe phase, you will not > see any errors if the rpmsg devices are just being published and created. Sorry something was wrong in my fw used yesterday for testing...so forget my previous status. I re-tested it and I have the same issue (-ENODEV) with rpmsg device probed on ns announcement rpmsg from remote side. So we confirm the issue. Regards Arnaud > > regards > Suman > >> >> Regards >> Arnaud >> >>> >>> regards >>> Suman >>> >>>> + >>>> if (rpdrv->callback) { >>>> strncpy(chinfo.name, rpdev->id.name, RPMSG_NAME_SIZE); >>>> chinfo.src = rpdev->src; >>>> @@ -490,6 +495,8 @@ static int rpmsg_dev_remove(struct device *dev) >>>> >>>> rpdrv->remove(rpdev); >>>> >>>> + dev_pm_domain_detach(dev, true); >>>> + >>>> if (rpdev->ept) >>>> rpmsg_destroy_ept(rpdev->ept); >>>> >>>> >>> > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] rpmsg: core: add support to power domains for devices 2018-09-26 12:48 ` Arnaud Pouliquen @ 2018-09-26 13:06 ` Srinivas Kandagatla 2018-09-26 17:14 ` Bjorn Andersson 0 siblings, 1 reply; 9+ messages in thread From: Srinivas Kandagatla @ 2018-09-26 13:06 UTC (permalink / raw) To: Arnaud Pouliquen, Suman Anna, ohad, bjorn.andersson Cc: linux-remoteproc, linux-arm-msm, linux-kernel, Loic Pallardy On 26/09/18 13:48, Arnaud Pouliquen wrote: > Sorry something was wrong in my fw used yesterday for testing...so > forget my previous status. > > I re-tested it and I have the same issue (-ENODEV) with rpmsg device > probed on ns announcement rpmsg from remote side. > So we confirm the issue. A revert patch is in the stable queue now for 4.14 and 4.18 stable. http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary Issue reported should be fixed soon!! --srini ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] rpmsg: core: add support to power domains for devices 2018-09-26 13:06 ` Srinivas Kandagatla @ 2018-09-26 17:14 ` Bjorn Andersson 0 siblings, 0 replies; 9+ messages in thread From: Bjorn Andersson @ 2018-09-26 17:14 UTC (permalink / raw) To: Srinivas Kandagatla Cc: Arnaud Pouliquen, Suman Anna, ohad, linux-remoteproc, linux-arm-msm, linux-kernel, Loic Pallardy On Wed 26 Sep 06:06 PDT 2018, Srinivas Kandagatla wrote: > On 26/09/18 13:48, Arnaud Pouliquen wrote: > > Sorry something was wrong in my fw used yesterday for testing...so > > forget my previous status. > > > > I re-tested it and I have the same issue (-ENODEV) with rpmsg device > > probed on ns announcement rpmsg from remote side. > > So we confirm the issue. > > > A revert patch is in the stable queue now for 4.14 and 4.18 stable. > http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary > > Issue reported should be fixed soon!! > Thanks guys! Regards, Bjorn ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-09-26 17:12 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-06-15 9:59 [PATCH] rpmsg: core: add support to power domains for devices Srinivas Kandagatla 2018-09-25 0:25 ` Suman Anna 2018-09-25 8:29 ` Srinivas Kandagatla 2018-09-25 17:20 ` Suman Anna 2018-09-25 17:25 ` Arnaud Pouliquen 2018-09-25 19:02 ` Suman Anna 2018-09-26 12:48 ` Arnaud Pouliquen 2018-09-26 13:06 ` Srinivas Kandagatla 2018-09-26 17:14 ` Bjorn Andersson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).