From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]:33880 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932110Ab2JRU5a (ORCPT ); Thu, 18 Oct 2012 16:57:30 -0400 Date: Thu, 18 Oct 2012 17:56:59 -0300 From: Mauro Carvalho Chehab To: Tony Lindgren Cc: linux-arm-kernel@lists.infradead.org, Ohad Ben-Cohen , Joerg Roedel , Laurent Pinchart , Omar Ramirez Luna , linux-omap@vger.kernel.org, Ido Yariv , linux-media@vger.kernel.org, mchehab@infradead.org Subject: Re: [PATCH 3/6] ARM: OMAP2+: Move plat/iovmm.h to include/linux/omap-iommu.h Message-ID: <20121018175659.431fe0b1@redhat.com> In-Reply-To: <20121018202842.11834.14375.stgit@muffinssi.local> References: <20121018202707.11834.1438.stgit@muffinssi.local> <20121018202842.11834.14375.stgit@muffinssi.local> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Tony, Em Thu, 18 Oct 2012 13:28:42 -0700 Tony Lindgren escreveu: > Looks like the iommu framework does not have generic functions > exported for all the needs yet. The hardware specific functions > are defined in files like intel-iommu.h and amd-iommu.h. Follow > the same standard for omap-iommu.h. > > This is needed because we are removing plat and mach includes > for ARM common zImage support. Further work should continue > in the iommu framework context as only pure platform data will > be communicated from arch/arm/*omap*/* code to the iommu > framework. > > Cc: Joerg Roedel > Cc: Ohad Ben-Cohen > Cc: Ido Yariv > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab > Cc: Omar Ramirez Luna > Cc: linux-media@vger.kernel.org > Signed-off-by: Tony Lindgren > --- > arch/arm/mach-omap2/iommu2.c | 1 > arch/arm/plat-omap/include/plat/iommu.h | 10 +-- > arch/arm/plat-omap/include/plat/iovmm.h | 89 ---------------------------- > drivers/iommu/omap-iommu-debug.c | 2 - > drivers/iommu/omap-iommu.c | 1 > drivers/iommu/omap-iovmm.c | 46 ++++++++++++++ > drivers/media/platform/omap3isp/isp.c | 1 > drivers/media/platform/omap3isp/isp.h | 2 - > drivers/media/platform/omap3isp/ispccdc.c | 1 > drivers/media/platform/omap3isp/ispstat.c | 1 > drivers/media/platform/omap3isp/ispvideo.c | 2 - Acked-by: Mauro Carvalho Chehab It is better if you send this entire series through the ARM tree, keeping this hole series altogether (and avoiding the risk of breaking bisectability if it goes through separate trees). So, you can add my ack for those header moves for drivers/media/platform/*. Regards, Mauro -- Cheers, Mauro