linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
To: "Javier Búcar" <jbucar@lifia.info.unlp.edu.ar>
Cc: Luis Polasek <lpolasek@gmail.com>,
	Mauro Carvalho Chehab <m.chehab@samsung.com>,
	linux-media@vger.kernel.org,
	Mauro Carvalho Chehab <mchehab@redhat.com>,
	Olivier GRENIE <olivier.grenie@parrot.com>,
	Patrick BOETTCHER <patrick.boettcher@parrot.com>
Subject: Re: dib8000 scanning not working on 3.10.3
Date: Thu, 22 Aug 2013 15:06:04 -0300	[thread overview]
Message-ID: <20130822180603.GA14784@localhost> (raw)
In-Reply-To: <CAJmEX9B=VAEXSto2omRTNcgVdX7akDBUAhJs7nwPUc9xhqFBbg@mail.gmail.com>

Mauro, Javier,

On Thu, Aug 22, 2013 at 02:47:33PM -0300, Javier Búcar wrote:
> 
> http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=173a64cb3fcff1993b2aa8113e53fd379f6a968f
> 
> This is a very big commit. I don't known where to fix it. Can you help

That's not a commit: that's a monster :-(

That should have been heavily splitted. Now a user (Javier and Luis)
is allegedly reporting a regression, but that commit is almost
impossible to understand, and probably very difficult to revert.

I hate to get grumpy, but this does not look nice!
Even the commit log sucks: what does ""dib8000: enhancement"" suppose to mean?

Javier/Luis: (for the second time) please try to avoid top-posting
in the future, it breaks the thread discussion.
-- 
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com

  reply	other threads:[~2013-08-22 18:06 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CAER7dwe+kkVoDbRt9Xj8+77tJnL29bxRzHbSPYOrck_HxVsENw@mail.gmail.com>
2013-07-31 18:47 ` Fwd: dib8000 scanning not working on 3.10.3 Luis Polasek
2013-08-01 16:36   ` Ezequiel Garcia
2013-08-01 17:15     ` Mauro Carvalho Chehab
2013-08-05 19:11       ` Luis Polasek
     [not found]       ` <CAER7dwe9biLNZKtW6xQmD8J0Qmh4dMTi=chpUuQ_Dq5KKxJ5UQ@mail.gmail.com>
2013-08-05 20:26         ` Mauro Carvalho Chehab
2013-08-07 18:48           ` Luis Polasek
2013-08-12 17:59             ` Ezequiel Garcia
2013-08-22 17:47             ` Javier Búcar
2013-08-22 18:06               ` Ezequiel Garcia [this message]
2013-08-22 18:23               ` Mauro Carvalho Chehab
2013-08-26 13:48                 ` Luis Polasek
2013-12-10 18:38   ` Fwd: " Ezequiel Garcia
     [not found]     ` <CAJmEX9Cxrm=8FdDAWngNA=v1j8idqcMYtNcNxByGKUNdMk95JA@mail.gmail.com>
2013-12-11 13:27       ` Ezequiel Garcia

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=20130822180603.GA14784@localhost \
    --to=ezequiel.garcia@free-electrons.com \
    --cc=jbucar@lifia.info.unlp.edu.ar \
    --cc=linux-media@vger.kernel.org \
    --cc=lpolasek@gmail.com \
    --cc=m.chehab@samsung.com \
    --cc=mchehab@redhat.com \
    --cc=olivier.grenie@parrot.com \
    --cc=patrick.boettcher@parrot.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).