All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex Courbot <acourbot@nvidia.com>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: Olof Johansson <olof@lixom.net>,
	Stephen Warren <swarren@wwwdotorg.org>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-fbdev@vger.kernel.org" <linux-fbdev@vger.kernel.org>,
	Alexandre Courbot <gnurou@gmail.com>
Subject: Re: [PATCH] simplefb: add support for a8b8g8r8 pixel format
Date: Fri, 07 Jun 2013 06:08:39 +0000	[thread overview]
Message-ID: <51B178E7.1060802@nvidia.com> (raw)
In-Reply-To: <20130606163240.GY19468@game.jcrosoft.org>

On 06/07/2013 01:32 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>> We're talking about adding a bunch of code instead of one line in a
>> table. Sorry, I NACK your NACK. If we get more entries we'll add the
>> parser, but not just for this.
> as there is no NACK so far I do take as a NACK

... which means?

Anyway, I have sent a new version of this patch that includes a summary 
motivating why we need it. Please explain clearly whether you accept it 
or not, and if not, what we need to do to add this mode in a 
satisfactory way.

Thanks,
Alex.


WARNING: multiple messages have this Message-ID (diff)
From: Alex Courbot <acourbot@nvidia.com>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: Olof Johansson <olof@lixom.net>,
	Stephen Warren <swarren@wwwdotorg.org>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-fbdev@vger.kernel.org" <linux-fbdev@vger.kernel.org>,
	Alexandre Courbot <gnurou@gmail.com>
Subject: Re: [PATCH] simplefb: add support for a8b8g8r8 pixel format
Date: Fri, 7 Jun 2013 15:08:39 +0900	[thread overview]
Message-ID: <51B178E7.1060802@nvidia.com> (raw)
In-Reply-To: <20130606163240.GY19468@game.jcrosoft.org>

On 06/07/2013 01:32 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>> We're talking about adding a bunch of code instead of one line in a
>> table. Sorry, I NACK your NACK. If we get more entries we'll add the
>> parser, but not just for this.
> as there is no NACK so far I do take as a NACK

... which means?

Anyway, I have sent a new version of this patch that includes a summary 
motivating why we need it. Please explain clearly whether you accept it 
or not, and if not, what we need to do to add this mode in a 
satisfactory way.

Thanks,
Alex.


  reply	other threads:[~2013-06-07  6:08 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-06  7:20 [PATCH] simplefb: add support for a8b8g8r8 pixel format Alexandre Courbot
2013-06-06  7:20 ` Alexandre Courbot
2013-06-06  7:59 ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06  7:59   ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06  8:12   ` Alex Courbot
2013-06-06  8:12     ` Alex Courbot
2013-06-06  8:24     ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06  8:24       ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06  8:27       ` Alex Courbot
2013-06-06  8:27         ` Alex Courbot
2013-06-06 14:50         ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06 14:50           ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06 16:17           ` Stephen Warren
2013-06-06 16:17             ` Stephen Warren
2013-06-06 16:29             ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06 16:29               ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06 16:45               ` Stephen Warren
2013-06-06 16:45                 ` Stephen Warren
2013-06-06 16:11     ` Stephen Warren
2013-06-06 16:11       ` Stephen Warren
2013-06-06 16:20       ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06 16:20         ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06 16:33         ` Olof Johansson
2013-06-06 16:33           ` Olof Johansson
2013-06-06 16:32           ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06 16:32             ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-07  6:08             ` Alex Courbot [this message]
2013-06-07  6:08               ` Alex Courbot

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=51B178E7.1060802@nvidia.com \
    --to=acourbot@nvidia.com \
    --cc=gnurou@gmail.com \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=olof@lixom.net \
    --cc=plagnioj@jcrosoft.com \
    --cc=swarren@wwwdotorg.org \
    --cc=tomi.valkeinen@ti.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.