All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Tony Lindgren <tony@atomide.com>
Cc: linux-arm-kernel@lists.infradead.org,
	"Omar Ramirez Luna" <omar.ramirez@ti.com>,
	"Mauro Carvalho Chehab" <mchehab@redhat.com>,
	"Sean Young" <sean@mess.org>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Timo Kokkonen" <timo.t.kokkonen@iki.fi>,
	linux-omap@vger.kernel.org,
	"Víctor Manuel Jáquez Leal" <vjaquez@igalia.com>,
	"Herbert Xu" <herbert@gondor.apana.org.au>,
	"Santosh Shilimkar" <santosh.shilimkar@ti.com>
Subject: Re: [PATCH 6/8] ARM: OMAP2+: Disable code that currently does not work with multiplaform
Date: Thu, 03 Jan 2013 23:53:35 +0100	[thread overview]
Message-ID: <2553128.bVQFtBdOtK@avalon> (raw)
In-Reply-To: <20130103224954.GJ25633@atomide.com>

Hi Tony,

On Thursday 03 January 2013 14:49:54 Tony Lindgren wrote:
> * Laurent Pinchart <laurent.pinchart@ideasonboard.com> [130103 13:25]:
> > Hi Tony,
> > 
> > On Thursday 03 January 2013 12:15:21 Tony Lindgren wrote:
> > > Hi all,
> > > 
> > > Looks like we need to also disable mailbox to avoid dependencies to
> > > the mailbox series as it's being moved to live under drivers.
> > > 
> > > And we need to disable omap3isp as it tries to use the now private
> > > include <plat/*.h> headers.
> > 
> > I've posted a patch to the linux-media mailing list to remove plat/*.h
> > headers that are not present anymore in v3.8. It will be pushed to v3.8,
> > as this breaks compilation of the driver.
> 
> Great, thanks for doing that.
> 
> > The driver still includes the plat/cpu.h header that can be removed as
> > well. I've just posted a patch that remove the header and will push it to
> > v3.9. The OMAP3 ISP part of this patch can thus be dropped.
> 
> OK. I suggest you merge the plat/cpu.h during the -rc cycle as well.
> With v3.8-rc2 plat/cpu.h no longer contains anything for omap2+, and
> we could make it now to do #error for omap2+ if the current users
> are fixed.

If Mauro is fine with the patch going to v3.8-rc I have no issue with that. Is 
this patch targetted at v3.8 or v3.9 ?

> Will drop the omap3isp part from this patch as you suggested though.

Thank you.

-- 
Regards,

Laurent Pinchart


WARNING: multiple messages have this Message-ID (diff)
From: laurent.pinchart@ideasonboard.com (Laurent Pinchart)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 6/8] ARM: OMAP2+: Disable code that currently does not work with multiplaform
Date: Thu, 03 Jan 2013 23:53:35 +0100	[thread overview]
Message-ID: <2553128.bVQFtBdOtK@avalon> (raw)
In-Reply-To: <20130103224954.GJ25633@atomide.com>

Hi Tony,

On Thursday 03 January 2013 14:49:54 Tony Lindgren wrote:
> * Laurent Pinchart <laurent.pinchart@ideasonboard.com> [130103 13:25]:
> > Hi Tony,
> > 
> > On Thursday 03 January 2013 12:15:21 Tony Lindgren wrote:
> > > Hi all,
> > > 
> > > Looks like we need to also disable mailbox to avoid dependencies to
> > > the mailbox series as it's being moved to live under drivers.
> > > 
> > > And we need to disable omap3isp as it tries to use the now private
> > > include <plat/*.h> headers.
> > 
> > I've posted a patch to the linux-media mailing list to remove plat/*.h
> > headers that are not present anymore in v3.8. It will be pushed to v3.8,
> > as this breaks compilation of the driver.
> 
> Great, thanks for doing that.
> 
> > The driver still includes the plat/cpu.h header that can be removed as
> > well. I've just posted a patch that remove the header and will push it to
> > v3.9. The OMAP3 ISP part of this patch can thus be dropped.
> 
> OK. I suggest you merge the plat/cpu.h during the -rc cycle as well.
> With v3.8-rc2 plat/cpu.h no longer contains anything for omap2+, and
> we could make it now to do #error for omap2+ if the current users
> are fixed.

If Mauro is fine with the patch going to v3.8-rc I have no issue with that. Is 
this patch targetted at v3.8 or v3.9 ?

> Will drop the omap3isp part from this patch as you suggested though.

Thank you.

-- 
Regards,

Laurent Pinchart

  reply	other threads:[~2013-01-03 22:52 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-19 19:26 [PATCH 0/8] Flip on multiplatform support for omap2plus for v3.9 Tony Lindgren
2012-12-19 19:26 ` Tony Lindgren
2012-12-19 19:26 ` [PATCH 1/8] ARM: OMAP2+: Limi omap initcalls to omap only on multiplatform kernels Tony Lindgren
2012-12-19 19:26   ` Tony Lindgren
2012-12-19 19:26 ` [PATCH 2/8] ARM: OMAP2+: Use omap initcalls Tony Lindgren
2012-12-19 19:26   ` Tony Lindgren
2012-12-19 19:26 ` [PATCH 3/8] ARM: OMAP: Fix i2c cmdline initcall for multiplatform Tony Lindgren
2012-12-19 19:26   ` Tony Lindgren
2012-12-19 19:26 ` [PATCH 4/8] ARM: OMAP: Fix dmaengine init " Tony Lindgren
2012-12-19 19:26   ` Tony Lindgren
2012-12-19 19:26 ` [PATCH 5/8] ARM: OMAP2+: Add multiplatform debug_ll support Tony Lindgren
2012-12-19 19:26   ` Tony Lindgren
2012-12-19 19:26 ` [PATCH 6/8] ARM: OMAP2+: Disable code that currently does not work with multiplaform Tony Lindgren
2012-12-19 19:26   ` Tony Lindgren
2013-01-03 20:15   ` Tony Lindgren
2013-01-03 20:15     ` Tony Lindgren
2013-01-03 21:23     ` Laurent Pinchart
2013-01-03 21:23       ` Laurent Pinchart
2013-01-03 22:49       ` Tony Lindgren
2013-01-03 22:49         ` Tony Lindgren
2013-01-03 22:53         ` Laurent Pinchart [this message]
2013-01-03 22:53           ` Laurent Pinchart
2013-01-03 23:00           ` Tony Lindgren
2013-01-03 23:00             ` Tony Lindgren
2012-12-19 19:26 ` [PATCH 7/8] ARM: OMAP2+: Enable ARCH_MULTIPLATFORM support Tony Lindgren
2012-12-19 19:26   ` Tony Lindgren
2012-12-19 19:26 ` [PATCH 8/8] ARM: OMAP2+: Add minimal support for booting vexpress Tony Lindgren
2012-12-19 19:26   ` Tony Lindgren
2012-12-20 15:55 ` [PATCH 0/8] Flip on multiplatform support for omap2plus for v3.9 Tony Lindgren
2012-12-20 15:55   ` Tony Lindgren
2012-12-21  0:11   ` Ezequiel Garcia
2012-12-21  0:11     ` Ezequiel Garcia
2012-12-21  1:05     ` Tony Lindgren
2012-12-21  1:05       ` Tony Lindgren
2012-12-21 12:48       ` Ezequiel Garcia
2012-12-21 12:48         ` Ezequiel Garcia
2012-12-21 16:48         ` Tony Lindgren
2012-12-21 16:48           ` Tony Lindgren
2012-12-21  4:01     ` Lokesh Vutla
2012-12-21  4:01       ` Lokesh Vutla
2012-12-21 17:48 ` [PATCH 9/8] ARM: OMAP2+: Remove now obsolete uncompress.h and debug-macro.S Tony Lindgren
2012-12-21 17:48   ` Tony Lindgren

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2553128.bVQFtBdOtK@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=herbert@gondor.apana.org.au \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=mchehab@redhat.com \
    --cc=omar.ramirez@ti.com \
    --cc=santosh.shilimkar@ti.com \
    --cc=sean@mess.org \
    --cc=timo.t.kokkonen@iki.fi \
    --cc=tony@atomide.com \
    --cc=vjaquez@igalia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.