Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 3 Jun 2020 17:33:27 -0700
From:   Jerry Snitselaar <jsnitsel@redhat.com>
To:     Lu Baolu <baolu.lu@linux.intel.com>
Cc:     Joerg Roedel <joro@8bytes.org>,
        David Woodhouse <dwmw2@infradead.org>,
        Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>,
        Qian Cai <cai@lca.pw>, iommu@lists.linux-foundation.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/2] iommu: Move Intel and AMD drivers into their own
 subdirectory
Message-ID: <20200604003327.hk7w7i5qs7z3z2w3@cantor>
Reply-To: Jerry Snitselaar <jsnitsel@redhat.com>
Mail-Followup-To: Lu Baolu <baolu.lu@linux.intel.com>,
        Joerg Roedel <joro@8bytes.org>,
        David Woodhouse <dwmw2@infradead.org>,
        Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>,
        Qian Cai <cai@lca.pw>, iommu@lists.linux-foundation.org,
        linux-kernel@vger.kernel.org
References: <20200602092616.24638-1-joro@8bytes.org>
 <148834f4-849e-acf1-c347-e25093aac518@linux.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Disposition: inline
In-Reply-To: <148834f4-849e-acf1-c347-e25093aac518@linux.intel.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu Jun 04 20, Lu Baolu wrote:
>Hi Joerg,
>
>On 6/2/20 5:26 PM, Joerg Roedel wrote:
>>Hi,
>>
>>two small patches to move the Intel and AMD IOMMU drivers into their own
>>subdirectory under drivers/iommu/ to make the file structure a bit less
>>cluttered.
>>
>
>Does the MAINTAINERS file need to update?
>
>Best regards,
>baolu
>

Yes, that should be updated to point at the new directories. Good catch.


>>Regards,
>>
>>	Joerg
>>
>>Joerg Roedel (2):
>>   iommu/amd: Move AMD IOMMU driver into subdirectory
>>   iommu/vt-d: Move Intel IOMMU driver into subdirectory
>>
>>  drivers/iommu/Makefile                         | 18 +++++++++---------
>>  drivers/iommu/{ => amd}/amd_iommu.h            |  0
>>  drivers/iommu/{ => amd}/amd_iommu_types.h      |  0
>>  .../{amd_iommu_debugfs.c => amd/debugfs.c}     |  0
>>  drivers/iommu/{amd_iommu_init.c => amd/init.c} |  2 +-
>>  drivers/iommu/{amd_iommu.c => amd/iommu.c}     |  2 +-
>>  .../iommu/{amd_iommu_v2.c => amd/iommu_v2.c}   |  0
>>  .../iommu/{amd_iommu_quirks.c => amd/quirks.c} |  0
>>  .../{intel-iommu-debugfs.c => intel/debugfs.c} |  0
>>  drivers/iommu/{ => intel}/dmar.c               |  2 +-
>>  drivers/iommu/{ => intel}/intel-pasid.h        |  0
>>  drivers/iommu/{intel-iommu.c => intel/iommu.c} |  2 +-
>>  .../irq_remapping.c}                           |  2 +-
>>  drivers/iommu/{intel-pasid.c => intel/pasid.c} |  0
>>  drivers/iommu/{intel-svm.c => intel/svm.c}     |  0
>>  drivers/iommu/{intel-trace.c => intel/trace.c} |  0
>>  16 files changed, 14 insertions(+), 14 deletions(-)
>>  rename drivers/iommu/{ => amd}/amd_iommu.h (100%)
>>  rename drivers/iommu/{ => amd}/amd_iommu_types.h (100%)
>>  rename drivers/iommu/{amd_iommu_debugfs.c => amd/debugfs.c} (100%)
>>  rename drivers/iommu/{amd_iommu_init.c => amd/init.c} (99%)
>>  rename drivers/iommu/{amd_iommu.c => amd/iommu.c} (99%)
>>  rename drivers/iommu/{amd_iommu_v2.c => amd/iommu_v2.c} (100%)
>>  rename drivers/iommu/{amd_iommu_quirks.c => amd/quirks.c} (100%)
>>  rename drivers/iommu/{intel-iommu-debugfs.c => intel/debugfs.c} (100%)
>>  rename drivers/iommu/{ => intel}/dmar.c (99%)
>>  rename drivers/iommu/{ => intel}/intel-pasid.h (100%)
>>  rename drivers/iommu/{intel-iommu.c => intel/iommu.c} (99%)
>>  rename drivers/iommu/{intel_irq_remapping.c => intel/irq_remapping.c} (99%)
>>  rename drivers/iommu/{intel-pasid.c => intel/pasid.c} (100%)
>>  rename drivers/iommu/{intel-svm.c => intel/svm.c} (100%)
>>  rename drivers/iommu/{intel-trace.c => intel/trace.c} (100%)
>>
>