RE: [PATCH 7/8] iommu: of: Handle IOMMU lookup failure with deferred probing or error

2016-09-07 Thread Sricharan
Hi Marek, >Hi Sricharan, > >On 2016-08-09 00:49, Sricharan R wrote: > > From: Laurent Pinchart > > > > Failures to look up an IOMMU when parsing the DT iommus property need to > > be handled separately from the .of_xlate() failures to support deferred >

RE: [PATCH 7/8] iommu: of: Handle IOMMU lookup failure with deferred probing or error

2016-09-07 Thread Sricharan
Hi Marek, >Hi Sricharan, > > >On 2016-08-12 17:40, Sricharan wrote: >> Hi Tomaz, >> + if (ops->add_device) + ops = ops->add_device(dev) ? ops : NULL; >>> Patch description fails to mention anything about this change. Also it >>> looks slightly

Re: [PATCH 7/8] iommu: of: Handle IOMMU lookup failure with deferred probing or error

2016-09-02 Thread Marek Szyprowski
Hi Sricharan, On 2016-08-09 00:49, Sricharan R wrote: > From: Laurent Pinchart > > Failures to look up an IOMMU when parsing the DT iommus property need to > be handled separately from the .of_xlate() failures to support deferred > probing. > > The

Re: [PATCH 7/8] iommu: of: Handle IOMMU lookup failure with deferred probing or error

2016-09-02 Thread Marek Szyprowski
Hi Sricharan, On 2016-08-12 17:40, Sricharan wrote: Hi Tomaz, + if (ops->add_device) + ops = ops->add_device(dev) ? ops : NULL; Patch description fails to mention anything about this change. Also it looks slightly incorrect to lose the error condition

Re: [PATCH 7/8] iommu: of: Handle IOMMU lookup failure with deferred probing or error

2016-08-12 Thread Tomasz Figa
Hi, On Tue, Aug 9, 2016 at 7:49 AM, Sricharan R wrote: > + > + if (ops->add_device) > + ops = ops->add_device(dev) ? ops : NULL; Patch description fails to mention anything about this change. Also it looks slightly incorrect to lose

[PATCH 7/8] iommu: of: Handle IOMMU lookup failure with deferred probing or error

2016-08-08 Thread Sricharan R
From: Laurent Pinchart Failures to look up an IOMMU when parsing the DT iommus property need to be handled separately from the .of_xlate() failures to support deferred probing. The lack of a registered IOMMU can be caused by the lack of a driver for