[pull] iommu: qcom_iommu for v4.14

[pull] iommu: qcom_iommu for v4.14

[Rob Clark]

Hi Joerg,

These patches have been on list for quite some time now.  The bindings
have r-b from Rob Herring, and the patch touching arm-smmu has Will's
a-b.  And all the review comments on the driver patches have been
addressed (despite no one stepping up w/ a r-b tag).  Please consider
merging this pull request for v4.14

BR,
-R

The following changes since commit 60110f262d408c3155e578359812c80fd1f786f9:

  Merge branches 'iommu/fixes', 'arm/exynos', 'arm/renesas',
'arm/rockchip', 'arm/omap', 'arm/mediatek', 'x86/vt-d' and 'core' into
next (2017-08-11 17:09:23 +0200)

are available in the git repository at:

  git://people.freedesktop.org/~robclark/linux

for you to fetch changes up to 8a4342ede167b8c095961846e908b9ac1c884650:

  iommu: qcom: initialize secure page table (2017-08-12 11:50:35 -0400)

----------------------------------------------------------------
Rob Clark (3):
      Docs: dt: document qcom iommu bindings
      iommu: arm-smmu: split out register defines
      iommu: add qcom_iommu

Stanimir Varbanov (1):
      iommu: qcom: initialize secure page table

 .../devicetree/bindings/iommu/qcom,iommu.txt       | 121 +++
 drivers/iommu/Kconfig                              |  10 +
 drivers/iommu/Makefile                             |   1 +
 drivers/iommu/arm-smmu-regs.h                      | 220 +++++
 drivers/iommu/arm-smmu.c                           | 211 +----
 drivers/iommu/qcom_iommu.c                         | 932 +++++++++++++++++++++
 6 files changed, 1293 insertions(+), 202 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/iommu/qcom,iommu.txt
 create mode 100644 drivers/iommu/arm-smmu-regs.h
 create mode 100644 drivers/iommu/qcom_iommu.c

Re: [pull] iommu: qcom_iommu for v4.14

[Rob Clark]

On Sun, Aug 13, 2017 at 10:27 AM, Rob Clark <robdclark@gmail.com> wrote:
> Hi Joerg,
>
> These patches have been on list for quite some time now.  The bindings
> have r-b from Rob Herring, and the patch touching arm-smmu has Will's
> a-b.  And all the review comments on the driver patches have been
> addressed (despite no one stepping up w/ a r-b tag).  Please consider
> merging this pull request for v4.14
>
> BR,
> -R
>
> The following changes since commit 60110f262d408c3155e578359812c80fd1f786f9:
>
>   Merge branches 'iommu/fixes', 'arm/exynos', 'arm/renesas',
> 'arm/rockchip', 'arm/omap', 'arm/mediatek', 'x86/vt-d' and 'core' into
> next (2017-08-11 17:09:23 +0200)
>
> are available in the git repository at:
>
>   git://people.freedesktop.org/~robclark/linux

ugg, something went wrong with my script.. that should be:

  git://people.freedesktop.org/~robclark/linux qcom_iommu-next

BR,
-R

>
> for you to fetch changes up to 8a4342ede167b8c095961846e908b9ac1c884650:
>
>   iommu: qcom: initialize secure page table (2017-08-12 11:50:35 -0400)
>
> ----------------------------------------------------------------
> Rob Clark (3):
>       Docs: dt: document qcom iommu bindings
>       iommu: arm-smmu: split out register defines
>       iommu: add qcom_iommu
>
> Stanimir Varbanov (1):
>       iommu: qcom: initialize secure page table
>
>  .../devicetree/bindings/iommu/qcom,iommu.txt       | 121 +++
>  drivers/iommu/Kconfig                              |  10 +
>  drivers/iommu/Makefile                             |   1 +
>  drivers/iommu/arm-smmu-regs.h                      | 220 +++++
>  drivers/iommu/arm-smmu.c                           | 211 +----
>  drivers/iommu/qcom_iommu.c                         | 932 +++++++++++++++++++++
>  6 files changed, 1293 insertions(+), 202 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/iommu/qcom,iommu.txt
>  create mode 100644 drivers/iommu/arm-smmu-regs.h
>  create mode 100644 drivers/iommu/qcom_iommu.c

Re: [pull] iommu: qcom_iommu for v4.14

[Joerg Roedel]

