/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/omap3-n900.dts
>
> Is there any chance that this bcm2048 driver will be fixed and
> moved out of staging by the end of this year? If not, then I want
> to remove it.
FWIW I do _not_ plan to work on this, since I'm too busy with other
things (e.g. moving the wl128x radio driver to use the new hci_ll
driver).
-- Sebastian
signature.asc
Description: PGP signature
t.bin
> /lib/firmware/ti-connectivity/wl128x-fw-5-mr.bin
> /lib/firmware/ti-connectivity/wl128x-fw-5-sr.bin
> /lib/firmware/ti-connectivity/wl128x-nvs.bin
> root@devuan:/home/user#
>
> Ideas welcome... ... ... am I supposed to compile wl128-nvs.bin into
> the kernel us
Hi Pavel,
On Sat, Dec 22, 2018 at 08:29:34PM +0100, Pavel Machek wrote:
> On Fri 2018-12-21 02:17:48, Sebastian Reichel wrote:
> > From: Sebastian Reichel
> >
> > Remove unused return code from fmc_prepare() and fmc_release() to
> > simplify the code a bit.
>
&g
Hi Marcel,
First of all thanks for your review.
On Sun, Dec 23, 2018 at 04:56:47PM +0100, Marcel Holtmann wrote:
> Hi Sebastian,
[...]
> > +static int ll_register_fm(struct ll_device *lldev)
> > +{
> > + struct device *dev = &lldev->serdev->dev;
&g
Hi,
On Fri, Dec 21, 2018 at 03:10:52PM -0600, Adam Ford wrote:
> On Fri, Dec 21, 2018 at 2:13 AM Sebastian Reichel wrote:
> >
> > From: Sebastian Reichel
> >
> > This driver has been superseded by the serdev based Bluetooth
> > hci_ll driver, which is initia
Hi,
On Fri, Dec 21, 2018 at 10:02:05AM -0800, Tony Lindgren wrote:
> * Sebastian Reichel [181221 01:18]:
> > The new code has been tested on the Motorola Droid 4. For testing the audio
> > should be configured to route Ext to Speaker or Headphone. Then you need to
> > plug
From: Sebastian Reichel
Move global radio_disconnected into device structure to
prepare converting this driver into a normal platform
device driver supporting multiple instances.
Signed-off-by: Sebastian Reichel
---
drivers/media/radio/wl128x/fmdrv.h | 1 +
drivers/media/radio/wl128x
From: Sebastian Reichel
Drop module version. We already have the kernel's version and
this module is mainline.
Signed-off-by: Sebastian Reichel
---
drivers/media/radio/wl128x/fmdrv.h| 1 -
drivers/media/radio/wl128x/fmdrv_common.c | 5 ++---
2 files changed, 2 insertions(
From: Sebastian Reichel
Add a node for the UART part of WiLink chip.
Cc: Enric Balletbo i Serra
Signed-off-by: Sebastian Reichel
---
This is compile tested only!
---
arch/arm/boot/dts/omap3-igep0020-rev-f.dts | 8
arch/arm/boot/dts/omap3-igep0030-rev-g.dts | 8
2 files
From: Sebastian Reichel
All TI_ST users have been migrated to the new serdev based HCILL
bluetooth driver. That driver is initialized from DT and does not
need any platform quirks.
Signed-off-by: Sebastian Reichel
---
arch/arm/mach-omap2/pdata-quirks.c | 52 --
1
From: Sebastian Reichel
This driver has been superseded by the serdev based Bluetooth
hci_ll driver, which is initialized from DT. All mainline users
have been converted and this driver can be safely dropped.
Signed-off-by: Sebastian Reichel
---
drivers/misc/Kconfig | 1 -
drivers
From: Sebastian Reichel
Remove unused return code from fmc_prepare() and fmc_release() to
simplify the code a bit.
Signed-off-by: Sebastian Reichel
---
drivers/media/radio/wl128x/fmdrv_common.c | 26 +--
drivers/media/radio/wl128x/fmdrv_common.h | 4 ++--
drivers/media
From: Sebastian Reichel
This fixes incorrect code in the TX/RX skb debug print
function and add stubs in receive/transmit packet path.
Signed-off-by: Sebastian Reichel
---
drivers/media/radio/wl128x/fmdrv_common.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git
From: Sebastian Reichel
Move global radio_dev into device structure to prepare converting
this driver into a normal platform device driver supporting multiple
instances.
Signed-off-by: Sebastian Reichel
---
drivers/media/radio/wl128x/fmdrv.h| 2 +-
drivers/media/radio/wl128x
From: Sebastian Reichel
All users of this driver have been converted to the serdev based
hci_ll driver. The unused driver can be safely dropped now.
Signed-off-by: Sebastian Reichel
---
drivers/bluetooth/Kconfig| 11 --
drivers/bluetooth/Makefile | 1 -
drivers/bluetooth/btwilink.c
From: Sebastian Reichel
This converts the wl128x FM radio module into a platform device.
It's a preparation for using it from hci_ll Bluetooth driver instead
of TI_ST.
Signed-off-by: Sebastian Reichel
---
drivers/media/radio/wl128x/fmdrv_common.c | 30 ---
1 file ch
From: Sebastian Reichel
This simplifies memory allocation and removes a few useless
errors in case of -ENOMEM errors.
Signed-off-by: Sebastian Reichel
---
drivers/media/radio/wl128x/fmdrv_common.c | 41 +++
1 file changed, 13 insertions(+), 28 deletions(-)
diff --git a
From: Sebastian Reichel
Add a node for the UART part of WiLink chip.
Cc: Adam Ford
Signed-off-by: Sebastian Reichel
---
This is compile tested only!
---
arch/arm/boot/dts/logicpd-torpedo-37xx-devkit.dts | 8
1 file changed, 8 insertions(+)
diff --git a/arch/arm/boot/dts/logicpd
ce initial get_frequency returns an error:
$ radio -f 100.0
Merry Christmas!
-- Sebastian
Sebastian Reichel (14):
ARM: dts: LogicPD Torpedo: Add WiLink UART node
ARM: dts: IGEP: Add WiLink UART node
ARM: OMAP2+: pdata-quirks: drop TI_ST/KIM support
media: wl128x-radio: remove module versi
From: Sebastian Reichel
All TI WiLink firmware files are loaded from the ti-connectivity
subdirectory, so let's also move the FM firmware.
Signed-off-by: Sebastian Reichel
---
drivers/media/radio/wl128x/fmdrv_common.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --
From: Sebastian Reichel
This updates the wl128x-radio driver to be used with hci_ll
instead of the deprecated TI_ST driver.
Signed-off-by: Sebastian Reichel
---
drivers/bluetooth/hci_ll.c| 115 --
drivers/media/radio/wl128x/Kconfig| 2 +-
drivers
Ok, nevertheless thank your for your reply.
Sebastian Süsens Tel. +49 4321 559 56-27
mycable GmbH Fax+49 4321 559 56-10
Gartenstrasse 10
24534 Neumuenster, Germany Email
Hey,
I use the driver mx6s_capture kernel 4.9.88
On the device tree it is registered with following name "fsl,imx6s-csi".
Hint:
I have no sub-devices on my systems only /dev/video0
----
Sebastian Süsens
s...@mycable.de
----
Sebastian Süsens Tel. +49 4321 559 56-27
mycable GmbH Fax+49 4321 559 56-10
Gartenstrasse 10
24534 Neumuenster, Germany Email s...@mycable.de
mycable GmbH, M
re also in Greg's USB tree for the
> next merge-window.
Just ignore what I just wrote here. I didn't take my pills yet.
Sebastian
On 2018-09-12 12:00:36 [+], Mauro Carvalho Chehab wrote:
> This is an automatic generated email to let you know that the following patch
> were queued:
The three patches you just enqueued are also in Greg's USB tree for the
next merge-window.
Sebastian
772x_write(struct i2c_client *client, u8 addr, u8
> > value)
> > {
> > - return i2c_smbus_write_byte_data(client, addr, value);
> > + return sccb_write_byte(client, addr, value);
> > }
Reviewed-by: Sebastian Reichel
> Minor nit: I'd rather drop these t
ers.
>
> With the sccb_read_byte() that issues two separated requests in order to
> avoid repeated start, the driver doesn't require I2C_FUNC_PROTOCOL_MANGLING.
>
> Cc: Sebastian Reichel
> Cc: Wolfram Sang
> Cc: Jacopo Mondi
> Cc: Laurent Pinchart
> Cc: Hans Ve
Ailus
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> since v2.2:
>
> - Fix property name in code.
>
> .../devicetree/bindings/media/i2c/nokia,smia.txt | 2 ++
> drivers/media/i2c/smiapp/smiapp-core.c | 16
>
> 2 files chang
t
> order by setting both horizontal and vertical flipping.
>
> This patch documents the "rotation" property for camera sensors, mirroring
> what is defined for displays in
> Documentation/devicetree/bindings/display/panel/panel.txt .
>
> Signed-off-by: Sakari Ailu
+ /* Fall through */
> + case 0:
> + break;
> + default:
> + dev_err(dev, "invalid rotation %u\n", rotation);
> + goto out_err;
> + }
> + }
> +
> /* NVM size is not mandatory */
> fwnode_property_read_u32(fwnode, "nokia,nvm-size", &hwcfg->nvm_size);
-- Sebastian
signature.asc
Description: PGP signature
ns
> + */
> +
> +#ifndef __SCCB_H__
> +#define __SCCB_H__
> +
> +#include
> +
> +int sccb_read_byte(struct i2c_client *client, u8 addr);
> +int sccb_write_byte(struct i2c_client *client, u8 addr, u8 data);
> +
> +#endif /* __SCCB_H__ */
The functions look very simple. Have you considered moving them into
sccb.h as static inline?
-- Sebastian
signature.asc
Description: PGP signature
1
> N: video1
> E: DEVNAME=/dev/video1
> E:
> DEVPATH=/devices/pci:00/:00:14.0/usb1/1-8/1-8:1.0/video4linux/video1
> E: MAJOR=81
> E: MINOR=1
> E: SUBSYSTEM=video4linux
> sre@earth ~ % lsusb -d 04ca:703c
> Bus 001 Device 004: ID 04ca:703c Lite-On Technology Corp.
-- Sebastian
signature.asc
Description: PGP signature
ngs/display/panel/panel.txt
-- Sebastian
> ---
> Documentation/devicetree/bindings/media/video-interfaces.txt | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt
> b/Documentation/devicetree/bindings/me
or flash and lens devices")
> Signed-off-by: Arnd Bergmann
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/i2c/et8ek8/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/media/i2c/et8ek8/Kconfig
> b/drivers/media/i2c/et8ek8/Kconfig
> i
Hi,
On Sun, Nov 12, 2017 at 12:27:29PM +0100, Pavel Machek wrote:
>
> Document dts support of LED/focus.
>
> Signed-off-by: Pavel Machek
Reviewed-by: Sebastian Reichel
-- Sebastian
> diff --git a/Documentation/devicetree/bindings/media/i2c/toshiba,et8ek8.txt
> b/Docume
TS working with old kernel. Also we want it
working the other way around (old DTS with new kernel), so we
need to add the "nokia,n900-ir" compatible to the generic PWM
IR driver.
Also the DT binding document needs update:
Documentation/devicetree/bindings/media/nokia,n900-ir
[Adding Ro
used-but-set-variable]
>bool is_available;
> ^~~~
>
> Fixes: 9ca465312132 ("media: v4l: fwnode: Support generic parsing of graph
> endpoints in a device")
> Cc: Sakari Ailus
> Signed-off-by: Mauro Carvalho Chehab
> ---
Reviewed-by: Sebastian Reichel
-- S
Hi,
On Thu, Oct 26, 2017 at 10:53:42AM +0300, Sakari Ailus wrote:
> Add flash and indicator LED phandles to the sensor node.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> arch/
Hi,
On Thu, Oct 26, 2017 at 10:53:41AM +0300, Sakari Ailus wrote:
> Parse async sub-devices related to the sensor by switching the async
> sub-device registration function.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> ---
Reviewed-by: Sebastian Reichel
-- Seb
Hi,
On Thu, Oct 26, 2017 at 10:53:40AM +0300, Sakari Ailus wrote:
> Parse async sub-devices related to the sensor by switching the async
> sub-device registration function.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> ---
Reviewed-by: Sebastian Reichel
-- Seb
Hi,
On Thu, Oct 26, 2017 at 10:53:39AM +0300, Sakari Ailus wrote:
> Parse async sub-devices related to the sensor by switching the async
> sub-device registration function.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
> ---
Reviewed-
tion to do that.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/i2c/smiapp/smiapp-core.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/driver
Hi,
On Thu, Oct 26, 2017 at 10:53:37AM +0300, Sakari Ailus wrote:
> Document optional lens-focus and flash-leds properties for the smiapp
> driver.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
Reviewed-by: Sebastian Reic
This should be useful for sensor drivers that do not have device specific
> requirements related to firmware information parsing or the async
> framework.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> driver
regular driver needs to
> know about these devices as such.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
Reviewd-by: Sebastian Reichel
-- Sebastian
> ---
> since v16:
>
> - use const char * const *props for string arrays wit
igned-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> since v16:
>
> - use const char * const *props for string arrays with property names.
>
> drivers/media/v4l2-core/v4l2-fwnode.c | 287
>
-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/v4l2-core/v4l2-fwnode.c | 69
> +++
> 1 file changed, 69 insertions(+)
>
> diff --git a/drivers/media/v4l2-core/v4l2-fwnode
Verkuil
> Acked-by: Pavel Machek
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/v4l2-core/v4l2-fwnode.c | 75
> include/media/v4l2-fwnode.h | 81
> ++-
> 2 files changed, 80 insertions(
ilus
> Cc: Rob Herring
> Cc: devicet...@vger.kernel.org
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
> Acked-by: Rob Herring
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> Documentation/devicetree/bindings/media/video-interfaces.txt | 8
> 1 file cha
>
> - mutex_lock(&list_lock);
> -
> ret = v4l2_async_notifier_try_all_subdevs(notifier);
> - if (ret)
> + if (ret < 0)
> goto err_unbind;
the error path does no unlock?
> ret = v4l2_async_notifier_try_complete(notifier);
> - if (ret)
> + if (ret < 0)
> goto err_unbind;
dito
> + ret = 0;
>
> /* Keep also completed notifiers on the list */
> list_add(¬ifier->list, ¬ifier_list);
>
> +out_unlock:
> mutex_unlock(&list_lock);
>
> - return 0;
> + return ret;
>
> err_unbind:
> /*
Otherwise:
Reviewed-by: Sebastian Reichel
-- Sebastian
signature.asc
Description: PGP signature
2_device is available and no notifier has pending sub-devices to bind.
> No complete callbacks are supported for sub-device notifiers.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/v4l2-core/v4l2-asy
sier
> to review.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/v4l2-core/v4l2-async.c | 69
> ++--
> 1 file changed, 50 insertions(+), 19 deletions(-)
>
&g
d by the bound
> callback.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/v4l2-core/v4l2-async.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff
e async sub-devices available,
> it can be done by inspecting the notifier's num_subdevs field which tells
> the number of async sub-devices.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/v4l
Hi,
On Thu, Oct 26, 2017 at 10:53:22AM +0300, Sakari Ailus wrote:
> If no source pads are found in an entity, print the name of the entity.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
gt; Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/v4l2-core/v4l2-async.c | 56
> +---
> 1 file changed, 39 insertions(+), 17 deletions(-)
>
> diff --git a/drivers/media/v4l2-co
able, this creates a potential
> security risk as the function pointers are mutable.
>
> To fix this, move the function pointers to a new
> v4l2_async_subdev_operations structure that can be made const in
> drivers.
>
> Signed-off-by: Laurent Pinchart
> Acked-by: Hans Ve
-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/platform/omap3isp/isp.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/omap3i
Reviewed-by: Laurent Pinchart
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/platform/omap3isp/isp.c | 121
> +++---
> drivers/media/platform/omap3isp/isp.h | 5 +-
> 2 files changed, 40 insertions(+), 86 deletions(-)
>
test_notify and undo the work that was done either in async
> sub-device or async notifier registration.
>
> Reported-by: Russell King
> Signed-off-by: Sakari Ailus
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/v4l2-core/v4l2-async.c | 78
> ++
Hi,
On Thu, Oct 05, 2017 at 12:50:26AM +0300, Sakari Ailus wrote:
> The V4L2 async wasn't part of the documentation build. Fix this.
>
> Signed-off-by: Sakari Ailus
> Reviewed-by: Niklas Söderlund
> Acked-by: Hans Verkuil
> Reviewed-by: Laurent Pinchart
Reviewe
v4l2_async_belongs v4l2_async_find_match
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Pavel Machek
> Reviewed-by: Laurent Pinchart
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/media/v4l2-core/v4l2-async.c | 19 ++---
gt; proceeds to take the lock.
>
> Fix this by first acquiring the lock and then proceeding with the check.
>
> Signed-off-by: Sakari Ailus
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/media/v4l2-core/v4l2-async.c | 18 +++---
> 1 file chang
or to cleaning up the subdevice to avoid this.
>
> Signed-off-by: Niklas Söderlund
> Signed-off-by: Sakari Ailus
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/media/v4l2-core/v4l2-async.c | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
&
; Signed-off-by: Sakari Ailus
> ---
Reviewed-by: Sebastian Reichel
-- Sebastian
> drivers/media/v4l2-core/v4l2-async.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/media/v4l2-core/v4l2-async.c
> b/drivers/media/v4l2-core/v4l2-async.c
> index 60a1a50b9537.
c locks.
> Considering the complexity of the re-probing and that it isn't really a
> solution to a problem but a workaround, remove re-probing instead.
>
> Signed-off-by: Sakari Ailus
> Acked-by: Hans Verkuil
> Acked-by: Laurent Pinchart
> ---
Reviewed-by: Sebastian
48 8b 78 10
[ 11.147366] RIP: dw2102_disconnect+0x1a/0x70 [dvb_usb_dw2102] RSP:
b75841bb7c28
[ 11.147433] CR2: 0050
[ 11.147489] ---[ end trace 9643f635d6b5f94e ]---
Sebastian
buscfg->bus.ccp2.lanecfg.data[0].pol =
> vep.bus.mipi_csi1.lane_polarity[1];
>
> - dev_dbg(dev, "data lane %u polarity %u, pos %u\n", i,
> + dev_dbg(dev, "data lane polarity %u, pos %u\n",
> buscfg->bus.ccp2.lanecfg.data[0].pol,
> buscfg->bus.ccp2.lanecfg.data[0].pos);
Reviewed-by: Sebastian Reichel
-- Sebastian
signature.asc
Description: PGP signature
Hi,
On Tue, Jul 18, 2017 at 01:01:11AM +0300, Sakari Ailus wrote:
> From: Pavel Machek
>
> Add support for parsing CSI1 configuration.
>
> Signed-off-by: Pavel Machek
> Signed-off-by: Sakari Ailus
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/media
I wonder if the right approach wouldn't be to always bail out for
errors. devm_regulator_get should provide a dummy regulator if
none is specified. If we get an error here it means something is
configured incorrectly or we have serious problems.
> dev_dbg(isp->dev,
> "Could not get regulator vdds_csib\n");
> ccp2->vdds_csib = NULL;
-- Sebastian
signature.asc
Description: PGP signature
are
> bad.
I would expect a dev_warn(dev, "Ignore endpoint (broken configuration)!");
-- Sebastian
> Signed-off-by: Sakari Ailus
> Tested-by: Pavel Machek
> ---
> drivers/media/platform/omap3isp/isp.c | 8 +---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
Hi,
On Thu, Jul 06, 2017 at 02:00:18AM +0300, Sakari Ailus wrote:
> Check that we do have a valid port in an endpoint, return an error if not.
>
> Signed-off-by: Sakari Ailus
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/media/platform/omap3isp/isp.c | 2 +-
&g
Hi,
On Thu, Jul 06, 2017 at 02:00:16AM +0300, Sakari Ailus wrote:
> From: Sakari Ailus
>
> CCP2 and CSI-1, are older single data lane serial busses.
>
> Signed-off-by: Sakari Ailus
> Signed-off-by: Pavel Machek
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
>
i Ailus
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/media/v4l2-core/v4l2-fwnode.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/media/v4l2-core/v4l2-fwnode.c
> b/drivers/media/v4l2-core/v4l2-fwnode.c
> index 15
Hi,
On Thu, Jul 06, 2017 at 02:00:15AM +0300, Sakari Ailus wrote:
> From: Sakari Ailus
>
> Just obtain it. It'll actually get used soon with CSI-1/CCP2.
>
> Signed-off-by: Sakari Ailus
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/media/v4
Sorry for the long delay, Johan.
2017-06-01 9:20 GMT+02:00 Johan Hovold :
> [ +CC: media list ]
>
> On Wed, May 31, 2017 at 08:25:42PM +0200, Sebastian wrote:
>
> What is the lsusb -v output for your device? And have you successfully
> used this device with this driver before?
Hi,
On Thu, Jun 15, 2017 at 03:32:10PM +0300, Sakari Ailus wrote:
> On Thu, Jun 15, 2017 at 11:24:26AM +0200, Sebastian Reichel wrote:
> > On Wed, Jun 14, 2017 at 12:47:16PM +0300, Sakari Ailus wrote:
> > > None of the flash operations are not mandatory and therefore there shoul
; Signed-off-by: Sakari Ailus
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> drivers/media/v4l2-core/v4l2-flash-led-class.c | 19 +++
> include/media/v4l2-flash-led-class.h | 6 +++---
> 2 files changed, 10 insertions(+), 15 deletions(-)
>
> dif
Hi,
On Wed, Jun 14, 2017 at 12:47:19PM +0300, Sakari Ailus wrote:
> From: Sakari Ailus
>
> Add the as3645a flash controller to the DT source as well as the flash
> property with the as3645a device phandle to the sensor DT node.
>
> Signed-off-by: Sakari Ailus
Reviewed-by:
Hi,
On Wed, Jun 14, 2017 at 12:47:16PM +0300, Sakari Ailus wrote:
> None of the flash operations are not mandatory and therefore there should
> be no need for the flash ops structure either. Accept NULL.
I think you negated one time too much :). Otherwise:
Reviewed-by: Sebastian R
s
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> Documentation/devicetree/bindings/media/video-interfaces.txt | 8
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt
> b/Documentation/devicetree/bi
ugh
> the mbus configuration of the active input to the output side.
>
> Signed-off-by: Sascha Hauer
> Signed-off-by: Philipp Zabel
> Signed-off-by: Steve Longerbeam
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> No changes since v4 [1]:
>
> This patch depends
>
> Signed-off-by: Sakari Ailus
> Acked-by: Pavel Machek
Reviewed-by: Sebastian Reichel
> diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt
> b/Documentation/devicetree/bindings/media/video-interfaces.txt
> index 0a33240..38e3916 100644
> --- a/Documen
d-by: Pavel Machek
Reviewed-by: Sebastian Reichel
> ---
> Documentation/devicetree/bindings/media/video-interfaces.txt | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt
> b/Documentation/devicetree/binding
Hi,
On Fri, May 05, 2017 at 11:28:34AM +0300, Sakari Ailus wrote:
> Sebastian Reichel wrote:
> > On Tue, May 02, 2017 at 01:25:47PM +0300, Sakari Ailus wrote:
> > > +- flash: An array of phandles that refer to the flash light sources
> > > + related to an image senso
eady in the first sentence :) Instead it
should be mentioned, that this is to be used by devices not having
their own embedded eeprom. How about:
eeprom: A phandle to the node of the EEPROM describing the camera
sensor (i.e. device specific calibration data), in case it differs
from the sensor node.
-- Sebastian
signature.asc
Description: PGP signature
related to an image sensor.
>
> +- lens-focus: A phandle to the node of the lens. Only valid for device
> + nodes that are related to an image sensor.
s/of the lens/of the focus lens controller/
Also I wonder about the second sentence. That sentence could
basically be adde
LED, then the phandles here refer to
> + the child nodes of the LED driver describing individual LEDs. Only
> + valid for device nodes that are related to an image sensor.
s/driver/controller/g - DT describes HW. Otherwise
Reviewed-by: Sebastian Reichel
-- Sebastian
signature.asc
Description: PGP signature
*vmux->format_mbus) *
> + num_pads, GFP_KERNEL);
> + vmux->endpoint = devm_kzalloc(dev, sizeof(*vmux->endpoint) *
> + (num_pads - 1), GFP_KERNEL);
devm_kcalloc(dev, num_pads, sizeof(*foo), GFP_KERNEL).
-- Sebastian
signature.asc
Description: PGP signature
i Ailus
Reviewed-by: Sebastian Reichel
-- Sebastian
> ---
> No changes since v2 [1].
>
> This was previously sent as part of Steve's i.MX media series [2].
>
> [1] https://patchwork.kernel.org/patch/9708235/
> [2] https://patchwork.kernel.org/patch/9647951/
>
>
> > > Change to PIN_INPUT so it can be used by the new CEC support.
> ...
>
> > Reviewed-by: Tomi Valkeinen
> >
> > Tony, can you queue this? It's safe to apply separately from the rest of
> > the HDMI CEC work.
>
> Sure will do.
I gu
630 instead of csiphy_routing_cfg_3430
and N900 has OMAP3430. Function should be called when you start (or
stop) using the camera:
csiphy_routing_cfg_3430(...)
csiphy_routing_cfg(...)
omap3isp_csiphy_config(...)
omap3isp_csiphy_acquire(...) & omap3isp_csiphy_release(...)
ccp2_s_stream(...)
-- Sebasti
Hi,
On Tue, Feb 14, 2017 at 02:39:56PM +0100, Pavel Machek wrote:
> From: Sebastian Reichel
>
> Without this, camera support breaks boot on N900.
That's kind of vague. I just checked my original patch and it looks
like I did not bother to write a proper patch description. I s
r Julien will want a different solution, but
> then... they'll have to suggest something doable now, not in couple of
> months.
>
> Does that sound like a plan?
>
> Mauro added to cc list, so we can get some input.
side note: It's an kernel-internal API only used by the media
subsystem. Code can be easily updated to use an improved API
at any time.
-- Sebastian
signature.asc
Description: PGP signature
.uni-oldenburg.de/fileadmin/user_upload/informatik/ag/svs/download/thesis/Reichel_Sebastian.pdf
The schematic is on page 37 (or 45 if your PDF reader does not
use different numbers for the preamble stuff).
--Sebastian
signature.asc
Description: PGP signature
interpreted
>as 0 (normal). This property is valid for serial busses only.
> -
> +- strobe: Whether the clock signal is used as clock or strobe. Used
> + with CCP2, for instance.
>
> Example
> ---
>
>
Reviewed-By: Sebastian Reichel
-- Sebastian
signature.asc
Description: PGP signature
rivers/power/supply/rx51_battery.c
TI BQ27XXX POWER SUPPLY DRIVER
R: Andrew F. Davis
F: include/linux/power/bq27xxx_battery.h
F: drivers/power/supply/bq27xxx_battery.c
F: drivers/power/supply/bq27xxx_battery_i2c.c
POWER SUPPLY CLASS/SUBSYSTEM and DRIVERS
M: Sebastian Reichel
L:
Hi,
On Thu, Dec 22, 2016 at 11:42:26PM +0100, Pavel Machek wrote:
> On Thu 2016-12-22 15:32:44, Sebastian Reichel wrote:
> > Hi Pavel,
> >
> > On Thu, Dec 22, 2016 at 02:39:38PM +0100, Pavel Machek wrote:
> > > N900 contains front and back camera, with a switch be
ement the remaining
> > bits.
>
> :-).
>
> > (*) Actually it does not for CCP2, but there are some old patches
> > from Sakari adding it for CCP2. It is implemented for parallel port
> > and CSI in this way.
>
> I think I got the patches in my tree. Camera currently works for me.
If you have working camera you have the CCP2 DT patches in your tree.
They are not yet mainline, though. As far as I can see.
-- Sebastian
signature.asc
Description: PGP signature
1 - 100 of 226 matches
Mail list logo
