From: joe@perches.com (Joe Perches)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 02/12] [media] dvb-pll: Add support for THOMSON DTT7546X tuner.
Date: Thu, 30 Jul 2015 03:08:48 -0700 [thread overview]
Message-ID: <1438250928.2677.10.camel@perches.com> (raw)
In-Reply-To: <20150730094738.GD488@griffinp-ThinkPad-X1-Carbon-2nd>
On Thu, 2015-07-30 at 10:47 +0100, Peter Griffin wrote:
> Hi Mauro / Joe,
>
> On Wed, 22 Jul 2015, Mauro Carvalho Chehab wrote:
>
> > Em Wed, 24 Jun 2015 18:17:37 -0700
> > Joe Perches <joe@perches.com> escreveu:
> >
> > > On Wed, 2015-06-24 at 16:11 +0100, Peter Griffin wrote:
> > > > This is used in conjunction with the STV0367 demodulator on
> > > > the STV0367-NIM-V1.0 NIM card which can be used with the STi
> > > > STB SoC's.
> > >
> > > Barely associated to this specific patch, but for
> > > dvb-pll.c, another thing that seems possible is to
> > > convert the struct dvb_pll_desc uses to const and
> > > change the "entries" fixed array size from 12 to []
> > >
> > > It'd save a couple KB overall and remove ~5KB of data.
> > >
> > > $ size drivers/media/dvb-frontends/dvb-pll.o*
> > > text data bss dec hex filename
> > > 8520 1552 2120 12192 2fa0 drivers/media/dvb-frontends/dvb-pll.o.new
> > > 5624 6363 2120 14107 371b drivers/media/dvb-frontends/dvb-pll.o.old
> >
> > Peter,
> >
> > Please add this patch on the next patch series you submit.
>
> Ok will do, I've added this patch with a slightly updated commit message
> to my series.
>
> Joe - Can I add your signed-off-by?
Signed-off-by: Joe Perches <joe@perches.com>
WARNING: multiple messages have this Message-ID (diff)
From: Joe Perches <joe@perches.com>
To: Peter Griffin <peter.griffin@linaro.org>
Cc: Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, srinivas.kandagatla@gmail.com,
maxime.coquelin@st.com, patrice.chotard@st.com,
lee.jones@linaro.org, hugues.fruchet@st.com,
linux-media@vger.kernel.org, devicetree@vger.kernel.org
Subject: Re: [PATCH 02/12] [media] dvb-pll: Add support for THOMSON DTT7546X tuner.
Date: Thu, 30 Jul 2015 03:08:48 -0700 [thread overview]
Message-ID: <1438250928.2677.10.camel@perches.com> (raw)
In-Reply-To: <20150730094738.GD488@griffinp-ThinkPad-X1-Carbon-2nd>
On Thu, 2015-07-30 at 10:47 +0100, Peter Griffin wrote:
> Hi Mauro / Joe,
>
> On Wed, 22 Jul 2015, Mauro Carvalho Chehab wrote:
>
> > Em Wed, 24 Jun 2015 18:17:37 -0700
> > Joe Perches <joe@perches.com> escreveu:
> >
> > > On Wed, 2015-06-24 at 16:11 +0100, Peter Griffin wrote:
> > > > This is used in conjunction with the STV0367 demodulator on
> > > > the STV0367-NIM-V1.0 NIM card which can be used with the STi
> > > > STB SoC's.
> > >
> > > Barely associated to this specific patch, but for
> > > dvb-pll.c, another thing that seems possible is to
> > > convert the struct dvb_pll_desc uses to const and
> > > change the "entries" fixed array size from 12 to []
> > >
> > > It'd save a couple KB overall and remove ~5KB of data.
> > >
> > > $ size drivers/media/dvb-frontends/dvb-pll.o*
> > > text data bss dec hex filename
> > > 8520 1552 2120 12192 2fa0 drivers/media/dvb-frontends/dvb-pll.o.new
> > > 5624 6363 2120 14107 371b drivers/media/dvb-frontends/dvb-pll.o.old
> >
> > Peter,
> >
> > Please add this patch on the next patch series you submit.
>
> Ok will do, I've added this patch with a slightly updated commit message
> to my series.
>
> Joe - Can I add your signed-off-by?
Signed-off-by: Joe Perches <joe@perches.com>
next prev parent reply other threads:[~2015-07-30 10:08 UTC|newest]
Thread overview: 75+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-24 15:10 [PATCH 00/12] Add c8sectpfe LinuxDVB demux driver Peter Griffin
2015-06-24 15:10 ` Peter Griffin
2015-06-24 15:10 ` Peter Griffin
2015-06-24 15:10 ` [PATCH 01/12] ARM: DT: STi: stihxxx-b2120: Add pulse-width properties to ssc2 & ssc3 Peter Griffin
2015-06-24 15:10 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 02/12] [media] dvb-pll: Add support for THOMSON DTT7546X tuner Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-25 1:17 ` Joe Perches
2015-06-25 1:17 ` Joe Perches
2015-06-25 1:17 ` Joe Perches
2015-07-22 21:58 ` Mauro Carvalho Chehab
2015-07-22 21:58 ` Mauro Carvalho Chehab
2015-07-30 9:47 ` Peter Griffin
2015-07-30 9:47 ` Peter Griffin
2015-07-30 9:47 ` Peter Griffin
2015-07-30 10:08 ` Joe Perches [this message]
2015-07-30 10:08 ` Joe Perches
2015-07-30 11:14 ` Michael Ira Krufky
2015-07-30 11:14 ` Michael Ira Krufky
2015-07-30 11:14 ` Michael Ira Krufky
2015-07-30 11:17 ` Michael Ira Krufky
2015-07-30 11:17 ` Michael Ira Krufky
2015-07-30 11:17 ` Michael Ira Krufky
2015-07-30 14:33 ` Peter Griffin
2015-07-30 14:33 ` Peter Griffin
2015-07-30 11:37 ` Michael Ira Krufky
2015-07-30 11:37 ` Michael Ira Krufky
2015-07-30 14:34 ` Peter Griffin
2015-07-30 14:34 ` Peter Griffin
2015-07-22 17:10 ` Mauro Carvalho Chehab
2015-07-22 17:10 ` Mauro Carvalho Chehab
2015-07-30 9:46 ` Peter Griffin
2015-07-30 9:46 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 03/12] [media] stv0367: Refine i2c error trace to include i2c address Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 04/12] [media] stv0367: Add support for 16Mhz reference clock Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 05/12] [media] tsin: c8sectpfe: Add DT bindings documentation for c8sectpfe driver Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 06/12] ARM: DT: STi: STiH407: Add c8sectpfe LinuxDVB DT node Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 07/12] [media] tsin: c8sectpfe: STiH407/10 Linux DVB demux support Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-07-22 21:46 ` Mauro Carvalho Chehab
2015-07-22 21:46 ` Mauro Carvalho Chehab
2015-07-22 21:46 ` Mauro Carvalho Chehab
2015-07-30 11:08 ` Peter Griffin
2015-07-30 11:08 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 08/12] [media] tsin: c8sectpfe: Add LDVB helper functions Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 09/12] [media] tsin: c8sectpfe: Add support for various ST NIM cards Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-07-22 21:53 ` Mauro Carvalho Chehab
2015-07-22 21:53 ` Mauro Carvalho Chehab
2015-07-30 9:43 ` Peter Griffin
2015-07-30 9:43 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 10/12] [media] tsin: c8sectpfe: Add c8sectpfe debugfs support Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 11/12] [media] tsin: c8sectpfe: Add Kconfig and Makefile for the driver Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-25 7:23 ` Paul Bolle
2015-06-25 7:23 ` Paul Bolle
2015-07-22 7:38 ` Peter Griffin
2015-07-22 7:38 ` Peter Griffin
2015-07-22 21:56 ` Mauro Carvalho Chehab
2015-07-22 21:56 ` Mauro Carvalho Chehab
2015-07-30 9:39 ` Peter Griffin
2015-07-30 9:39 ` Peter Griffin
2015-06-24 15:11 ` [PATCH 12/12] MAINTAINERS: Add c8sectpfe driver directory to STi section Peter Griffin
2015-06-24 15:11 ` Peter Griffin
2015-06-24 15:11 ` Peter Griffin
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=1438250928.2677.10.camel@perches.com \
--to=joe@perches.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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.