03.10.2020 09:59, Nicolin Chen пишет: > In tegra_smmu_(de)attach_dev() functions, we poll DTB for each > client's iommus property to get swgroup ID in order to prepare > "as" and enable smmu. Actually tegra_smmu_configure() prepared > an fwspec for each client, and added to the fwspec all swgroup > IDs of client DT node in DTB. > > So this patch uses fwspec in tegra_smmu_(de)attach_dev() so as > to replace the redundant DT polling code. > > Signed-off-by: Nicolin Chen <nicoleotsuka@xxxxxxxxx> > --- I'm still not highly impressed by seeing the !fwspec check in this patch. But I'm not a maintainer of the SMMU driver, hence will leave it up to Thierry and Joerg to decide whether this is good or needs to be improved. Otherwise this patch is good to me, thanks. I tested it on Nexus 7, which is Tegra30. Reviewed-by: Dmitry Osipenko <digetx@xxxxxxxxx> Tested-by: Dmitry Osipenko <digetx@xxxxxxxxx>
