2020-06-02 09:28:34

by Joerg Roedel

Subject: [PATCH 0/2] iommu: Move Intel and AMD drivers into their own subdirectory

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.

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%)

--
2.17.1

2020-06-03 07:28:15

by Jerry Snitselaar

Subject: Re: [PATCH 0/2] iommu: Move Intel and AMD drivers into their own subdirectory

On Tue Jun 02 20, 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.
>
>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%)
>
>--
>2.17.1
>

Reviewed-by: Jerry Snitselaar <[email protected]>

2020-06-04 00:20:20

by Lu Baolu

Subject: Re: [PATCH 0/2] iommu: Move Intel and AMD drivers into their own subdirectory

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

> 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%)
>

2020-06-04 00:35:39

by Jerry Snitselaar

Subject: Re: [PATCH 0/2] iommu: Move Intel and AMD drivers into their own subdirectory

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%)
>>
>