All of lore.kernel.org
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: "\"Буди Романто, AreMa Inc\"" <info@are.ma>, linux-media@vger.kernel.org
Cc: "\"Буди Романто, AreMa Inc\"" <knightrider@are.ma>,
	m.chehab@samsung.com, mchehab@osg.samsung.com,
	hdegoede@redhat.com, laurent.pinchart@ideasonboard.com,
	mkrufky@linuxtv.org, sylvester.nawrocki@gmail.com,
	g.liakhovetski@gmx.de, peter.senna@gmail.com
Subject: Re: [PATCH] pt3 (pci, tc90522, mxl301rf, qm1d1c0042): pt3_unregister_subdev(), pt3_unregister_subdev(), cleanups...
Date: Fri, 03 Oct 2014 07:54:14 +0300	[thread overview]
Message-ID: <542E2BF6.2090800@iki.fi> (raw)
In-Reply-To: <1412275758-31340-1-git-send-email-knightrider@are.ma>

On 10/02/2014 09:49 PM, Буди Романто, AreMa Inc wrote:
> DVB driver for Earthsoft PT3 PCIE ISDB-S/T receiver
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>
> Status: stable
>
> Changes:
> - demod & tuners converted to I2C binding model
> - i586 & x86_64 clean compile
> - lightweight & yet precise CNR calculus
> - raw CNR (DVBv3)
> - DVBv5 CNR @ 0.0001 dB (ref: include/uapi/linux/dvb/frontend.h, not 1/1000 dB!)
> - removed (unused?) tuner's *_release()
> - demod/tuner binding: pt3_unregister_subdev(), pt3_unregister_subdev()
> - some cleanups

These drivers are already committed, like you have noticed. There is 
surely a lot of issues that could be improved, but it cannot be done by 
big patch which replaces everything. You need to just take one issue at 
the time, fix/improve it, send patch to mailing list for review. One 
patch per one logical change.

regards
Antti

-- 
http://palosaari.fi/

  reply	other threads:[~2014-10-03  4:54 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-02 18:49 [PATCH] pt3 (pci, tc90522, mxl301rf, qm1d1c0042): pt3_unregister_subdev(), pt3_unregister_subdev(), cleanups Буди Романто, AreMa Inc
2014-10-03  4:54 ` Antti Palosaari [this message]
2014-10-03  5:45   ` AreMa Inc.
2014-10-03 10:52     ` Mauro Carvalho Chehab
2014-10-04  7:16       ` AreMa Inc.
2014-10-04  8:42         ` AreMa Inc.
2014-10-05  9:36         ` AreMa Inc.
2014-10-05 12:29           ` Mauro Carvalho Chehab
2014-10-05 13:04             ` AreMa Inc.
2014-10-05 14:26               ` Mauro Carvalho Chehab
  -- strict thread matches above, loose matches on Subject: below --
2014-10-03  5:18 Буди Романто, AreMa Inc

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=542E2BF6.2090800@iki.fi \
    --to=crope@iki.fi \
    --cc=g.liakhovetski@gmx.de \
    --cc=hdegoede@redhat.com \
    --cc=info@are.ma \
    --cc=knightrider@are.ma \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=m.chehab@samsung.com \
    --cc=mchehab@osg.samsung.com \
    --cc=mkrufky@linuxtv.org \
    --cc=peter.senna@gmail.com \
    --cc=sylvester.nawrocki@gmail.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.