Hi Vladimir, On Fri, 12 Nov 2021 at 16:02, Vladimir Zapolskiy <vladimir.zapolskiy@xxxxxxxxxx> wrote: > > Hi Bhupesh, > > On 11/10/21 12:59 PM, Bhupesh Sharma wrote: > > From: Thara Gopinath <thara.gopinath@xxxxxxxxxx> > > > > BAM dma engine associated with certain hardware blocks could require > > relevant interconnect pieces be initialized prior to the dma engine > > initialization. For e.g. crypto bam dma engine on sm8250. Such requirement > > is passed on to the bam dma driver from dt via the "interconnects" > > property. Add support in bam_dma driver to check whether the interconnect > > path is accessible/enabled prior to attempting driver intializations. > > > > Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> > > Cc: Rob Herring <robh+dt@xxxxxxxxxx> > > Signed-off-by: Bhupesh Sharma <bhupesh.sharma@xxxxxxxxxx> > > [Make header file inclusion alphabetical and use 'devm_of_icc_get()'] > > Signed-off-by: Thara Gopinath <thara.gopinath@xxxxxxxxxx> > > please let me ask you to swap your and Thara's sob tags above, there is > a rule applicable to all cases dealing with someone's else changes: > > From Documentation/process/submitting-patches.rst: > > Any further SoBs (Signed-off-by:'s) following the author's SoB are from > people handling and transporting the patch, but were not involved in its > development. SoB chains should reflect the **real** route a patch took > as it was propagated to the maintainers and ultimately to Linus, with > the first SoB entry signalling primary authorship of a single author. Sure, I will fix it in v6. Regards, Bhupesh
