| Message ID | 152923285268.19253.8187284565521042664.sendpatchset@little-apple (mailing list archive) |
|---|---|
| State | Rejected |
| Headers | show |
Hi Magnus, On Sun, Jun 17, 2018 at 12:52 PM Magnus Damm <magnus.damm@gmail.com> wrote: > From: Magnus Damm <damm+renesas@opensource.se> > > Here's my local modification to test IPMMU with SYS-DMAC and the SCIF > console on r8a7795 H3ULCB. The dmac1 and dmac2 devices together with > audio dmacs are added to the IPMMU white list. The dmac0 device is > disabled in DT to easily be able to associate the /proc/interrupt > counters with the console activity. > > A quick test has been performed by checking the console boot log output > (no errors and DMACs are assigned to the IPMMU) and also validating that > SCIF console activity results in incrementing /proc/interrupts counters > associated with the DMAC channels. Audio has not been tested. > > Not-Signed-off-by: Magnus Damm <damm+renesas@opensource.se> Thanks for your patch! Just wondering from the oneline-summary: where is the deferred probe (not) supposed to happen? Gr{oetje,eeting}s, Geert
--- 0001/arch/arm64/boot/dts/renesas/r8a7795.dtsi +++ work/arch/arm64/boot/dts/renesas/r8a7795.dtsi @@ -839,6 +839,7 @@ <&ipmmu_ds0 10>, <&ipmmu_ds0 11>, <&ipmmu_ds0 12>, <&ipmmu_ds0 13>, <&ipmmu_ds0 14>, <&ipmmu_ds0 15>; + status = "disabled"; }; dmac1: dma-controller@e7300000 { --- 0001/drivers/iommu/ipmmu-vmsa.c +++ work/drivers/iommu/ipmmu-vmsa.c @@ -756,6 +756,18 @@ static int ipmmu_init_platform_device(st static bool ipmmu_slave_whitelist(struct device *dev) { + if (!strcmp(dev_name(dev), "e7300000.dma-controller")) + return true; + + if (!strcmp(dev_name(dev), "e7310000.dma-controller")) + return true; + + if (!strcmp(dev_name(dev), "ec700000.dma-controller")) + return true; + + if (!strcmp(dev_name(dev), "ec720000.dma-controller")) + return true; + /* By default, do not allow use of IPMMU */ return false; }
From: Magnus Damm <damm+renesas@opensource.se> Here's my local modification to test IPMMU with SYS-DMAC and the SCIF console on r8a7795 H3ULCB. The dmac1 and dmac2 devices together with audio dmacs are added to the IPMMU white list. The dmac0 device is disabled in DT to easily be able to associate the /proc/interrupt counters with the console activity. A quick test has been performed by checking the console boot log output (no errors and DMACs are assigned to the IPMMU) and also validating that SCIF console activity results in incrementing /proc/interrupts counters associated with the DMAC channels. Audio has not been tested. Not-Signed-off-by: Magnus Damm <damm+renesas@opensource.se> --- Used together with renesas-drivers-2018-06-05-v4.17 arch/arm64/boot/dts/renesas/r8a7795.dtsi | 1 + drivers/iommu/ipmmu-vmsa.c | 12 ++++++++++++ 2 files changed, 13 insertions(+)