Re: [PATCH v2 11/18] iommu: exynos: remove useless device_add/remove callbacks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wednesday 22 October 2014 11:15:00 Alban Browaeys wrote:
> Le mardi 16 septembre 2014 à 13:54 +0200, Marek Szyprowski a écrit :
> > The driver doesn't need to do anything important in device add/remove
> > callbacks, because initialization will be done from device-tree specific
> > callbacks added later. IOMMU groups created by current code were never
> > used.
> > 
> > Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
> 
> 
> The exyons iommu init fails if those are removed, that is it never reach
> init_done:
> 1. exynos_iommu_setup
> 2. - exynos_iommu_init
> 3. ---bus_set_iommu
> 4. ------ add_iommu_group
> 
> that is (4) add_iommu_group returns ENODEV to bus_set_iommu, the latter
> doing so to exynos_iommu_init. Which thus error out before the init_done
> is set to true.

On a related note, I found that the exynos iommu driver does not check
for the presence of an exynos iommu before calling bus_set_iommu,
and this breaks multiplatform kernels.

Please fix.

	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux