--
Уважаемый пользователь,
Ваш почтовый ящик превысил обычный 100 Мб хранения, вы не сможете
получать или отправлять электронную почту, пока не увеличите квоту
почтового ящика. Чтобы увеличить квоту почты, нажмите на ссылку ниже и
заполните необходимые данные, чтобы увеличить квоту
Signed-off-by: Javier Martinez Canillas
---
Changes in v2:
- Just move ov2680_check_id() before ov2680_v4l2_init() - Suggested by Sakari.
drivers/media/i2c/ov2680.c | 14 --
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/drivers/media/i2c/ov2680.c b/drivers/media/i2c/ov26
Hi Sakari,
Thanks for the feedback.
On 09/01/2018 01:46 PM, Sakari Ailus wrote:
> Hi Javier,
>
> On Fri, Aug 31, 2018 at 05:19:06PM +0200, Javier Martinez Canillas wrote:
>> The driver registers the subdev async in the middle of the probe function
>> but this has to be
Hi Marco,
On 08/01/2018 05:49 PM, Marco Felsch wrote:
> Hi Javier,
>
> On 18-07-31 15:30, Marco Felsch wrote:
>> Hi Javier,
>>
>> On 18-07-31 14:52, Javier Martinez Canillas wrote:
>>> Hi Marco,
>>>
>>> On 07/31/2018 02:36 PM, Marco Felsch wr
y I said that the output port
should be the first one in your enum tvp5150_ports and not the last one.
> Regards,
> Marco
>
Best regards,
--
Javier Martinez Canillas
Software Engineer - Desktop Hardware Enablement
Red Hat
Hi Mauro,
On 07/31/2018 12:06 PM, Mauro Carvalho Chehab wrote:
> Em Tue, 31 Jul 2018 10:52:56 +0200
> Javier Martinez Canillas escreveu:
>
>> Hello Mauro,
>>
>> On 07/30/2018 08:18 PM, Mauro Carvalho Chehab wrote:
>>> Em Thu, 28 Jun 2018 18:20
Hello Mauro,
On 07/30/2018 08:18 PM, Mauro Carvalho Chehab wrote:
> Em Thu, 28 Jun 2018 18:20:50 +0200
> Marco Felsch escreveu:
>
>> From: Javier Martinez Canillas
>>
>> Commit f7b4b54e6364 ("[media] tvp5150: add HW input connectors support")
>>
https://www.kernel.org/doc/Documentation/devicetree/bindings/media/video-interfaces.txt
[1]: https://www.kernel.org/doc/Documentation/devicetree/bindings/graph.txt
[2]: https://www.kernel.org/doc/Documentation/acpi/dsd/graph.txt
[3]: https://linuxtv.org/downloads/v4l-dvb-apis/kapi/v4l2-fwnode.html
[4]: https://www.kernel.org/doc/Documentation/acpi/dsdt-override.txt
[5]: https://www.kernel.org/doc/Documentation/acpi/initrd_table_override.txt
Best regards,
Javier
for my 2-in-1.
>
> Great, I'll add you to the list of potential testers for an ATOMISP solution
> :-)
>
The ATOMISP driver was removed from staging by commit 51b8dc5163d
("media: staging: atomisp: Remove driver"). Do you mean that there's a
plan to bring that driver back?
Best regards,
Javier
2].
>
Thanks for working on this! I've felt guilty that I never re-worked my
patch-set after the discussion from [2].
I'll try to review these patches next week.
Best regards,
--
Javier Martinez Canillas
Software Engineer - Desktop Hardware Enablement
Red Hat
subdev8
pad0: Sink
pad1: Source
[fmt:UYVY2X8/720x240 field:alternate
crop.bounds:(0,0)/720x480
crop:(0,0)/720x480]
-> "OMAP3 ISP CCDC":0 [ENABLED]
Best regards,
--
Javier Martinez Canillas
Software Engineer - Desktop Hardware Enablement
Red Hat
egrate them in a particular camera stack, and should be as
>> platform-agnostic as possible.
>>
>> In the case of the Android camera HAL, ideally it would be a glue that
>> could be used with different camera vendors (probably with some kind of
>> vendor-specific configuration
Location yet to be defined, but it will either be together with
>>>> OSS Japan or at Google. I'll confirm the address tomorrow.
>>>
>>> More details about the meeting:
>>>
>>> Date: June, 19
>>> Site: Google
>>> Address: 〒106-6126 Tokyo, Minato, Roppongi, 6 Chome−10−1 Roppongi Hills
>>> Mori Tower 44F
>>>
>>> Please confirm who will be attending the meeting.
>>
>> I plan to attend the meeting via Google Hangouts.
>
> Well, the afternoon part of the meeting at least :-)
>
I also plan to attend via Google Hangouts. At least the hours that are
reasonable on my timezone.
Best regards,
Javier
Sorry for resurrecting this thread but I'm still quite interested on
making this scenario work:
> OK, I've performed some tests with several resolutions and gop sizes,
here is the table with the results:
>
> Always playing 3 streams
>
> | Resolution | QP | GopSize | Kind of content |
Hello Philipp,
On 14/03/18 16:11, Philipp Zabel wrote:
Hi Javier,
On Wed, 2018-03-14 at 15:35 +0100, Javier Martin wrote:
[...]
The encoder is running on a different system with an older 4.1.0 kernel.
Altough the firmware version in the code is 3.1.1 as well.
Do you think I should try
Hello,
On 14/03/18 14:57, Philipp Zabel wrote:
On Wed, 2018-03-14 at 13:05 +0100, Javier Martin wrote:
Sorry everyone about my previous e-mail with all the HTML garbage. Here
is the plain text answer instead.
Hi Philipp,
thanks for your answer.
On 13/03/18 12:20, Philipp Zabel wrote
Sorry everyone about my previous e-mail with all the HTML garbage. Here
is the plain text answer instead.
Hi Philipp,
thanks for your answer.
On 13/03/18 12:20, Philipp Zabel wrote:
> Hi Javier,
>
> On Mon, 2018-03-12 at 17:54 +0100, Javier Martin wrote:
>> Hi,
>> we hav
y kind of Coda IP documentation it's quite hard to me to
perform any additional debugging.
Has anyone experienced these kind of performance issues too? I'm open to
any suggestions and willing to perform extra tests to get to the bottom
of this.
Regards,
Javier.
[1] https://github.com/genesi/i
Hello Mauro,
On 12/14/2017 06:02 PM, Mauro Carvalho Chehab wrote:
> Em Wed, 6 Dec 2017 01:33:05 +0100
> Javier Martinez Canillas <javi...@redhat.com> escreveu:
>
>> Commit f7b4b54e6364 ("[media] tvp5150: add HW input connectors support")
>> added
Hello,
I have scanned the DVB-T channels of Zaragoza, Spain
I wanted to send them to share and that may be useful for other users.
Javier García
---
El software de antivirus Avast ha analizado este correo electrónico en busca de
virus.
https://www.avast.com/antivirus
nly anymore. But all the code
related to the DT parsing logic and input entities creation are removed.
Suggested-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Signed-off-by: Javier Martinez Canillas <javi...@redhat.com>
Acked-by: Laurent Pinchart <laurent.pinch...@ideasonboa
Hello Philipp,
On Thu, Aug 17, 2017 at 3:05 PM, Philipp Zabel <p.za...@pengutronix.de> wrote:
> Hi,
>
> On Tue, 2016-12-13 at 12:39 -0300, Javier Martinez Canillas wrote:
>> Commit f7b4b54e6364 ("[media] tvp5150: add HW input connectors support")
>> added
ken w.r.t reporting a proper MODALIAS for OF
registered devices, it will report a MODALIAS=i2c:as3645a in this
case. So if you build this as a module, autoload won't work.
In theory this will be fixed soon, but for now you should add a
i2c_device_id table and export it with MODULE_DEVICE_TABLE(i2c,...) if
you care about module autoloading.
Best regards,
Javier
On 08/09/2017 01:05 PM, Kieran Bingham wrote:
> On 09/08/17 11:58, Javier Martinez Canillas wrote:
>> Hi Kieran,
>>
>> On 08/09/2017 12:29 PM, Kieran Bingham wrote:
>>> Hi Javier,
>>>
>>> Thankyou for the patch
>>
>> You are welcome
Hi Kieran,
On 08/09/2017 12:29 PM, Kieran Bingham wrote:
> Hi Javier,
>
> Thankyou for the patch
>
You are welcome.
> On 09/08/17 10:37, Javier Martinez Canillas wrote:
>> The I2C core always reports a MODALIAS of the form i2c: even if the
>> device was regist
/adv748x/adv748x.ko | grep alias
alias: of:N*T*Cadi,adv7482C*
alias: of:N*T*Cadi,adv7482
alias: of:N*T*Cadi,adv7481C*
alias: of:N*T*Cadi,adv7481
alias: i2c:adv7482
alias: i2c:adv7481
Signed-off-by: Javier Martinez Canillas <javi...@redhat.
On Thu, Jul 13, 2017 at 5:47 PM, Javier Martinez Canillas
<jav...@dowhile0.org> wrote:
> On Thu, Jul 13, 2017 at 5:38 PM, Sakari Ailus <sakari.ai...@iki.fi> wrote:
[snip]
>>
>> Shouldn't these be set to the corresponding driver structs' id_table
>&g
vimc_sen_driver_ids[] = {
^~~
Reported-by: Mauro Carvalho Chehab <mche...@s-opensource.com>
Suggested-by: Sakari Ailus <sakari.ai...@iki.fi>
Signed-off-by: Javier Martinez Canillas <javi...@redhat.com>
---
drivers/media/
ecause the platform subsystem fallbacks
to the driver's name if an .id_table isn't defined:
http://elixir.free-electrons.com/linux/latest/source/drivers/base/platform.c#L964
Best regards,
Javier
Hello Gustavo,
On 04/06/2017 10:08 AM, Gustavo Padovan wrote:
> Hi Javier,
>
> 2017-04-05 Javier Martinez Canillas <jav...@osg.samsung.com>:
>
>> Hello Gustavo,
>>
>> On 04/05/2017 11:09 AM, Gustavo Padovan wrote:
>>> 2017-04-03 Javier Martinez Cani
Hello Gustavo,
On 04/05/2017 11:09 AM, Gustavo Padovan wrote:
> 2017-04-03 Javier Martinez Canillas <jav...@osg.samsung.com>:
>
>> Hello Mauro and Gustavo,
>>
>> On 04/03/2017 07:16 AM, Mauro Carvalho Chehab wrote:
>>> Hi Gustavo,
>>>
>>&g
2016/4/12/983
[1]: https://hastebin.com/kadagidino.diff
[2]: https://patchwork.kernel.org/patch/9472623/
Best regards,
Javier
ke to do some tests with it on devices with mem2mem drivers.
> My plan is to use an Exynos board for such thing, but I guess that
> the DRM driver for it currently doesn't. I'm seeing internally if someone
> could be sure that Exynos driver upstream will become ready for such
> tests.
>
!= VB2_MEMORY_DMABUF)
> + return -EINVAL;
> +
I wonder if is correct to check this. Only one side of the pipeline uses
V4L2_MEMORY_DMABUF while the other uses V4L2_MEMORY_MMAP + VIDIOC_EXPBUF.
So that means that fences can only be used in one direction?
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
be able to help you since as
mentioned I didn't test that.
> Cheers,
> Marian
>
Best regards,
Javier
age of "media: s5p-mfc: Add support for probe-time
preallocated block based allocator".
[0]: https://patchwork.kernel.org/patch/9596737/
> Thanks,
> Marian
>
Best regards,
Javier
9a7bb83f0d8 ]---
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> Fixes: af93574678108 ("[media] MFC: Add MFC 5.1 V4L2 driver")
> CC: sta...@vger.kernel.org # v4.5+
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
gt;
>
Thanks for the patch, but Krzysztof sent the exact same patch before [0]. There
was feedback from Sylwester at the time that you can also look at [0]. Could you
please take that into account and post a patch according to what he suggested?
[0]:
http://lists.infradead.org/pipermail
hat case both the left and right device is treated as mfc0 only.
>
I see, so likely you were facing the issue described in patch 1/2 after this
patch since the driver doesn't set alloc_devs when IOMMU is disabled, right?
In any case, I guess these patches have been superseded by Marek's se
;
> q->ops = get_dec_queue_ops();
>
Please correct me if I'm wrong, but AFAIU this shouldn't be needed in
the s5p-mfc driver since the videobuf2 core either uses the vb2_queue
.dev field or the vb2_queue .alloc_devs. And the latter is set in the
s5p_mfc_queue_setup() function, so the .dev field shouldn't be used.
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
change in the future so the correct approach is to have an
OF device ID table if the devices are registered via OF.
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
drivers/media/i2c/soc_camera/ov5642.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers
change in the future so the correct approach is to have an
OF device ID table if the devices are registered via OF.
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
drivers/media/i2c/tc358743.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/med
change in the future so the correct approach is to have an
OF device ID table if the devices are registered via OF.
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
drivers/media/radio/si4713/si4713.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers
-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
drivers/media/i2c/et8ek8/et8ek8_driver.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/i2c/et8ek8/et8ek8_driver.c
b/drivers/media/i2c/et8ek8/et8ek8_driver.c
index bec4a563a09c..f39f5179dd95 100644
--- a/drivers/media/i2c/
irmware() call
> from the s5p_mfc_probe() function to the s5p_mfc_configure_dma_memory().
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
Hello Marek,
On 02/20/2017 10:23 AM, Marek Szyprowski wrote:
> Hi Javier,
>
> On 2017-02-17 22:13, Javier Martinez Canillas wrote:
>> On 02/14/2017 04:52 AM, Marek Szyprowski wrote:
>>> It turned out that all versions of MFC v6+ hardware doesn't have a strict
>>
2dec capture-io-mode=dmabuf ! v4l2video0convert \
output-io-mode=dmabuf-import capture-io-mode=dmabuf ! kmssink
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Tested-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
v6+ devices in device tree. Now the driver is fully
> functional without them.
>
Great!
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
The patch looks good to me though and it works on my tests,
I've just one comment below.
Reviewed-by: Javier Martinez Canil
gt; specific DMA-mapping and IOMMU glue code functions.
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Tested-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
er to force larger (or smaller) buffer (for example by adding
> "s5p_mfc.mem=16M" to kernel command line).
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Tested-by: Javier Mart
ng
> @@ -1188,26 +1160,74 @@ static int s5p_mfc_configure_dma_memory(struct
> s5p_mfc_dev *mfc_dev)
> DMA_BIT_MASK(32));
> vb2_dma_contig_set_max_seg_size(mfc_dev->mem_dev[BANK2_CTX],
> DMA_BIT_MASK(32
arek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
> patches.
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
?
Also, ret is not declared so this patch will cause a compile error, breaking
git bisect-ability.
> + return ret;
> +
> + mfc_dev->dma_base[BANK1_CTX] = mfc_dev->fw_buf.dma;
Same comment than before, probably you wanted to use fw_buf->dma here?
The rest of the patch looks good to me.
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
Hello Marek,
On 02/14/2017 04:52 AM, Marek Szyprowski wrote:
> Use s5p_mfc_priv_buf structure for keeping the firmware image. This will
> help handling of firmware buffer allocation in the next patches.
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
m>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Tested-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
d-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Tested-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
Hello Marek,
On 02/14/2017 04:51 AM, Marek Szyprowski wrote:
> Remove unused structures, definitions and functions that are no longer
> called from the driver code.
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas &l
Hello Marek,
On 02/14/2017 04:51 AM, Marek Szyprowski wrote:
> Replace custom code with generic helper to retrieve driver data.
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Tested-by:
fy
> code in the next patches.
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Tested-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
ll
> help to simplify code in the next patches.
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
> ---
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Tested-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Jav
> driver will access device or context queue in error handling path, which
> might not be initialized yet, what causes kernel panic. Fix this by moving
> initialization of all static members as early as possible.
>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com&
Hello Sylwester,
On 02/13/2017 11:34 AM, Sylwester Nawrocki wrote:
> Hi Javier,
>
> On 02/13/2017 01:53 PM, Javier Martinez Canillas wrote:
>> Any comments on this series?
>
> The patches look good to me, I will Ack the patches in case
> Mauro wants to apply them di
Hello,
On Wed, Feb 1, 2017 at 5:05 PM, Javier Martinez Canillas
<jav...@osg.samsung.com> wrote:
> Hello,
>
> Commit 652bb68018a5 ("[media] exynos-gsc: do proper bytesperline and
> sizeimage calculation") fixed corrupted frames for most exynos-gsc
> formats, but e
Hello Shuah,
On 02/01/2017 11:25 PM, Shuah Khan wrote:
> On 01/24/2017 02:42 PM, Javier Martinez Canillas wrote:
>> There isn't an ioctl to enum the supported field orders, so a user-space
>> application can call VIDIOC_TRY_FMT using different field orders to know
>>
From: Thibault Saunier <thibault.saun...@osg.samsung.com>
Those are useful formats that should be handled.
Signed-off-by: Thibault Saunier <thibault.saun...@osg.samsung.com>
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
drivers/media/platform/exynos-g
mat didn't work when used as output (only as input).
This patch series fixes (1).
Best regards,
Javier
Thibault Saunier (2):
[media] exynos-gsc: Do not swap cb/cr for semi planar formats
[media] exynos-gsc: Add support for NV{16,21,61}M pixel formats
drivers/media/platform/exynos-gsc/gsc-c
las.dufre...@collabora.com>
Signed-off-by: Thibault Saunier <thibault.saun...@osg.samsung.com>
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
drivers/media/platform/exynos-gsc/gsc-core.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/media/platfo
] Not supported field order(4)
Instead of printing this as an error, just keep it as debug information.
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
drivers/media/platform/exynos-gsc/gsc-core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/d
sw_dt_ids) here or
otherwise module autoloading won't work.
Best regards,
Javier
--
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
CLK_IS_CRITICAL. So please also ignore my clk patch.
Sorry for the noise and the mess with these patches. Thought I also tested
on linux-next to see if the issue was solved there, but it seems I didn't.
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To un
Hello,
On 01/20/2017 07:06 AM, Javier Martinez Canillas wrote:
> Hello Marek,
>
> On 01/20/2017 05:37 AM, Marek Szyprowski wrote:
>
> [snip]
>>
>> Please send this patch instead of adding more clocks to the power domains.
>> This way we will avoid adding more
Hello,
On 01/19/2017 07:36 PM, Javier Martinez Canillas wrote:
> Commit 15f90ab57acc ("[media] exynos-gsc: Make driver functional when
> CONFIG_PM is unset") removed the implicit dependency that the driver
> had with CONFIG_PM, since it relied on the config option to be ena
t want to mark more clocks as CLK_IGNORE_UNUSED
or CLK_IS_CRITICAL but I agree this can be done as a temporary workaround
until a proper runtime PM based solution gets merged.
> Best regards
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscrib
Hello Marek,
On 01/20/2017 05:08 AM, Marek Szyprowski wrote:
> Hi Javier,
>
[snip]
>> Ok, I misunderstood the relationship between runtime PM and the power domains
>> then. I thought the power domains were only powered on when the runtime PM
>> framework resumed a
gement at ->remove()")
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Acked-by: Marek Szyprowski <m.szyprow...@samsung.com>
---
Changes in v2:
- Added Marek Szyprowski's Acked-by tag.
drivers/media/platform/exynos-gsc/gsc-core.c | 1 +
1 file changed, 1 in
he runtime resume callback had to be executed on the probe function.
But there's no need to do this if CONFIG_PM is enabled, since in this
case the runtime PM resume callback will be called by VIDIOC_STREAMON
ioctl, so the resume handler will call the GSC HW reset function.
Signed-off-by: Javie
Hello Marek,
On 01/19/2017 11:56 AM, Javier Martinez Canillas wrote:
> On 01/19/2017 11:17 AM, Marek Szyprowski wrote:
[snip]
>
> Also when removing the exynos_gsc driver, I get the same error:
>
> # rmmod s5p_mfc
> [ 106.405972] s5p-mfc 1100.codec: Removing 11
Hello Marek,
Thanks a lot for your feedback.
On 01/19/2017 11:17 AM, Marek Szyprowski wrote:
> Hi Javier,
>
> On 2017-01-18 01:30, Javier Martinez Canillas wrote:
>> Commit 15f90ab57acc ("[media] exynos-gsc: Make driver functional when
>> CONFIG_PM is unset")
Hello Marek,
On 01/19/2017 11:12 AM, Marek Szyprowski wrote:
> Hi Javier,
>
> On 2017-01-18 01:30, Javier Martinez Canillas wrote:
>> Commit a006c04e6218 ("[media] exynos-gsc: Fixup clock management at
>> ->remove()") changed the driver's
the reset in probe unneeded.
Fixes: 15f90ab57acc ("[media] exynos-gsc: Make driver functional when CONFIG_PM
is unset")
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
I-ve only tested with CONFIG_PM enabled since my Exynos5422 Odroid
XU4 board fails to b
gement at ->remove()")
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
drivers/media/platform/exynos-gsc/gsc-core.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/platform/exynos-gsc/gsc-core.c
b/drivers/media/platform/exynos-gsc/gsc-core.c
index c
are gone.
I understand that's what Sakari's RFC patches do. I'll try to make some
time tomorrow to test and review his patches.
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this list: send the line "unsubscribe linux-media"
nly anymore. But all the code
related to the DT parsing logic and input entities creation are removed.
Suggested-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
Hello Laurent,
I've tested this patch on
rking for both
composite input AIP1A (after changing the hardcoded selected input) and AIP1B.
The patches also look good to me, so please feel free to add my Reviewed-by and
Tested-by tags on these.
I wasn't able to test S-Video since my S-Video source broke (an old DVD player)
but this never wor
pdata-quirks anyways. So I agree with you that the input / output
signals lookup from DT could be done as a follow-up.
[0]: https://lkml.org/lkml/2016/4/12/983
[1]: https://lkml.org/lkml/2016/4/27/678
[2]: https://lkml.org/lkml/2016/11/11/346
Best regards,
Javier
--
To unsubscribe from this li
t; different. We are not interested in matching the pointer, what we
> want to match is that the path is the one we are expecting. Change to
> use of_node_cmp() so that we continue matching after the overlay has
> been removed and reapplied.
>
> Cc: Mauro Carvalho Chehab &l
any of the
patches in the media/v4.9-2 tag to be merged in v4.9-rc7.
>> Oops, sorry, I didn't check it properly. Would be nice to see that
>> patch also in linux-next.
>
> I'll likely add all patches at linux-next today, after handling more
> patches.
>
> Thanks,
>
port {
cam_ep: endpoint {
...
};
};
};
}
}
> Cc: Mauro Carvalho Chehab <mche...@kernel.org>
> Cc: Javier Martinez Canillas <jav...@osg.samsu
[adding Kevin as cc since he was interested in this feature as well]
Hello Hans,
On 11/11/2016 01:11 PM, Hans Verkuil wrote:
> This old mail came up in a discussion today so let me comment on this:
>
Thanks a lot for the feedback.
> On 04/27/2016 09:12 PM, Javier Martinez Canil
On Fri, Nov 11, 2016 at 12:50 PM, Javier Martinez Canillas
<jav...@dowhile0.org> wrote:
[snip]
> So if you can comment on this and see if the DT bindings fits your,
> would be very useful.
>
Sorry, I wanted to write "if the DT bindings fits your needs, it would
be very us
w.mail-archive.com/linux-media@vger.kernel.org/msg96393.html
[1]: http://www.spinics.net/lists/linux-media/msg99421.html
[2]: http://www.spinics.net/lists/linux-media/msg99987.html
Best regards,
Javier
--
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
com
./drivers/pinctrl/qcom
./drivers/soc/qcom
./drivers/clk/qcom
$ find -name *qualcomm
./drivers/net/ethernet/qualcomm
> Regards,
>
> Hans
>
Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message
ds on HAS_DMA
> select VIDEOBUF2_DMA_CONTIG
> select V4L2_MEM2MEM_DEV
>
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Tested-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung
(Exynos 5433)
I would also add 5800 to the list.
Besides these minor comments, the patch looks good to me:
Reviewed-by: Javier Martinez Canillas <jav...@osg.samsung.com>
And I've also tested in an Exynos5800 Peach Pi Chromebook:
Tested-by: Javier Martinez Canillas <jav...@osg.samsu
ext branch:
git://linuxtv.org/snawrocki/samsung.git for-v4.10/media/next
So feel free to add for the whole series:
Tested-by: Javier Martinez Canillas <jav...@osg.samsung.com>
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this li
Hello Benoit,
On Wed, Sep 28, 2016 at 5:22 PM, Benoit Parrot <bpar...@ti.com> wrote:
> ti_vpe module currently does not get loaded automatically.
> Added MODULE_DEVICE_TABLE hint to the driver to assist.
>
> Signed-off-by: Benoit Parrot <bpar...@ti.com>
> ---
R
drivers/staging/media/s5p-cec/s5p-cec.ko | grep alias
$
After this patch:
$ modinfo drivers/staging/media/s5p-cec/s5p-cec.ko | grep alias
alias: of:N*T*Csamsung,s5p-cecC*
alias: of:N*T*Csamsung,s5p-cec
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
d
drivers/staging/media//st-cec/stih-cec.ko | grep alias
$
After this patch:
$ modinfo drivers/staging/media//st-cec/stih-cec.ko | grep alias
alias: of:N*T*Cst,stih-cecC*
alias: of:N*T*Cst,stih-cec
Signed-off-by: Javier Martinez Canillas <jav...@osg.samsung.com>
---
drivers/s
1 - 100 of 896 matches
Mail list logo