Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

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

 




Stephen Warren <swarren@xxxxxxxxxxxxx> wrote @ Thu, 31 Oct 2013 17:35:24 +0100:

...
> >> ... and for the driver to explicitly parse that property, and wait
> >> until the driver for iommu_phandle is ready. Exactly the same as any
> >> other resource/service dependency.
> >>
> >> That will solve all the problems.
> >>
> >> The only downside is that every driver needs to contain code to parse
> >> that property. However, I think that's just one function call; the
> >> actual implementation of that function can be unified somewhere inside
> >> core code in drivers/iommu/.
> > 
> > Yes, but only missing part now is that, we could do this with
> > "bus_notifier", but the current bus_notifier doesn't have the feature
> > to return error(-EPROBE_DEFER). This could be modified so that
> > bus_notifier could return (-EPROBE_DEFER) to postpone
> > probing. Alternatively this could be done in some core probe code as
> > well as Thierry pointed out.
> > 
> > [1] In the reply of "[PATCHv3 14/19] iommu/tegra: smmu: Get "nvidia,memory-clients" from DT"
> 
> I think this should be done explicitly in drivers. It's much
> simpler,

Yes, we need to insert some IOMMU specific code in driver?

> and doesn't encode any knowledge of driver-specific bindings into some
> common bus notifier code.

I think that we cannot do that in drivers. We want to use drivers as
they are without any modifications indicating its dependency on
IOMMU because most of drivers are existing ones which nvidia doesn't
implement. We want to set up any IOMMU related thingy implicitly,
hided from driver. That's why we need to do this in bus_notifier or
driver core code.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux