| Message ID | 1437051319-9904-2-git-send-email-laurent.pinchart@ideasonboard.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
* Laurent Pinchart <laurent.pinchart@ideasonboard.com> [150716 05:57]: > The OMAP3 ISP is now fully supported in DT, remove its instantiation > from C code. Please feel to queue this along with the second patch in this series, this should not cause any merge conflicts: Acked-by: Tony Lindgren <tony@atomide.com> -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Laurent, Laurent Pinchart wrote: > The OMAP3 ISP is now fully supported in DT, remove its instantiation > from C code. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > arch/arm/mach-omap2/devices.c | 53 ------------------------------------------- > arch/arm/mach-omap2/devices.h | 19 ---------------- > 2 files changed, 72 deletions(-) > delete mode 100644 arch/arm/mach-omap2/devices.h If you remove the definitions, arch/arm/mach-omap2/board-cm-t35.c will no longer compile. Could you remove the camera support there as well? My understanding is the board might be supported in DT but I'm not sure about camera. Cc Mike and Igor.
Hi Sakari, On Thursday 16 July 2015 18:45:22 Sakari Ailus wrote: > Laurent Pinchart wrote: > > The OMAP3 ISP is now fully supported in DT, remove its instantiation > > from C code. > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > --- > > > > arch/arm/mach-omap2/devices.c | 53 -------------------------------------- > > arch/arm/mach-omap2/devices.h | 19 ---------------- > > 2 files changed, 72 deletions(-) > > delete mode 100644 arch/arm/mach-omap2/devices.h > > If you remove the definitions, arch/arm/mach-omap2/board-cm-t35.c will > no longer compile. Could you remove the camera support there as well? > > My understanding is the board might be supported in DT but I'm not sure > about camera. > > Cc Mike and Igor. commit 11cd7b8c2773d01e4b40e38568ae62c471a2ea10 Author: Tony Lindgren <tony@atomide.com> Date: Mon May 4 10:48:07 2015 -0700 ARM: OMAP2+: Remove legacy booting support for cm-t35 Merged in v4.2-rc1 :-)
Laurent Pinchart wrote: > Hi Sakari, > > On Thursday 16 July 2015 18:45:22 Sakari Ailus wrote: >> Laurent Pinchart wrote: >>> The OMAP3 ISP is now fully supported in DT, remove its instantiation >>> from C code. >>> >>> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> >>> --- >>> >>> arch/arm/mach-omap2/devices.c | 53 -------------------------------------- >>> arch/arm/mach-omap2/devices.h | 19 ---------------- >>> 2 files changed, 72 deletions(-) >>> delete mode 100644 arch/arm/mach-omap2/devices.h >> >> If you remove the definitions, arch/arm/mach-omap2/board-cm-t35.c will >> no longer compile. Could you remove the camera support there as well? >> >> My understanding is the board might be supported in DT but I'm not sure >> about camera. >> >> Cc Mike and Igor. > > commit 11cd7b8c2773d01e4b40e38568ae62c471a2ea10 > Author: Tony Lindgren <tony@atomide.com> > Date: Mon May 4 10:48:07 2015 -0700 > > ARM: OMAP2+: Remove legacy booting support for cm-t35 > > > Merged in v4.2-rc1 :-) Oh, I missed that one. Good!
Hi Sakari, On 07/16/2015 07:58 AM, Tony Lindgren wrote: > * Laurent Pinchart <laurent.pinchart@ideasonboard.com> [150716 05:57]: >> The OMAP3 ISP is now fully supported in DT, remove its instantiation >> from C code. > > Please feel to queue this along with the second patch in this series, > this should not cause any merge conflicts: > > Acked-by: Tony Lindgren <tony@atomide.com> Just wondering if you have already queued this, I see the v4l changes in linux-next, but not this patch. Also, can you confirm if this series is making it into 4.3? regards Suman -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Tony, On 09/03/2015 05:34 PM, Suman Anna wrote: > Hi Sakari, > > On 07/16/2015 07:58 AM, Tony Lindgren wrote: >> * Laurent Pinchart <laurent.pinchart@ideasonboard.com> [150716 05:57]: >>> The OMAP3 ISP is now fully supported in DT, remove its instantiation >>> from C code. >> >> Please feel to queue this along with the second patch in this series, >> this should not cause any merge conflicts: >> >> Acked-by: Tony Lindgren <tony@atomide.com> > > Just wondering if you have already queued this, I see the v4l changes in > linux-next, but not this patch. Also, can you confirm if this series is > making it into 4.3? > This patch is not in 4.3-rc1, can you pick this up for the next merge window? I am gonna send out some additional cleanup patches (removing legacy mailbox and iommu pieces) on top on this patch. The second patch in this series did make it. regards Suman -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
* Suman Anna <s-anna@ti.com> [150914 09:33]: > Hi Tony, > > On 09/03/2015 05:34 PM, Suman Anna wrote: > > Hi Sakari, > > > > On 07/16/2015 07:58 AM, Tony Lindgren wrote: > >> * Laurent Pinchart <laurent.pinchart@ideasonboard.com> [150716 05:57]: > >>> The OMAP3 ISP is now fully supported in DT, remove its instantiation > >>> from C code. > >> > >> Please feel to queue this along with the second patch in this series, > >> this should not cause any merge conflicts: > >> > >> Acked-by: Tony Lindgren <tony@atomide.com> > > > > Just wondering if you have already queued this, I see the v4l changes in > > linux-next, but not this patch. Also, can you confirm if this series is > > making it into 4.3? > > > > This patch is not in 4.3-rc1, can you pick this up for the next merge > window? I am gonna send out some additional cleanup patches (removing > legacy mailbox and iommu pieces) on top on this patch. The second patch > in this series did make it. OK tagging this one for next, will apply once I'm done with fixes. Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
* Tony Lindgren <tony@atomide.com> [150914 09:37]: > * Suman Anna <s-anna@ti.com> [150914 09:33]: > > Hi Tony, > > > > On 09/03/2015 05:34 PM, Suman Anna wrote: > > > Hi Sakari, > > > > > > On 07/16/2015 07:58 AM, Tony Lindgren wrote: > > >> * Laurent Pinchart <laurent.pinchart@ideasonboard.com> [150716 05:57]: > > >>> The OMAP3 ISP is now fully supported in DT, remove its instantiation > > >>> from C code. > > >> > > >> Please feel to queue this along with the second patch in this series, > > >> this should not cause any merge conflicts: > > >> > > >> Acked-by: Tony Lindgren <tony@atomide.com> > > > > > > Just wondering if you have already queued this, I see the v4l changes in > > > linux-next, but not this patch. Also, can you confirm if this series is > > > making it into 4.3? > > > > > > > This patch is not in 4.3-rc1, can you pick this up for the next merge > > window? I am gonna send out some additional cleanup patches (removing > > legacy mailbox and iommu pieces) on top on this patch. The second patch > > in this series did make it. > > OK tagging this one for next, will apply once I'm done with fixes. Applying into omap-for-v4.4/cleanup finally thanks. Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Tony, On Monday 12 October 2015 10:05:59 Tony Lindgren wrote: > * Tony Lindgren <tony@atomide.com> [150914 09:37]: > > * Suman Anna <s-anna@ti.com> [150914 09:33]: > >> On 09/03/2015 05:34 PM, Suman Anna wrote: > >>> On 07/16/2015 07:58 AM, Tony Lindgren wrote: > >>>> * Laurent Pinchart [150716 05:57]: > >>>> The OMAP3 ISP is now fully supported in DT, remove its instantiation > >>>>> from C code. > >>>> > >>>> Please feel to queue this along with the second patch in this series, > >>>> this should not cause any merge conflicts: > >>>> > >>>> Acked-by: Tony Lindgren <tony@atomide.com> > >>> > >>> Just wondering if you have already queued this, I see the v4l changes > >>> in linux-next, but not this patch. Also, can you confirm if this > >>> series is making it into 4.3? > >> > >> This patch is not in 4.3-rc1, can you pick this up for the next merge > >> window? I am gonna send out some additional cleanup patches (removing > >> legacy mailbox and iommu pieces) on top on this patch. The second patch > >> in this series did make it. > > > > OK tagging this one for next, will apply once I'm done with fixes. > > Applying into omap-for-v4.4/cleanup finally thanks. Is see that the patch is in your master branch but not your for-next branch. Will it make it to v4.4-rc1 ?
diff --git a/arch/arm/mach-omap2/devices.c b/arch/arm/mach-omap2/devices.c index a69bd67e9028..9374da313e8e 100644 --- a/arch/arm/mach-omap2/devices.c +++ b/arch/arm/mach-omap2/devices.c @@ -33,7 +33,6 @@ #include "common.h" #include "mux.h" #include "control.h" -#include "devices.h" #include "display.h" #define L3_MODULES_MAX_LEN 12 @@ -67,58 +66,6 @@ static int __init omap3_l3_init(void) } omap_postcore_initcall(omap3_l3_init); -#if defined(CONFIG_IOMMU_API) - -#include <linux/platform_data/iommu-omap.h> - -static struct resource omap3isp_resources[] = { - { - .start = OMAP3430_ISP_BASE, - .end = OMAP3430_ISP_BASE + 0x12fc, - .flags = IORESOURCE_MEM, - }, - { - .start = OMAP3430_ISP_BASE2, - .end = OMAP3430_ISP_BASE2 + 0x0600, - .flags = IORESOURCE_MEM, - }, - { - .start = 24 + OMAP_INTC_START, - .flags = IORESOURCE_IRQ, - } -}; - -static struct platform_device omap3isp_device = { - .name = "omap3isp", - .id = -1, - .num_resources = ARRAY_SIZE(omap3isp_resources), - .resource = omap3isp_resources, -}; - -static struct omap_iommu_arch_data omap3_isp_iommu = { - .name = "mmu_isp", -}; - -int omap3_init_camera(struct isp_platform_data *pdata) -{ - if (of_have_populated_dt()) - omap3_isp_iommu.name = "480bd400.mmu"; - - omap3isp_device.dev.platform_data = pdata; - omap3isp_device.dev.archdata.iommu = &omap3_isp_iommu; - - return platform_device_register(&omap3isp_device); -} - -#else /* !CONFIG_IOMMU_API */ - -int omap3_init_camera(struct isp_platform_data *pdata) -{ - return 0; -} - -#endif - #if defined(CONFIG_OMAP2PLUS_MBOX) || defined(CONFIG_OMAP2PLUS_MBOX_MODULE) static inline void __init omap_init_mbox(void) { diff --git a/arch/arm/mach-omap2/devices.h b/arch/arm/mach-omap2/devices.h deleted file mode 100644 index f61eb6e5d136..000000000000 --- a/arch/arm/mach-omap2/devices.h +++ /dev/null @@ -1,19 +0,0 @@ -/* - * arch/arm/mach-omap2/devices.h - * - * OMAP2 platform device setup/initialization - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; either version 2 of the License, or - * (at your option) any later version. - */ - -#ifndef __ARCH_ARM_MACH_OMAP_DEVICES_H -#define __ARCH_ARM_MACH_OMAP_DEVICES_H - -struct isp_platform_data; - -int omap3_init_camera(struct isp_platform_data *pdata); - -#endif
The OMAP3 ISP is now fully supported in DT, remove its instantiation from C code. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- arch/arm/mach-omap2/devices.c | 53 ------------------------------------------- arch/arm/mach-omap2/devices.h | 19 ---------------- 2 files changed, 72 deletions(-) delete mode 100644 arch/arm/mach-omap2/devices.h