On Oct 14 Mauro Carvalho Chehab wrote:
> There are lots of drivers on media that breaks long line strings in order to
> fit into 80 columns. This was an usual practice to make checkpatch happy.
This was practice even before checkpatch existed.
--
Stefan Richter
-==- =-=- -===
strings at the source code. So, join those continuation
> > lines.
> >
> > Signed-off-by: Mauro Carvalho Chehab
>
> I prefer this patch because of the same reason in patch comment.
>
> Reviewed-by: Takashi Sakamoto
Acked-by: Stefan Richter
> > ---
&
ade to use dev_XYZ().
(On an unrelated note, this reminds me that I still need to take care of
Mauro's patches "Add a keymap for FireDTV board" and "firedtv: Port it to
use rc_core" from May 28, 2012.)
--
Stefan Richter
-==- =-=- =--==
http://arcgraph.de/sr/
--
To unsubs
> Would the above be OK for you?
ACK.
--
Stefan Richter
-==- =-=- =-=--
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
The Linux console did not have an issue with the
FireDTV's RC though.
The fix is to insert EV_SYN events after the key-down/-up events.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-rc.c |8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
Index: b/dr
The Linux console did not have an issue with the
FireDTV's RC though.
The fix is to insert EV_SYN events after the key-down/-up events.
Dimitry notes that EV_SYN is also necessary between down and up,
otherwise userspace could combine their state.
Signed-off-by: Stefan Richter
---
dr
Dimitry notes that EV_SYN is also necessary between down and up,
otherwise userspace could combine their state.
Signed-off-by: Stefan Richter
---
Hi Mauro,
since you already pushed out the first version of "firedtv: fix remote
control with newer Xorg evdev", here is the differential pa
rs.
How so? Then there is something wrong with the backported sources. If
CONFIG_IEEE1394 is not defined, neither make nor gcc ever see anything
that includes ieee1394 headers. Vice versa regarding CONFIG_FIREWIRE and
the newer firewire headers.
--
Stefan Richter
-=-==-== --=- --=-=
http://
e}; controlled by whether
CONFIG_{IEEE1394,FIREWIRE} are defined or not.
I will have a look at your changes later.
--
Stefan Richter
-=-==-== --=- --=-=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord
On Feb 05 Stefan Richter wrote:
> On Feb 03 Hans Verkuil wrote:
> > It would be nice to remove this since building the firedtv driver for older
> > kernels
> > always gives problems on ubuntu due to some missing ieee1394 headers.
>
> How so? Then there is somethin
hing.
If you rewrite the changelog, you can add
Reviewed-by: Stefan Richter
if you like.
--
Stefan Richter
-=-==-== --=- --==-
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
removal are implemented.
Signed-off-by: Stefan Richter
---
Applies after Hans' "firedtv: remove dependency on the deleted ieee1394
stack.",
http://git.linuxtv.org/hverkuil/media_tree.git?a=commitdiff;h=f02c316436eef3baf349c489545edc7ade419ff6
drivers/media/dvb/firewire/fired
.config having to match the kernel sources
applies just as well and fully describes the problem and its solution. :-)
--
Stefan Richter
-=-==-== --=- -=--=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to m
On Feb 09 Stefan Richter wrote:
> > https://bugs.launchpad.net/ubuntu/+source/linux-kernel-headers/+bug/134222
Correction:
Bug 134222 has *nothing* to do with the removal of the older ieee1394
stack. The bug is about
1. a defect during assembling the linux-kernel-headers package.
ce-feedback stuff. How about
calling struct keymap_entry perhaps struct input_keymap_entry
instead, to keep namespaces tidy?
- I take it from your description that scan codes are fundamentally
variable-length data. How about defining it as __u8 scancode[0]?
--
Stefan Richter
-=
Stefan Richter wrote:
> - I take it from your description that scan codes are fundamentally
> variable-length data. How about defining it as __u8 scancode[0]?
Forget this; that would make it difficult to extend the ABI later by
adding more struct members.
--
Stefan R
he mail user agent, and there appear to
be some whitespace issues in it. I will have a closer look at it later today
and repost the patch so that Mauro can apply it without manual intervention.
--
Stefan Richter
-=-==-=- =--= -
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the
property = fdtv_get_property;
> + ops->set_property = fdtv_set_property;
> +
> ops->read_status= fdtv_read_status;
> ops->read_ber = fdtv_read_ber;
> ops->read_signal_strength = fdtv_read_signal_strength;
[...]
(Hmm, note to self: Can't DVB core provide empty default methods?)
--
Stefan Richter
-=-==-=- =--= -
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
It adds support for tuning with PSK8 modulation, pilot and rolloff
with the S2 versions of firedtv.
Signed-off-by: Tommy Jonsson
Signed-off-by: Stefan Richter (trivial
simplification)
---
Update: resend with whitespace preserved, fe pointer does not have to be
put into function parameter lis
It adds support for tuning with PSK8 modulation, pilot and rolloff
with the S2 versions of firedtv.
Signed-off-by: Tommy Jonsson
Signed-off-by: Stefan Richter (trivial
simplification)
---
Sorry, missed two space-before-tab.
drivers/media/dvb/firewire/firedtv-avc.c | 30 +-
tions(-)
--
Stefan Richter
-=-==-=- =-=- -==-=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
re/firedtv-avc.c | 520 +++---
drivers/media/dvb/firewire/firedtv-dvb.c |1
drivers/media/dvb/firewire/firedtv-fw.c | 39 -
drivers/media/dvb/firewire/firedtv.h |8
5 files changed, 306 insertions(+), 275 deletions(-)
--
Stefan Richter
-=-==--= =-==
Cache only addresses of whole pages, not of each buffer chunk. Besides,
page addresses can be obtained by page_address() instead of kmap() since
they were allocated in lowmem.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-fw.c | 19 ---
1 file changed
which packet buffer to set the interrupt flag.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-fw.c | 13 -
1 file changed, 4 insertions(+), 9 deletions(-)
Index: linux-2.6.32-rc7/drivers/media/dvb/firewire/firedtv-fw.c
All read transactions initiated by firedtv are only quadlet-sized, hence
the backend->read call can be simplified a little.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-1394.c |4 ++--
drivers/media/dvb/firewire/firedtv-avc.c |8
drivers/media/
FCP
transactions, let it wait for an interim response.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-1394.c |9 +
drivers/media/dvb/firewire/firedtv-avc.c | 437 +-
drivers/media/dvb/firewire/firedtv-dvb.c |1 -
drivers/media/dvb/firewire/fire
time out, usually much shorter).
Hence simplify the code to plain mutex_lock.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-avc.c | 51 ---
1 file changed, 17 insertions(+), 34 deletions(-)
Index: linux-2.6.32-rc7/drivers/media/dvb/firewire/firedtv
length response will be taken.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-avc.c | 146 ++-
1 file changed, 65 insertions(+), 81 deletions(-)
Index: linux-2.6.32-rc7/drivers/media/dvb/firewire/firedtv-avc.c
On 18 Nov, Stefan Richter wrote:
> The following three patches are applicable after "firedtv: port to new
> firewire core" from 2009-11-08:
...
> The rest of this patch set additionally requires the latest firedtv as
> of 2.6.32-rc7:
...
I updated the "firedtv" bra
ndation for lirc. I have no idea whether a
pulse/space -> scancode -> keycode translation would be practical there.)
--
Stefan Richter
-=-==--= =-== =-===
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to
a new interface, just using an existing mechanism (sysfs). Not
all of sysfs in itself is an interface really; rather there is a number
of interfaces which are implemented by means of sysfs.
sysfs is primarily meant for simple textual attributes though, not for
I/O streams.
--
Stefan Richter
-=
us quo.
- No regressions would be best.
--
Stefan Richter
-=-==--= =-== ==-==
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
he driver to clean the
> current
> keymap table and allocate a new one with the specified size.
> This will avoid playing with memory allocation for every new key and will
> provide a simple
> way to say to the driver to discard the current keybable, since a new o
an alternative --- but this does not affect LIRC and
whether to merge it in any way.
PS: Drop your "specific vs. generic tools" terminology already. Your
configfs based proposal requires "specific" tools as well, it's just
that they can be implemented
machine), they /can/ be and are shipped everywhere, they /do/
auto-load when the device is inserted. And if there should be an issue
with power management (is there any?), then improve the ABI and libusb
can surely be improved. I don't see why a device with a userspace
driver cannot be incl
Jon Smirl wrote:
> On Sat, Nov 28, 2009 at 1:17 PM, Stefan Richter
> wrote:
>> Jon Smirl wrote:
>>> There are two very basic things that we need to reach consensus on first.
>>>
>>> 1) Unification with mouse/keyboard in evdev - put IR on equal footing.
>
Jon Smirl wrote:
> On Sat, Nov 28, 2009 at 2:30 PM, Stefan Richter
> wrote:
>> Jon Smirl wrote:
>>> If these drivers are for specific USB devices it is straight forward
>>> to turn them into kernel based drivers. If we are going for plug and
>>> play this
Jon Smirl wrote:
> On Sat, Nov 28, 2009 at 2:45 PM, Stefan Richter
> wrote:
>> Jon Smirl wrote:
>>> Also, how do you create the devices for each remote? You would need to
>>> create these devices before being able to do EVIOCSKEYCODE to them.
>> The input su
Stefan Richter wrote:
> Jon Smirl wrote:
>> We have one IR receiver device and multiple remotes. How does the
>> input system know how many devices to create corresponding to how many
>> remotes you have?
>
> If several remotes are to be used on the same receiver, then
Jon Smirl wrote:
> On Sat, Nov 28, 2009 at 3:29 PM, Stefan Richter
> wrote:
>> Jon Smirl wrote:
>>> We have one IR receiver device and multiple remotes. How does the
>>> input system know how many devices to create corresponding to how many
>>> remotes yo
fdtv_fw_exit() is part of the firedtv driver's .exit.text section.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-fw.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-2.6.33-rc2/drivers/media/dvb/firewire/firedtv
. The firewire variant of firedtv's device probe is not
affected.
The fix makes sure that such an unexpected firmware is safely recognized
by fdtv_alloc as an unknown firmware.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-1394.c | 10 +++---
1 file chang
Stefan Richter wrote on 2009-12-26:
> fdtv_fw_exit() is part of the firedtv driver's .exit.text section.
...
> --- linux-2.6.33-rc2.orig/drivers/media/dvb/firewire/firedtv-fw.c
> +++ linux-2.6.33-rc2/drivers/media/dvb/firewire/firedtv-fw.c
> @@ -332,7 +332,7 @@ int __init
have been simpler if you had first
inserted an hg:v4l-dvb only patch which simply reverts the divergence of
firedtv in hg from mainline git.
This divergence was introduced by some kind of hg->git export mistake.
That's not a big issue but it may cause another mistake when the next
hg->gi
This way my worries are unfounded. :-)
--
Stefan Richter
-=-==-=- --=- =
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
r 8-byte block write
requests on the other hand needs to remain slab-allocated in order to
keep the allocation around until end of AT DMA.
Signed-off-by: Stefan Richter
---
Tested with firedtv, snd-firewire-speakers, sind-isight, and Coriander
on a kernel with CONFIG_DMA_API_DEBUG=y.
The patch
Henrik Kurelid wrote:
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2, 6, 22)
This expression is always true. :-)
--
Stefan Richter
-=-==--= -=== =---=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a mess
ength)) {
printk(KERN_INFO "%ssu=%x.%x l=%d: %-8s - %s\n",
[...]
Shouldn't the three return statements in debug_fcp_opcode_flag_set be
'return 0' rather than one?
--
Stefan Richter
-=-==--= -=== =---=
http://arcgraph.de/sr/
--
To unsubscribe from th
urn 1;
}
By the way, from here it looks as if your MUA converted tabs to spaces.
In your other patch too.
--
Stefan Richter
-=-==--= -=== =---=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
urn 1;
}
By the way, from here it looks as if your MUA converted tabs to spaces.
In your other patch too.
--
Stefan Richter
-=-==--= -=== =---=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
urn 1;
}
By the way, from here it looks as if your MUA converted tabs to spaces.
In your other patch too.
--
Stefan Richter
-=-==--= -=== =---=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
urn 1;
}
By the way, from here it looks as if your MUA converted tabs to spaces.
In your other patch too.
--
Stefan Richter
-=-==--= -=== =---=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
ning after antenna
was plugged out and back in with DVB-T. Does not fix this with DVB-C,
but also doesn't regress on DVB-C.
Signed-off-by: Stefan Richter
---
Update (Stefan R):
Make a function static, small whitespace changes, extended changelog.
drivers/media/dvb/firewire/fired
Shrinks source and kernel object size a bit.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-avc.c | 109 ++-
1 file changed, 49 insertions(+), 60 deletions(-)
Index: b/drivers/media/dvb/firewire/firedtv-avc.c
sent
to the driver. This will be usable since the CA support is still poorly
tested for lots of CAMs and CA systems.
Signed-off-by: Henrik Kurelid
Signed-off-by: Stefan Richter
---
Update (Stefan R):
Remove pre 2.6.22 compatibility #ifdefs, small whitespace changes.
drivers/media/dvb/firewire
akefile.
(Also shows that even very long exposure in linux-next does not catch
runtime issues like this one.)
Stefan Richter
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http:/
Eliminate
drivers/media/dvb/firewire/firedtv-avc.c: In function 'debug_fcp':
drivers/media/dvb/firewire/firedtv-avc.c:156: warning: format '%d' expects type
'int', but argument 5 has type 'size_t'
Signed-off-by: Stefan Richter
---
Mauro, if you don'
] = SFE_VENDOR_OPCODE_TUNE_QPSK;
+ if (fdtv->type == FIREDTV_DVB_S2)
+ c->operand[3] = SFE_VENDOR_OPCODE_TUNE_QPSK2;
+ else
+ c->operand[3] = SFE_VENDOR_OPCODE_TUNE_QPSK;
c->operand[4] = (params->frequency >> 24) & 0xff;
Commit 34aecd2851bba5c2b7dae2f0dbe8e629b1c5e4ac was made obsolete
and invalid by commit 40cf65d149053889c8876c6c2b4ce204fde55baa.
Signed-off-by: Stefan Richter
---
drivers/media/dvb/firewire/firedtv-avc.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/media
Tobias Klauser wrote:
The C99 specification states in section 6.11.5:
The placement of a storage-class specifier other than at the beginning
of the declaration specifiers in a declaration is an obsolescent
feature.
Signed-off-by: Tobias Klauser
Acked-by: Stefan Richter
---
drivers/media
time.)
- In firedtv: Implement reception of HD channels. (Somebody else
needs to do it since I'm not familiar with the DVB subsystem.)
--
Stefan Richter
-=-=-=== -=-= -==-=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the
so because I had to learn it too; I certainly wasn't born
with an instinct knowledge how to do it properly.
(Disclaimer: I have no stake in drivers/media/ ABIs. But I am involved
in maintaining a userspace ABI elsewhere in drivers/firewire/, and one of
the userspace libraries that use this AB
t. On the other hand, Mauro's solution is redundant
to the uname(2) syscall.
¹) Yes, it is still an ABI version, nothing less. With all its backwards
and forwards compatibility ramifications.
--
Stefan Richter
-=-==-== -==- ==---
http://arcgraph.de/sr/
--
To unsubscribe from this li
erating system distributor, or driver source code
is not released within kernel source code.
--
Stefan Richter
-=-==-== -==- ==---
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
On Jun 24 Devin Heitmueller wrote:
> On Fri, Jun 24, 2011 at 5:20 PM, Stefan Richter
> wrote:
> > Easier:
> > "I run Ubuntu 10.4".
> > "I run kernel 2.6.32."
> > One of these is usually already included in the first post or IRC message
>
Let's just pass through the error returns of lower level I/O
code where applicable, and -EACCES (permission denied) when a seemingly
valid but negative FCP response or an unknown-to-firedtv CA message is
received.
Signed-off-by: Stefan Richter
Cc: Henrik Kurelid
---
Only tested on FireD
Let's just pass through the error returns of lower level I/O
code where applicable, and -EACCES (permission denied) when a seemingly
valid but negative FCP response or an unknown-to-firedtv CA message is
received.
Signed-off-by: Stefan Richter
Cc: Henrik Kurelid
---
Resent, hopefully w
roduction now though and weren't
exactly on the cheap side. OTOH one can drive up to 3 DVB FireWire
receivers on a single FireWire bus; and for those who need even more
there are dual link FireWire PCI and PCIe cards readily available.
--
Stefan Richter
-=-==-=- -=-= =-===
http://arcgraph
Jed wrote:
> On 24/05/10 4:18 AM, Stefan Richter wrote:
>> Jed wrote:
>>> Ideally it'd be dual DVB-C, the only one I've found is more than dual
>>> DVB-C& is far too expensive.
>>
>> If you need two receivers but can only spare up to one
options that replaced older
options and need to be migrated to 'on' per default in make oldconfig.
I think this is not the case here.)
--
Stefan Richter
-=-==-=- -=-= ==--=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
t
Mauro Carvalho Chehab wrote:
> Stefan Richter wrote:
[CONFIG_RC_MAP et al]
>> Please leave the default of new options at N.
>>
>> (Unless this were a special case of new options that replaced older
>> options and need to be migrated to 'on' per default in make
ft compilers define unsigned
long always only as 32 bits wide AFAIK.
--
Stefan Richter
-=-==-=- -==- --=-=
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
ady, "git commit --amend [-a] [...]" lets
you alter the very last commit of course.
--
Stefan Richter
-=-==-=- -==- -
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
current driver code enough to connect the dots?
Is the transport stream different from DVB-C HD streams so that changes
to the isochronous I/O part would be required?
--
Stefan Richter
-=-==-=- --=- -===-
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe
. The version number is now copied
from the original PMT.
Signed-off-by: Henrik Kurelid
Signed-off-by: Stefan Richter
---
This patch got stuck somehow on the long way upstream. :-)
Would be good to get into one of the next .34-rc releases.
drivers/media/dvb/firewire/firedtv-avc.c |2 +-
1
specified.
This is needed for users with CAMs that have different manufacturer id and
ca_system_id and that uses applications that take this into account, like
MythTV.
Signed-off-by: Henrik Kurelid
Signed-off-by: Stefan Richter (rebased, format of
comment)
---
Would be good to get into an .34-rc too
hich retries a
kind of dummy operation with different system IDs until success,
initially during fdtv_dvb_register/ fdtv_ca_register?
(I don't know how CI works, and alas I don't have a CAM myself for
testing and am not very keen on getting one...)
--
Stefan Richter
-=-==-=- --==
ca_system_ids can
be specified.
This is needed for users with CAMs that have different manufacturer id and
ca_system_id and that uses applications that take this into account, like
MythTV.
Signed-off-by: Henrik Kurelid
Signed-off-by: Stefan Richter (rebased, type of
num_fake_ca_system_ids, format
Mauro Carvalho Chehab wrote:
> Stefan Richter wrote:
>
>> The Digital Everywhere firmware have the shortcoming that ca_info_enq and
>> ca_info are not supported. This means that we can never retrieve the correct
>> ca_system_id to present in the CI message CA_INFO. C
Stefan Richter wrote:
> I already posted an updated version of the patch which correctly defines
> num_fake_ca_system_ids as an unsigned long.
err, unsigned int of course, as http://patchwork.kernel.org/patch/82912/.
--
Stefan Richter
-=-==-=- --== --==-
http://arcgraph.de/sr/
/read_write.c::no_llseek.
Signed-off-by: Stefan Richter
---
This patch is motivated by Arnd's
"bkl removal: make unlocked_ioctl mandatory"
http://git.kernel.org/?p=linux/kernel/git/arnd/playground.git;a=blobdiff;f=drivers/media/dvb/firewire/firedtv-ci.c;h=7ab89035c101240
_ioctl and remove .kernel_ioctl and the
temporarily introduced dvbdev.c::legacy_usercopy().
Applies to:
drivers/media/dvb/ttpci/av7110.c
drivers/media/dvb/ttpci/av7110_av.c
drivers/media/dvb/ttpci/av7110_ca.c
drivers/media/dvb/dvb-core/dvb_frontend.c
Signed-off-by: Stef
there are also
- remaining .ioctl that need to be checked for possible concurrency
issues, then converted to .unlocked_ioctl,
- remaining .llseek uses (all implicit) which need to be checked
whether they should be no_llseek() (accompanied by nonseekable_open)
or generic_file_llse
Stefan Richter wrote:
>> linux/drivers/media/dvb/dvb-core/dvbdev.c: lock_kernel();
>> linux/drivers/media/dvb/dvb-core/dvbdev.c: unlock_kernel();
>> linux/drivers/media/dvb/dvb-core/dvbdev.c: unlock_kernel();
>
> This is from when the BKL was pushed
I leave
> that to the DVB experts.
As a dvb/firewire/firedtv user, I started to mess around with dvbdev and
firedtv: https://patchwork.kernel.org/patch/88778/
--
Stefan Richter
-=-==-=- -=-- =
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubs
e
>> that to the DVB experts.
>
> The main issue is at dvbdev, since it is used by all devices. We need to get
> rid
> of it.
Get rid of its lock_kernel in open and its locked ioctl, or of the
dvbdev 'library' itself?
> That's said, Stefan Richter sent a patch me
No need to get angry that it hasn't happened yet. :-) It's just a matter
of the right people joining the effort at the right time.
Thanks Devin for the offer,
--
Stefan Richter
-=-==-== =--= ==-==
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On Sep 27 Greg KH wrote:
> On Tue, Sep 27, 2011 at 09:44:09AM +0200, Stefan Richter wrote:
> > Adding Cc: staging maintainer and mailinglist.
> >
> > On Sep 26 Devin Heitmueller wrote:
> > > On Sun, Sep 25, 2011 at 11:25 PM, Mauro Carvalho Chehab
> > >
ipped and
disabled.
There are obviously several already existing menuconfigs in the video section
which do not have these sentences; so maybe don't add the above, or add it
separately across the board, or whatever. I find these sentences helpful when
running "make ol
, but not many developers take the time
to do so.)
Again, thanks for the effort and also thanks to Devin for making it
possible.
--
Stefan Richter
-=-==-== =-=- =
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
rand[7] & 0x7f); i++) {
> - size <<= 8;
> - size += r->operand[8 + i];
> - }
> -#endif
> return r->operand[7];
> }
>
Hi Henrik,
get_ca_object_pos() is used in avc_ca_get_mmi() when the AVC response
nction
It is due to an actual bug but I don't know the fix.
--
Stefan Richter
-=-==-=- =-=- =
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On Jun 08 Mauro Carvalho Chehab wrote:
> The DVB API was originally defined using typedefs. This is against
> Kernel CodingStyle, and there's no good usage here. While we can't
> remove its usage on userspace, we can avoid its usage in Kernelspace.
[...]
Acked-by: Stefan Rich
an file path.
The F: drivers/media/firewire/ pattern and the "FireWire media drivers"
title are currently synonymous with firedtv. If more drivers get added
there, this can be revisited.
I don't have documentation or DVB-S2 devices to test, but I have DVB-C
and DVB-T devices for testing
:
> fdtv->demux.dmx.close(&fdtv->demux.dmx);
> fail_rem_frontend:
> fdtv->demux.dmx.remove_frontend(&fdtv->demux.dmx,
> &fdtv->frontend);
--
Stefan Richter
-=-==-== ==-- =
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
ncomplete element type
etc. pp.
--
Stefan Richter
-=-==-== ==-- =
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
s
# CONFIG_DVB_FE_CUSTOMISE is not set
# DVB-S (satellite) frontends
# DVB-T (terrestrial) frontends
# DVB-C (cable) frontends
# SEC control devices for DVB-S
# CONFIG_DVB_DUMMY_FE is not set
--
Stefan Richter
-=-===-- -=-= ==-=-
http://arcgraph.de/sr/
--
To unsubscribe from this list: send
and transmitters, whereas
Kconfig options in the analog and digital TV categories silently do
"select RC_CORE if INPUT" for combined tuner + IR/RF rx/tx hardware?
--
Stefan Richter
-=-===-- -=-= ===--
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscr
On May 28 Mauro Carvalho Chehab wrote:
> Em 28-05-2012 06:48, Stefan Richter escreveu:
> > c) The RC_CORE_SUPP help text gives the impression that RC core is
> > always needed if there is hardware with an IR feature. But the firedtv
> > driver is a case where the driver dir
"rc-winfast"
> #define RC_MAP_WINFAST_USBII_DELUXE "rc-winfast-usbii-deluxe"
> +#define RC_MAP_FIREDTV"rc-firedtv"
>
> /*
> * Please, do not just append newer Remote Controller names at the end.
The comment says that
1 - 100 of 136 matches
Mail list logo
