All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: hvaibhav@ti.com, linux-media@vger.kernel.org,
	Tony Lindgren <tony@atomide.com>,
	linux-omap@vger.kernel.org, Archit Taneja <archit@ti.com>
Subject: Re: [PATCH 0/2] omap_vout: remove cpu_is_* uses
Date: Wed, 28 Nov 2012 16:13:41 +0100	[thread overview]
Message-ID: <1421983.jJNXU7RvjW@avalon> (raw)
In-Reply-To: <1352727220-22540-1-git-send-email-tomi.valkeinen@ti.com>

Hi Tomi,

On Monday 12 November 2012 15:33:38 Tomi Valkeinen wrote:
> Hi,
> 
> This patch removes use of cpu_is_* funcs from omap_vout, and uses omapdss's
> version instead. The other patch removes an unneeded plat/dma.h include.
> 
> These are based on current omapdss master branch, which has the omapdss
> version code. The omapdss version code is queued for v3.8. I'm not sure
> which is the best way to handle these patches due to the dependency to
> omapdss. The easiest option is to merge these for 3.9.
> 
> There's still the OMAP DMA use in omap_vout_vrfb.c, which is the last OMAP
> dependency in the omap_vout driver. I'm not going to touch that, as it
> doesn't look as trivial as this cpu_is_* removal, and I don't have much
> knowledge of the omap_vout driver.
> 
> Compiled, but not tested.

Tested on a Beagleboard-xM.

Tested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

The patches depend on unmerged OMAP DSS patches. Would you like to push this 
series through linuxtv or through your DSS tree ? The later might be easier, 
depending on when the required DSS patches will hit mainline.

> Tomi Valkeinen (2):
>   [media] omap_vout: use omapdss's version instead of cpu_is_*
>   [media] omap_vout: remove extra include
> 
>  drivers/media/platform/omap/omap_vout.c    |    4 +--
>  drivers/media/platform/omap/omap_voutlib.c |   38 ++++++++++++++++++-------
>  drivers/media/platform/omap/omap_voutlib.h |    3 +++
>  3 files changed, 32 insertions(+), 13 deletions(-)

-- 
Regards,

Laurent Pinchart


  parent reply	other threads:[~2012-11-28 15:12 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-12 13:33 [PATCH 0/2] omap_vout: remove cpu_is_* uses Tomi Valkeinen
2012-11-12 13:33 ` Tomi Valkeinen
2012-11-12 13:33 ` [PATCH 1/2] [media] omap_vout: use omapdss's version instead of cpu_is_* Tomi Valkeinen
2012-11-12 13:33   ` Tomi Valkeinen
2012-11-12 13:33 ` [PATCH 2/2] [media] omap_vout: remove extra include Tomi Valkeinen
2012-11-12 13:33   ` Tomi Valkeinen
2012-11-12 17:37 ` [PATCH 0/2] omap_vout: remove cpu_is_* uses Tony Lindgren
2012-11-28 15:13 ` Laurent Pinchart [this message]
2012-11-29  9:30   ` Tomi Valkeinen
2012-11-29  9:30     ` Tomi Valkeinen
2012-11-29  9:36     ` Laurent Pinchart
2012-11-29 16:29       ` Tony Lindgren
2012-11-29 16:39         ` Laurent Pinchart
2012-11-29 17:05           ` Mauro Carvalho Chehab
2012-11-29 17:08             ` Tomi Valkeinen
2012-11-29 17:08               ` Tomi Valkeinen

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=1421983.jJNXU7RvjW@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=archit@ti.com \
    --cc=hvaibhav@ti.com \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=tomi.valkeinen@ti.com \
    --cc=tony@atomide.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.