>>> On 23.02.17 at 14:59, <julien.gr...@arm.com> wrote: > On 23/02/17 07:39, Jan Beulich wrote: >>>>> On 22.02.17 at 21:29, <sg...@codeaurora.org> wrote: >>> Appreciate the feedback. I want to use these defines for parsing out SMMUv3 >>> components defined in ACPI. Since, I am picking these defines directly from >>> Linux, I did not want to make this a part of a newly developed patch set. >> >> What's wrong with the patch here being part of a patch set, allowing >> reviewers to understand context? > > The IORT will be used to retrieve information to configure different > drivers (ITS and SMMUs). All the drivers don't have dependency between > each other but the IORT. As they will be contributed by different > stakeholders, it would be good to get this header checked-in in Xen as > soon as possible to have a common ground for adding drivers. > > Ideally, I would like to see the IORT parsing going at the same time as > this header. So everyone could re-use it without having to depend on > unrelated series.
Well, you basically confirm that the header on its own isn't of much use - you want parsing code to go in at the same time. That is what I'm asking for: Have some user of the new definitions to allow seeing context of what they are needed for. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
