| Message ID | 20200521001006.2725-1-s-anna@ti.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | TI K3 DSP remoteproc driver for C66x DSPs | expand |
Hi Bjorn, On 5/20/20 7:10 PM, Suman Anna wrote: > Hi All, > > The following is v2 of the K3 DSP remoteproc driver supporting the C66x DSPs > on the TI K3 J721E SoCs. The patches are based on the latest commit on the > rproc-next branch, 7dcef3988eed ("remoteproc: Fix an error code in > devm_rproc_alloc()"). I realized I also had the R5F patches on my branch, so the third patch won't apply cleanly (conflict on Makefile). Let me know if you want a new revision posted for you to pick up the series. regards Suman > > v2 includes a new remoteproc core patch (patch 1) that adds an OF helper > for parsing the firmware-name property. This is refactored out to avoid > replicating the code in various remoteproc drivers. Please see the > individual patches for detailed changes. > > The main dependent patches from the previous series are now staged in > rproc-next branch. The only dependency for this series is the common > ti-sci-proc helper between R5 and DSP drivers [1]. Please see the initial > cover-letter [2] for v1 details. > > regards > Suman > > [1] https://patchwork.kernel.org/patch/11456379/ > [2] https://patchwork.kernel.org/cover/11458573/ > > Suman Anna (4): > remoteproc: Introduce rproc_of_parse_firmware() helper > dt-bindings: remoteproc: Add bindings for C66x DSPs on TI K3 SoCs > remoteproc/k3-dsp: Add a remoteproc driver of K3 C66x DSPs > remoteproc/k3-dsp: Add support for L2RAM loading on C66x DSPs > > .../bindings/remoteproc/ti,k3-dsp-rproc.yaml | 190 +++++ > drivers/remoteproc/Kconfig | 13 + > drivers/remoteproc/Makefile | 1 + > drivers/remoteproc/remoteproc_core.c | 23 + > drivers/remoteproc/remoteproc_internal.h | 2 + > drivers/remoteproc/ti_k3_dsp_remoteproc.c | 773 ++++++++++++++++++ > 6 files changed, 1002 insertions(+) > create mode 100644 Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml > create mode 100644 drivers/remoteproc/ti_k3_dsp_remoteproc.c >
On Thu 21 May 11:59 PDT 2020, Suman Anna wrote: > Hi Bjorn, > > On 5/20/20 7:10 PM, Suman Anna wrote: > > Hi All, > > > > The following is v2 of the K3 DSP remoteproc driver supporting the C66x DSPs > > on the TI K3 J721E SoCs. The patches are based on the latest commit on the > > rproc-next branch, 7dcef3988eed ("remoteproc: Fix an error code in > > devm_rproc_alloc()"). > > I realized I also had the R5F patches on my branch, so the third patch won't > apply cleanly (conflict on Makefile). Let me know if you want a new revision > posted for you to pick up the series. > That should be fine, thanks for the heads up! Will give Mathieu a day or two to take a look as well. Regards, Bjorn > regards > Suman > > > > > v2 includes a new remoteproc core patch (patch 1) that adds an OF helper > > for parsing the firmware-name property. This is refactored out to avoid > > replicating the code in various remoteproc drivers. Please see the > > individual patches for detailed changes. > > > > The main dependent patches from the previous series are now staged in > > rproc-next branch. The only dependency for this series is the common > > ti-sci-proc helper between R5 and DSP drivers [1]. Please see the initial > > cover-letter [2] for v1 details. > > > > regards > > Suman > > > > [1] https://patchwork.kernel.org/patch/11456379/ > > [2] https://patchwork.kernel.org/cover/11458573/ > > > > Suman Anna (4): > > remoteproc: Introduce rproc_of_parse_firmware() helper > > dt-bindings: remoteproc: Add bindings for C66x DSPs on TI K3 SoCs > > remoteproc/k3-dsp: Add a remoteproc driver of K3 C66x DSPs > > remoteproc/k3-dsp: Add support for L2RAM loading on C66x DSPs > > > > .../bindings/remoteproc/ti,k3-dsp-rproc.yaml | 190 +++++ > > drivers/remoteproc/Kconfig | 13 + > > drivers/remoteproc/Makefile | 1 + > > drivers/remoteproc/remoteproc_core.c | 23 + > > drivers/remoteproc/remoteproc_internal.h | 2 + > > drivers/remoteproc/ti_k3_dsp_remoteproc.c | 773 ++++++++++++++++++ > > 6 files changed, 1002 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml > > create mode 100644 drivers/remoteproc/ti_k3_dsp_remoteproc.c > > >
Gents, On Thu, May 21, 2020 at 12:01:41PM -0700, Bjorn Andersson wrote: > On Thu 21 May 11:59 PDT 2020, Suman Anna wrote: > > > Hi Bjorn, > > > > On 5/20/20 7:10 PM, Suman Anna wrote: > > > Hi All, > > > > > > The following is v2 of the K3 DSP remoteproc driver supporting the C66x DSPs > > > on the TI K3 J721E SoCs. The patches are based on the latest commit on the > > > rproc-next branch, 7dcef3988eed ("remoteproc: Fix an error code in > > > devm_rproc_alloc()"). > > > > I realized I also had the R5F patches on my branch, so the third patch won't > > apply cleanly (conflict on Makefile). Let me know if you want a new revision > > posted for you to pick up the series. > > > > That should be fine, thanks for the heads up! > > Will give Mathieu a day or two to take a look as well. I don't see having the time to review this set before the middle/end of next week. I also understand we are crunched by time if we want to get this in for the upcoming merge window. If memory serves me well there wasn't anything controversial about this work. Under normal circumstances I'd give it a final look but I trust Suman to have carried out what we agreed on. Bjorn - if you are happy with this set then go ahead and queue it. Thanks, Mathieu > > Regards, > Bjorn > > > regards > > Suman > > > > > > > > v2 includes a new remoteproc core patch (patch 1) that adds an OF helper > > > for parsing the firmware-name property. This is refactored out to avoid > > > replicating the code in various remoteproc drivers. Please see the > > > individual patches for detailed changes. > > > > > > The main dependent patches from the previous series are now staged in > > > rproc-next branch. The only dependency for this series is the common > > > ti-sci-proc helper between R5 and DSP drivers [1]. Please see the initial > > > cover-letter [2] for v1 details. > > > > > > regards > > > Suman > > > > > > [1] https://patchwork.kernel.org/patch/11456379/ > > > [2] https://patchwork.kernel.org/cover/11458573/ > > > > > > Suman Anna (4): > > > remoteproc: Introduce rproc_of_parse_firmware() helper > > > dt-bindings: remoteproc: Add bindings for C66x DSPs on TI K3 SoCs > > > remoteproc/k3-dsp: Add a remoteproc driver of K3 C66x DSPs > > > remoteproc/k3-dsp: Add support for L2RAM loading on C66x DSPs > > > > > > .../bindings/remoteproc/ti,k3-dsp-rproc.yaml | 190 +++++ > > > drivers/remoteproc/Kconfig | 13 + > > > drivers/remoteproc/Makefile | 1 + > > > drivers/remoteproc/remoteproc_core.c | 23 + > > > drivers/remoteproc/remoteproc_internal.h | 2 + > > > drivers/remoteproc/ti_k3_dsp_remoteproc.c | 773 ++++++++++++++++++ > > > 6 files changed, 1002 insertions(+) > > > create mode 100644 Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml > > > create mode 100644 drivers/remoteproc/ti_k3_dsp_remoteproc.c > > > > >
On Thu 21 May 15:23 PDT 2020, Mathieu Poirier wrote: > Gents, > > On Thu, May 21, 2020 at 12:01:41PM -0700, Bjorn Andersson wrote: > > On Thu 21 May 11:59 PDT 2020, Suman Anna wrote: > > > > > Hi Bjorn, > > > > > > On 5/20/20 7:10 PM, Suman Anna wrote: > > > > Hi All, > > > > > > > > The following is v2 of the K3 DSP remoteproc driver supporting the C66x DSPs > > > > on the TI K3 J721E SoCs. The patches are based on the latest commit on the > > > > rproc-next branch, 7dcef3988eed ("remoteproc: Fix an error code in > > > > devm_rproc_alloc()"). > > > > > > I realized I also had the R5F patches on my branch, so the third patch won't > > > apply cleanly (conflict on Makefile). Let me know if you want a new revision > > > posted for you to pick up the series. > > > > > > > That should be fine, thanks for the heads up! > > > > Will give Mathieu a day or two to take a look as well. > > I don't see having the time to review this set before the middle/end of next > week. I also understand we are crunched by time if we want to get this in > for the upcoming merge window. > > If memory serves me well there wasn't anything controversial about this work. > Under normal circumstances I'd give it a final look but I trust Suman to have > carried out what we agreed on. > > Bjorn - if you are happy with this set then go ahead and queue it. > Thanks Mathieu. I looked through the patches again and saw that we're still waiting for a ack on the dt binding, so I guess I need to hold off on this a little bit longer. Regards, Bjorn
Hi All, The following is v2 of the K3 DSP remoteproc driver supporting the C66x DSPs on the TI K3 J721E SoCs. The patches are based on the latest commit on the rproc-next branch, 7dcef3988eed ("remoteproc: Fix an error code in devm_rproc_alloc()"). v2 includes a new remoteproc core patch (patch 1) that adds an OF helper for parsing the firmware-name property. This is refactored out to avoid replicating the code in various remoteproc drivers. Please see the individual patches for detailed changes. The main dependent patches from the previous series are now staged in rproc-next branch. The only dependency for this series is the common ti-sci-proc helper between R5 and DSP drivers [1]. Please see the initial cover-letter [2] for v1 details. regards Suman [1] https://patchwork.kernel.org/patch/11456379/ [2] https://patchwork.kernel.org/cover/11458573/ Suman Anna (4): remoteproc: Introduce rproc_of_parse_firmware() helper dt-bindings: remoteproc: Add bindings for C66x DSPs on TI K3 SoCs remoteproc/k3-dsp: Add a remoteproc driver of K3 C66x DSPs remoteproc/k3-dsp: Add support for L2RAM loading on C66x DSPs .../bindings/remoteproc/ti,k3-dsp-rproc.yaml | 190 +++++ drivers/remoteproc/Kconfig | 13 + drivers/remoteproc/Makefile | 1 + drivers/remoteproc/remoteproc_core.c | 23 + drivers/remoteproc/remoteproc_internal.h | 2 + drivers/remoteproc/ti_k3_dsp_remoteproc.c | 773 ++++++++++++++++++ 6 files changed, 1002 insertions(+) create mode 100644 Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml create mode 100644 drivers/remoteproc/ti_k3_dsp_remoteproc.c