| Message ID | 20140313200135.GA5981@atomide.com |
|---|---|
| State | New |
| Headers | show |
On Thursday 13 March 2014, Tony Lindgren wrote: > Hi, > > Resending these two, looks like my updated scripts have some issues hitting > the mailing lists.. > > The following changes since commit 18c49af3ee9e32a535413a949672bea726699f04: > > ARM: dts: am335x-evmsk: enable dual_emac mode (2014-03-05 11:53:36 -0800) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap tags/omap-for-v3.15/dt-part3 > > for you to fetch changes up to e1902bbe44844597a38c8cbae30ca895f6e126ee: > > ARM: dts: Add MMC2/SDIO/WLAN support for cm-t3530 (2014-03-12 10:40:37 -0700) > > ---------------------------------------------------------------- > Device tree related changes to the omap iommu driver as that > is finally getting updated. Also few trivial board related > .dts updates to add more devices. Pulled into next/dt, thanks! I haven't really been following the iommu discussions, so I have to trust you are doing the right thing here. What is the status on iommu support through DT? Do we have or require a generic binding for iommus? Are you following that, or will you have to do incompatible changes to do that? Arnd
* Arnd Bergmann <arnd@arndb.de> [140317 07:18]: > On Thursday 13 March 2014, Tony Lindgren wrote: > > Hi, > > > > Resending these two, looks like my updated scripts have some issues hitting > > the mailing lists.. > > > > The following changes since commit 18c49af3ee9e32a535413a949672bea726699f04: > > > > ARM: dts: am335x-evmsk: enable dual_emac mode (2014-03-05 11:53:36 -0800) > > > > are available in the git repository at: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap tags/omap-for-v3.15/dt-part3 > > > > for you to fetch changes up to e1902bbe44844597a38c8cbae30ca895f6e126ee: > > > > ARM: dts: Add MMC2/SDIO/WLAN support for cm-t3530 (2014-03-12 10:40:37 -0700) > > > > ---------------------------------------------------------------- > > Device tree related changes to the omap iommu driver as that > > is finally getting updated. Also few trivial board related > > .dts updates to add more devices. > > Pulled into next/dt, thanks! Thanks, looks like the merge ddf071148934047b1e87bf89e823e74f267f0b84 introduces a build failure though: @@@ -22,8 -23,7 +23,7 @@@ #include "common-board-devices.h" #include "dss-common.h" #include "control.h" - #include "omap-secure.h" -#include "omap_device.h" +#include "soc.h" struct pdata_init { const char *compatible; Those need to be added back to keep things building. > I haven't really been following the iommu discussions, so I have to trust > you are doing the right thing here. What is the status on iommu > support through DT? Do we have or require a generic binding for iommus? There are certainly various things that could be improved in the drivers/iommu, just doing grep EXPORT_SYMBOL drivers/iommu shows quite a few things that should probably be implemented in a generic way. Currently the bindings don't do much, but could probably be simplified further. Things like ti,#tlb-entries and ti,iommu-bus-err-back may not even be needed and could be set by the driver match .data based on the compatible flag alone. This set contains the independent SoC related changes related to get things moving for the drivers/iommu changes as discussed here: http://www.spinics.net/lists/arm-kernel/msg312144.html Some things like reset are still being done using platform_data with a promise of future patches: http://www.spinics.net/lists/linux-omap/msg104328.html > Are you following that, or will you have to do incompatible changes to > do that? I'm not following too closely on the drivers/iommu parts, mostly following just the moving away from platform data part at this point. Suman can probably describe what needs to be done more accurately. Regards, Tony
Hi Arnd, Tony, On 03/17/2014 11:45 AM, Tony Lindgren wrote: > * Arnd Bergmann <arnd@arndb.de> [140317 07:18]: >> On Thursday 13 March 2014, Tony Lindgren wrote: >>> Hi, >>> >>> Resending these two, looks like my updated scripts have some issues hitting >>> the mailing lists.. >>> >>> The following changes since commit 18c49af3ee9e32a535413a949672bea726699f04: >>> >>> ARM: dts: am335x-evmsk: enable dual_emac mode (2014-03-05 11:53:36 -0800) >>> >>> are available in the git repository at: >>> >>> git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap tags/omap-for-v3.15/dt-part3 >>> >>> for you to fetch changes up to e1902bbe44844597a38c8cbae30ca895f6e126ee: >>> >>> ARM: dts: Add MMC2/SDIO/WLAN support for cm-t3530 (2014-03-12 10:40:37 -0700) >>> >>> ---------------------------------------------------------------- >>> Device tree related changes to the omap iommu driver as that >>> is finally getting updated. Also few trivial board related >>> .dts updates to add more devices. >> >> Pulled into next/dt, thanks! > > Thanks, looks like the merge ddf071148934047b1e87bf89e823e74f267f0b84 > introduces a build failure though: > > @@@ -22,8 -23,7 +23,7 @@@ > #include "common-board-devices.h" > #include "dss-common.h" > #include "control.h" > - #include "omap-secure.h" > -#include "omap_device.h" > +#include "soc.h" > > struct pdata_init { > const char *compatible; > > Those need to be added back to keep things building. > >> I haven't really been following the iommu discussions, so I have to trust >> you are doing the right thing here. What is the status on iommu >> support through DT? Do we have or require a generic binding for iommus? > > There are certainly various things that could be improved in the > drivers/iommu, just doing grep EXPORT_SYMBOL drivers/iommu shows quite > a few things that should probably be implemented in a generic way. Yes, Laurent is working towards removing omap-iovmm.c completely and started a discussion/cleanup series towards thats[1]. I expect some of the other export symbols for OMAP to also vanish slowly once iovmm is removed. > > Currently the bindings don't do much, but could probably be simplified > further. Things like ti,#tlb-entries and ti,iommu-bus-err-back may not > even be needed and could be set by the driver match .data based on the > compatible flag alone. Yeah, I was in fact looking for feedback on exactly these properties [2]. I haven't removed this as the only feedback I got was positive for the changes [3]. > > This set contains the independent SoC related changes related to get > things moving for the drivers/iommu changes as discussed here: > > http://www.spinics.net/lists/arm-kernel/msg312144.html > > Some things like reset are still being done using platform_data with > a promise of future patches: > > http://www.spinics.net/lists/linux-omap/msg104328.html > >> Are you following that, or will you have to do incompatible changes to >> do that? > > I'm not following too closely on the drivers/iommu parts, mostly > following just the moving away from platform data part at this point. > Suman can probably describe what needs to be done more accurately. I am not expecting any changes to bindings as this series is mainly adding the OMAP iommu DT nodes and having the OMAP IOMMU driver functional with the DT-based IOMMU devices. We are yet to add any DTS client devices use IOMMU. regards Suman [1] http://marc.info/?l=linux-omap&m=139423949120188&w=2 [2] http://marc.info/?l=linux-omap&m=139231544416973&w=2 [3] http://marc.info/?l=linux-omap&m=139338074931831&w=2
Hi, Resending these two, looks like my updated scripts have some issues hitting the mailing lists.. The following changes since commit 18c49af3ee9e32a535413a949672bea726699f04: ARM: dts: am335x-evmsk: enable dual_emac mode (2014-03-05 11:53:36 -0800) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap tags/omap-for-v3.15/dt-part3 for you to fetch changes up to e1902bbe44844597a38c8cbae30ca895f6e126ee: ARM: dts: Add MMC2/SDIO/WLAN support for cm-t3530 (2014-03-12 10:40:37 -0700) ---------------------------------------------------------------- Device tree related changes to the omap iommu driver as that is finally getting updated. Also few trivial board related .dts updates to add more devices. ---------------------------------------------------------------- Florian Vaussard (4): ARM: OMAP3: remove deprecated CONFIG_OMAP_IOMMU_IVA2 ARM: dts: OMAP3: Update ISP IOMMU node ARM: dts: OMAP3: Add IVA IOMMU node ARM: dts: OMAP4: Add IOMMU nodes Stefan Roese (1): ARM: dts: Add MMC2/SDIO/WLAN support for cm-t3530 Suman Anna (6): ARM: OMAP3: fix iva mmu programming issues ARM: OMAP2+: change the ISP device archdata MMU name for DT ARM: OMAP2+: use pdata quirks for iommu reset lines ARM: OMAP5: hwmod data: add mmu data for ipu & dsp ARM: OMAP2+: extend iommu pdata-quirks to OMAP5 ARM: dts: OMAP5: Add IOMMU nodes Yegor Yefremov (1): ARM: dts: am335x-evmsk: enable DMA controller for USB arch/arm/boot/dts/am335x-evmsk.dts | 4 ++ arch/arm/boot/dts/omap3-cm-t3530.dts | 36 +++++++++++++ arch/arm/boot/dts/omap3.dtsi | 15 ++++-- arch/arm/boot/dts/omap4.dtsi | 15 ++++++ arch/arm/boot/dts/omap5.dtsi | 15 ++++++ arch/arm/mach-omap2/clockdomains3xxx_data.c | 2 +- arch/arm/mach-omap2/devices.c | 3 ++ arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | 12 ++--- arch/arm/mach-omap2/omap_hwmod_54xx_data.c | 83 +++++++++++++++++++++++++++++ arch/arm/mach-omap2/pdata-quirks.c | 24 +++++++++ arch/arm/plat-omap/Kconfig | 3 -- 11 files changed, 197 insertions(+), 15 deletions(-)