On Sun, Aug 13, 2017 at 10:27:36AM -0400, Rob Clark wrote:
> Hi Joerg,
> 
> These patches have been on list for quite some time now.  The bindings
> have r-b from Rob Herring, and the patch touching arm-smmu has Will's
> a-b.  And all the review comments on the driver patches have been
> addressed (despite no one stepping up w/ a r-b tag).  Please consider
> merging this pull request for v4.14

Okay, I applied it, not from this pull-request, but from your last
post. Some comments for the future below.

> Rob Clark (3):
>       Docs: dt: document qcom iommu bindings

This patch has no commit-message. Usually that is a reason for me to
reject the patch. I applied it anyway this time because it has an
Reviewed-By from a device-tree maintainer, and if its fine for him, I
can live with it.

For the other three patches I edited the Subject line to match the
convention of the iommu-tree, which is "iommu/<driver>: _Capital Letter_...

The patches below are now named:

>       iommu: arm-smmu: split out register defines
        iommu/arm-smmu: Split out register defines

>       iommu: add qcom_iommu
        iommu/qcom: Add qcom_iommu
> 
> Stanimir Varbanov (1):
>       iommu: qcom: initialize secure page table
        iommu/qcom: Initialize secure page table

Please follow that convention for future patches.


Thanks,

	Joerg

Re: [pull] iommu: qcom_iommu for v4.14

[Rob Clark]

On Tue, Aug 15, 2017 at 11:42 AM, Joerg Roedel <joro@8bytes.org> wrote:
> On Sun, Aug 13, 2017 at 10:27:36AM -0400, Rob Clark wrote:
>> Hi Joerg,
>>
>> These patches have been on list for quite some time now.  The bindings
>> have r-b from Rob Herring, and the patch touching arm-smmu has Will's
>> a-b.  And all the review comments on the driver patches have been
>> addressed (despite no one stepping up w/ a r-b tag).  Please consider
>> merging this pull request for v4.14
>
> Okay, I applied it, not from this pull-request, but from your last
> post. Some comments for the future below.

Thanks.  Patches from list are the same.

For future reference to you prefer to pick up patches from list?
(Dave prefers pull req's for drm, but I guess the volume of patches
for an iommu driver should be lower.)

>> Rob Clark (3):
>>       Docs: dt: document qcom iommu bindings
>
> This patch has no commit-message. Usually that is a reason for me to
> reject the patch. I applied it anyway this time because it has an
> Reviewed-By from a device-tree maintainer, and if its fine for him, I
> can live with it.
>
> For the other three patches I edited the Subject line to match the
> convention of the iommu-tree, which is "iommu/<driver>: _Capital Letter_...
>
> The patches below are now named:
>
>>       iommu: arm-smmu: split out register defines
>         iommu/arm-smmu: Split out register defines
>
>>       iommu: add qcom_iommu
>         iommu/qcom: Add qcom_iommu
>>
>> Stanimir Varbanov (1):
>>       iommu: qcom: initialize secure page table
>         iommu/qcom: Initialize secure page table
>
> Please follow that convention for future patches.
>

will do, thanks

BR,
-R

>
> Thanks,
>
>         Joerg
>

Re: [pull] iommu: qcom_iommu for v4.14

[Joerg Roedel]

On Tue, Aug 15, 2017 at 12:52:11PM -0400, Rob Clark wrote:
> On Tue, Aug 15, 2017 at 11:42 AM, Joerg Roedel <joro@8bytes.org> wrote:
> > On Sun, Aug 13, 2017 at 10:27:36AM -0400, Rob Clark wrote:
> >> Hi Joerg,
> >>
> >> These patches have been on list for quite some time now.  The bindings
> >> have r-b from Rob Herring, and the patch touching arm-smmu has Will's
> >> a-b.  And all the review comments on the driver patches have been
> >> addressed (despite no one stepping up w/ a r-b tag).  Please consider
> >> merging this pull request for v4.14
> >
> > Okay, I applied it, not from this pull-request, but from your last
> > post. Some comments for the future below.
> 
> Thanks.  Patches from list are the same.
> 
> For future reference to you prefer to pick up patches from list?
> (Dave prefers pull req's for drm, but I guess the volume of patches
> for an iommu driver should be lower.)

For now please send any patch for the qcom iommu directly to me (and Cc
relevant ML) and then I take care of them. Can you also add a
MAINTAINERS entry for it?


Regards,

	Joerg