On Fri, Jan 4, 2019 at 7:49 PM Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> wrote: > > Hi John, > > On Fri, Jan 04, 2019 at 12:56:26PM -0800, John Stultz wrote: > > Try to add DMA support to the uart nodes following > > the assignments made in the dts from the victoria vendor kernel > > here: > > https://consumer.huawei.com/en/opensource/detail/?siteCode=worldwide&keywords=p10&fileType=openSourceSoftware&pageSize=10&curPage=1 > > > > Cc: Tanglei Han <hantanglei@xxxxxxxxxx> > > Cc: Zhuangluan Su <suzhuangluan@xxxxxxxxxxxxx> > > Cc: Ryan Grachek <ryan@xxxxxxxxx> > > Cc: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> > > Cc: Wei Xu <xuwei5@xxxxxxxxxxxxx> > > Cc: Rob Herring <robh+dt@xxxxxxxxxx> > > Cc: Mark Rutland <mark.rutland@xxxxxxx> > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > > Cc: devicetree@xxxxxxxxxxxxxxx > > Signed-off-by: John Stultz <john.stultz@xxxxxxxxxx> > > --- > > arch/arm64/boot/dts/hisilicon/hi3660.dtsi | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/hisilicon/hi3660.dtsi b/arch/arm64/boot/dts/hisilicon/hi3660.dtsi > > index 20ae40d..aaa2b04 100644 > > --- a/arch/arm64/boot/dts/hisilicon/hi3660.dtsi > > +++ b/arch/arm64/boot/dts/hisilicon/hi3660.dtsi > > @@ -466,6 +466,8 @@ > > compatible = "arm,pl011", "arm,primecell"; > > reg = <0x0 0xfdf02000 0x0 0x1000>; > > interrupts = <GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH>; > > + dma-names = "rx", "tx"; > > + dmas = <&dma0 0 &dma0 1>; > > Usage of DMA channel 0 contradicts with the description provided in > patch, "dma: k3dma: Add support to dma_avail_chan". Hrm. Good point. I'll double check w/ Dr Su on this, I'm not sure if that inconsistency is due to the the vendor kernel (where these came from) having different reserved channels or just something overlooked if the uart0 is not actually being used (as we find on hikey960 as well). thanks -john