29 Mar, 2013
18 commits
-
ioctl's number is unsigned int. Fix it at vidioc_default.
Signed-off-by: Mauro Carvalho Chehab
-
Correct spelling typo in staging/media/davinci
Signed-off-by: Masanari Iida
Signed-off-by: Lad, Prabhakar
Signed-off-by: Mauro Carvalho Chehab -
This sensor is used by the "SpeedLink Vicious And Devine Laplace webcam" and
others. It supports resolutions up to 1600x1200 (at 7-8 fps), but for
resolutions higher than 640x480, further driver changes will be necessary,
such as sensor output resolution switching (including further configuration
changes), bridge xclk adjustment and disabling of 16 bit (12 bit) output formats
at high resolutions. Image quality should also needs to be improved.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
The Windows driver also probes at least two further i2c addresses (0x22 >> 1
and 0x66 >> 1). I've got some hints that they are very likely used by Samsung
and Kodak sensors, which are known to be used in Empia devices, too.
We havn't seen any devices using these sensors yet and don't know how to probe
them properly, so leave a comment.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
OmniVision sensors are used as well in Empiatech based cameras such as the
"SpeedLink Vicious And Devine Laplace" webcam (EM2765 + Omnivision OV2640).
With this patch applied, OminiVision sensors with 8 bit address and register
width are detected (recent models have a 16 bit address width and use different
client addresses).
The most commonly used sensors (including the ones listed by Empiatech) are
detected properly, although there is no support for them yet.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
Other sensors like the ones from OmniVision need a different probing procedure,
so it makes sense have separate functions for each manufacturer/sensor type.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
Add further Micron chip IDs to be able to identify all Micron sensors listed
by Empiatech.
Also probe the two alternate i2c addresses used by Micron sensors with 8 bit
address and 16 bit register width.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
em28xx-cards.c is very large and the sensor/camera related code is growing,
so move this code to a separate source code file em28xx-camera.c.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
Now that the board hints and the sensor initialization/configuration have been
separated, em28xx_detect_sensor() is the better name for this function.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
Sensor detection and initialization/configuration are currently mixed together.
This works as long as all devices with a particular sensor are working with the
same board configuration. In the long run, this will be not sufficient, so
separate these both steps to make the code more flexible and future proof.
This also makes the code more consistent, because the initialization of the
MT9V011 sensor subdevice is already separated.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
The current board hint code is mixed together with the sensor detection and
initialization code. It actually selects a board depending on the detected
sensor type only, with the result that 3 of the 6 webcam boards are currently
dead.
Separate it and move it to em28xx_hint_board() which already contains the board
hints for analog capturing+TV and DVB devices.
This way, we have all board hints at a common place which makes it easier
to extend the code and reduces the risk of regressions.
It also makes it possible again to use the boards EM2750_BOARD_DLCW_130,
EM2820_BOARD_VIDEOLOGY_20K14XUSB and EM2860_BOARD_NETGMBH_CAM (using the module
parameter "card").
NOTE: the current board hint logic for webcams is preserved. Not more not less.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
Some devices without DVB support (such as the "Terratec Grabby" and
"Easycap DC-60") provide isochronous DVB USB endpoints with wMaxPacketSize set
to 0 bytes for all alt settings.
Ignore these endpoints and avoid registering a DVB device node and loading the
DVB driver extension.Signed-off-by: Frank Schäfer
Tested-by: Timo Teräs
Signed-off-by: Mauro Carvalho Chehab -
-ERANGE is the right error code when the value is outside of the menu
range, but -EINVAL must be reported for invalid values inside the range.Signed-off-by: Laurent Pinchart
Acked-by: Hans Verkuil
Signed-off-by: Mauro Carvalho Chehab -
The Windows driver writes the output resolution to registers 0x34 (width / 16)
and 0x35 (height / 16) always.
We don't know yet what these registers are used for.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
Streaming doesn't work with the EM2765 if bit 5 of the output format register
0x27 is set.
It's actually not clear if really has to be set for the other chips, but for
now let's keep it to avoid regressions and add a comment to the code.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
The em25xx/em276x/em277x/em278x frame data format is different to the one used
by the em2710/em2750/em28xx chips.
With the recent cleanups and reorganization of the frame data processing code it
can be easily extended to support these devices.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
This chip can be found in the SpeedLink VAD Laplace webcam (1ae7:9003 and 1ae7:9004).
Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab -
The webcam "SpeedLink VAD Laplace" (em2765 + ov2640) uses a special algorithm
for i2c communication with the sensor, which is connected to a second i2c bus.
We don't know yet how to find out which devices support/use it.
It's very likely used by all em25xx and em276x+ bridges.
Tests with other em28xx chips (em2820, em2882/em2883) show, that this
algorithm always succeeds there although no slave device is connected.
The algorithm likely also works for real i2c client devices (OV2640 uses SCCB),
because the Windows driver seems to use it for probing Samsung and Kodak
sensors.Signed-off-by: Frank Schäfer
Signed-off-by: Mauro Carvalho Chehab
26 Mar, 2013
22 commits
-
au8522_attach is dependent on CONFIG_DVB_AU8522_DTV, not CONFIG_DVB_AU8522.
Signed-off-by: Hans Verkuil
Signed-off-by: Mauro Carvalho Chehab -
sizeof() when applied to a pointer typed expression gives the
size of the pointer, not that of the pointed data.Signed-off-by: Wei Yongjun
Acked-by: Hans Verkuil
Signed-off-by: Mauro Carvalho Chehab -
Add the missing unlock before return from function af9035_ctrl_msg()
in the error handling case.Signed-off-by: Wei Yongjun
Acked-by: Antti Palosaari
Reviewed-by: Antti Palosaari
Signed-off-by: Mauro Carvalho Chehab -
If those callbacks fail, it should return zero, and not a random
value. The previous code assumed that all drivers would only change
signal strength if it succeeds, but this may not be true.Signed-off-by: Mauro Carvalho Chehab
-
Now that those functions are only used when the corresponding
function calls are defined, we don't need to check if those
function calls are present at the structure before using it.Signed-off-by: Mauro Carvalho Chehab
-
While the driver gets AFC from the tuner, it doesn't return it
back via V4L2 API due to a mistake at the return. fix it.Signed-off-by: Mauro Carvalho Chehab
-
If the tuner frontend does not support get_rf_strength, then don't set
the has_signal callback. Ditto for get_afc.
Both callbacks overwrite the signal and afc fields of struct v4l2_tuner
but that should only happen if the tuner can actually detect this. If
it can't, then it should leave those fields alone so other subdevices
can try and detect the signal/afc.
This fixes the bug where the au8522 detected a signal and then tuner-core
overwrote it with 0 since the xc5000 tuner does not support get_rf_strength.Signed-off-by: Hans Verkuil
Signed-off-by: Mauro Carvalho Chehab -
- open/close/read and poll need to take the core lock as well.
- when the tuner goes to sleep we should set std_set_in_tuner_core
to 0 since the tuner loses the firmware at that time.
- initialize the tuner if std_set_in_tuner_core == 0 whenever:
1) g/s_tuner, s_std or s_frequency is called
2) read or poll is called
3) streamon is calledSigned-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
After loading the module the initial video routing is not setup.
Explicitly call s_input to get this right.Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
Just opening a device shouldn't have any side-effects.
Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
Turn it into a simple function.
Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
The driver crashed when the device was disconnected while an application
still had a device node open. Fixed by using the release() callback of struct
v4l2_device.Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
Now that au0828 has been converted to the control framework these
compatilibity ops are no longer needed.Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
Also get rid of unnecessary format type check.
Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
- V4L2_CAP_AUDIO was set, but enumaudio was not implemented.
- audioset was never filled by enum_input
- ctrl_ainput was never updated when switching the video input
- g_audio was broken due to faulty logic: g_audio should set the
index, it doesn't receive it from the user.Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
No code change, just fixing a wrong indentation.
Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
- define an initial frequency
- return an error if g_frequency is called for an invalid tuner index
- get the clamped frequency value after setting it: i.e. the tuner driver
may clamp the given frequency to a valid frequency range and ctrl_freq
should get that actual clamped frequency.
- remove obsolete tuner type checks (done by the core).Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab -
Signed-off-by: Hans Verkuil
Reviewed-by: Devin Heitmueller
Signed-off-by: Mauro Carvalho Chehab
