All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.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, 3 Jan 2013 14:49:54 -0800	[thread overview]
Message-ID: <20130103224954.GJ25633@atomide.com> (raw)
In-Reply-To: <2770938.Qr8SHcYrK2@avalon>

* 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.

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

Regards,

Tony

WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 6/8] ARM: OMAP2+: Disable code that currently does not work with multiplaform
Date: Thu, 3 Jan 2013 14:49:54 -0800	[thread overview]
Message-ID: <20130103224954.GJ25633@atomide.com> (raw)
In-Reply-To: <2770938.Qr8SHcYrK2@avalon>

* 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.

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

Regards,

Tony

  reply	other threads:[~2013-01-03 22:50 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 [this message]
2013-01-03 22:49         ` Tony Lindgren
2013-01-03 22:53         ` Laurent Pinchart
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=20130103224954.GJ25633@atomide.com \
    --to=tony@atomide.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=herbert@gondor.apana.org.au \
    --cc=laurent.pinchart@ideasonboard.com \
    --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=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.