public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Michael Hunold <hunold@convergence.de>
To: Andrew Morton <akpm@osdl.org>
Cc: Michael Hunold <hunold@linuxtv.org>,
	torvalds@osdl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 7/9] tda1004x DVB frontend update
Date: Mon, 23 Feb 2004 23:52:14 +0100	[thread overview]
Message-ID: <403A841E.9090700@convergence.de> (raw)
In-Reply-To: <20040223140943.7e58eb5c.akpm@osdl.org>

Hello Andrew,

On 02/23/04 23:09, Andrew Morton wrote:
>> 	// read it!
>>-	lseek(fd, tda10045h_fwinfo[fwinfo_idx].fw_offset, 0);
>>+        lseek(fd, fw_offset, 0);
>> 	if (read(fd, firmware, fw_size) != fw_size) {

> was there some plan to convert DVB over to using the firmware loader?

Yes. But as I wrote in the mail to Christoph, we currently don't have a 
chance to use some in-kernel structure (pci device, i2c bus) that 
automatically exports the firmware loading magic through sysfs.

Because of this, we would have to write our own sysfs backend for the 
dvb i2c subsystem, in order to get proper firmware loading support.

As I already mentioned in another mail, we want to go back to the kernel 
i2c subsystem.

Currently, the stuff is running quite stable and is "in use".

Changing the i2c subsystem would require changes in all frontend drivers 
+ plus in the dvb drivers exporting the i2c facilities.

Is such a big change acceptable for 2.6 if it fixes these horrible hacks 
or is this 2.7 stuff?

CU
Michael.

  reply	other threads:[~2004-02-23 22:54 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-02-23 21:04 [PATCH 0/9] LinuxTV.org DVB update Michael Hunold
2004-02-23 21:04 ` [PATCH 1/9] Update the DVB subsystem docs Michael Hunold
2004-02-23 21:04   ` [PATCH 2/9] Update saa7146 driver core Michael Hunold
2004-02-23 21:04     ` [PATCH 3/9] Minor DVB Skystar2 updates Michael Hunold
2004-02-23 21:04       ` [PATCH 4/9] DVB core update Michael Hunold
2004-02-23 21:04         ` [PATCH 5/9] Misc. DVB frontend updates Michael Hunold
2004-02-23 21:05           ` [PATCH 6/9] stv0299 DVB frontend update Michael Hunold
2004-02-23 21:05             ` [PATCH 7/9] tda1004x " Michael Hunold
2004-02-23 21:05               ` [PATCH 8/9] av7110 DVB driver update Michael Hunold
2004-02-23 21:05                 ` [PATCH 9/9] TTUSB-Budget " Michael Hunold
2004-02-23 21:18               ` [PATCH 7/9] tda1004x DVB frontend update Christoph Hellwig
2004-02-23 22:47                 ` Michael Hunold
2004-02-23 22:54                   ` Christoph Hellwig
2004-02-23 23:01                     ` Michael Hunold
2004-02-23 22:09               ` Andrew Morton
2004-02-23 22:52                 ` Michael Hunold [this message]
2004-02-23 23:11                   ` Andrew Morton
2004-02-23 23:15                   ` Ralph Metzler
2004-02-24 11:34                   ` Marcel Holtmann
2004-02-23 22:23   ` [PATCH 1/9] Update the DVB subsystem docs Richard B. Johnson
2004-02-23 22:57     ` Michael Hunold

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=403A841E.9090700@convergence.de \
    --to=hunold@convergence.de \
    --cc=akpm@osdl.org \
    --cc=hunold@linuxtv.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@osdl.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox