, first enable the parent hwmod, and after setup and reset is done,
restore the parent hwmod to postsetup_state.
This method is better. It's less intrusive and only takes place at setup time.
Reviewed-by: Archit Taneja archit.tan...@gmail.com
--
To unsubscribe from this list: send the line
tries to fix the issue at setup
time by making sure the DSS core hwmod is enabled when a DSS submodule hwmod
is
being setup.
Quickly tested on OMAP4 Panda and OMAP5 uEVM.
The series looks good to me.
Reviewed-by: Archit Taneja archit.tan...@gmail.com
Archit
--
To unsubscribe from this list
Hi,
On Tuesday 24 June 2014 03:34 PM, Tomi Valkeinen wrote:
Make the omapdrm driver use the new HDMI ops when possible.
omapdrm will call set_hdmi_mode (when available) to tell the encoder
driver whether the monitor is a DVI or HDMI monitor, and if it's an HDMI
monitor, omapdrm will call
, and therefore, removed.
- Some of the struct names are shortened, and members are named more
appropriately.
Archit Taneja (2):
arm: omap2+: hwmod: Add refcounting for modulemode shared by multiple
hwmods
arm: omap5 hwmod data: Example: Make DSS hwmods share MODULEMODE
fields
arch/arm
their
mainclks as dss_dss_clk, this needs to be changed later).
Signed-off-by: Archit Taneja arc...@ti.com
---
arch/arm/mach-omap2/omap_hwmod_54xx_data.c | 40 --
1 file changed, 27 insertions(+), 13 deletions(-)
diff --git a/arch/arm/mach-omap2/omap_hwmod_54xx_data.c
to the struct with a
spinlock.
Signed-off-by: Archit Taneja arc...@ti.com
---
arch/arm/mach-omap2/omap_hwmod.c | 145 ++-
arch/arm/mach-omap2/omap_hwmod.h | 35 +++---
2 files changed, 153 insertions(+), 27 deletions(-)
diff --git a/arch/arm/mach-omap2
/uninits it's ports. These new functions retrieve
the dpi_data pointer from 'port-data', and not from the platform device's
data(pdev-dev) like in the non-DT path.
We add some code in dss_uninit_ports() to pass a pointer to the DPI port in
dpi_uninit_port().
Signed-off-by: Archit Taneja arc...@ti.com
a static instance defined for dpi_data, which is accessed by top
level DPI ops. This will be removed when the driver dynamically allocates
dpi_data for each DPI instance.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dpi.c | 187 +---
1
be used to store dpi_data.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dpi.c | 46 ++---
drivers/video/fbdev/omap2/dss/dss.c | 6 ++---
drivers/video/fbdev/omap2/dss/dss.h | 2 +-
3 files changed, 36 insertions(+), 18 deletions
to 1 in the tfp410 and tpd12s015 encoder drivers.
Signed-off-by: Archit Taneja arc...@ti.com
---
.../fbdev/omap2/displays-new/encoder-tfp410.c | 1 +
.../fbdev/omap2/displays-new/encoder-tpd12s015.c | 1 +
drivers/video/fbdev/omap2/dss/dss-of.c | 58
path, since we don't support multiple
instances in the non-DT case.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dpi.c | 26 +++---
1 file changed, 23 insertions(+), 3 deletions(-)
diff --git a/drivers/video/fbdev/omap2/dss/dpi.c
b/drivers/video
, such that
omapdss_of_find_source_for_first_ep is simplified.
Archit Taneja (7):
OMAPDSS: DPI: Use DPI driver data
OMAPDSS: DPI: Allocate driver data
OMAPDSS: DPI: Store dpi_data pointer in the DT port's data
OMAPDSS: DSS: init dss ports cleanly
OMAPDSS: DT: Get source endpoint by matching reg
since we will have multiple DPI
ports to uninit in the future.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dss.c | 81 ++---
drivers/video/fbdev/omap2/dss/dss.h | 26 +++-
drivers/video/fbdev/omap2/dss/sdi.c | 2 +-
3 files
Add an argument which describes which DPI instance we are referring to when
selecting it's overlay manager.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dpi.c | 2 +-
drivers/video/fbdev/omap2/dss/dss.c | 12 ++--
drivers/video/fbdev/omap2/dss/dss.h | 2
table.
Signed-off-by: Archit Taneja arc...@ti.com
---
arch/arm/mach-omap2/control.c | 22 +-
arch/arm/mach-omap2/control.h | 2 +-
arch/arm/mach-omap2/io.c | 2 +-
3 files changed, 23 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-omap2/control.c b/arch/arm/mach
in CTRL_CORE_CONTROL_IO_2.
Add dss_deshdcp gate-clock node under dra7-ctrl-core. There are separate bit
fields for enabling and disabling the clock. We just map the enable bit since
this clock doesn't cause any impact if left enabled.
Signed-off-by: Archit Taneja arc...@ti.com
---
arch/arm/boot/dts/dra7xx
or the system to suspend, and only
a very few flops get this clock all the time. So there is negligible impact.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/clk/ti/clk-7xx.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/ti/clk-7xx.c b/drivers/clk
aren't mapped.
Signed-off-by: Archit Taneja arc...@ti.com
---
arch/arm/boot/dts/dra7.dtsi | 13 +
1 file changed, 13 insertions(+)
diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index 149b550..14d1905 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm
From: Tero Kristo t-kri...@ti.com
Retry init is needed if clockdomains are registered before the corresponding
clocks are ready. In this case, the clockdomain info is added to a list
which will be processed once the clockdomains for next PRCM module are
processed.
Signed-off-by: Tero Kristo
code.
I picked up 2 patches from Tero's WIP branch below, and added DRA7's control
module core block as a clock provider. The DSS_DESHDCP clock is easy to
implement after that.
https://github.com/t-kristo/linux-pm/tree/3.14-rc4-cm-prm-driver-wip
Archit Taneja (5):
ARM: PRCM: split PRCM module
-by: Archit Taneja arc...@ti.com
---
arch/arm/mach-omap2/cm_common.c | 18 ++
arch/arm/mach-omap2/control.c | 15 +++
arch/arm/mach-omap2/control.h | 1 +
arch/arm/mach-omap2/io.c | 4 +++
arch/arm/mach-omap2/prcm-common.h | 5
arch/arm/mach-omap2
On Tuesday 27 May 2014 02:34 PM, Tomi Valkeinen wrote:
On 26/05/14 12:28, Archit Taneja wrote:
SoCs containing DSS until now had only one DPI instance. DRA7x has 3 DPI
instances.
In order to support multiple instances, we allocate a driver data
struct(dpi_data) for each instance
On Tuesday 27 May 2014 02:04 PM, Tomi Valkeinen wrote:
On 26/05/14 12:28, Archit Taneja wrote:
In omapdss_of_find_source_for_first_ep, we retrieve a source endpoint's DT node,
and then see what omapdss output has the matching device_node pointer in
omap_dss_find_output_by_node.
For all DPI
Hi,
On Tuesday 27 May 2014 03:50 PM, Rajendra Nayak wrote:
On Monday 26 May 2014 04:14 PM, Archit Taneja wrote:
Generally, IP blocks/modules within a clock domain each have their own
CM_x_CLKCTRL register, each having it's own MODULEMODE field to manage the
module.
snip
@@ -2751,6 +2820,13
On Tuesday 27 May 2014 03:54 PM, Tomi Valkeinen wrote:
On 27/05/14 12:49, Archit Taneja wrote:
On Tuesday 27 May 2014 02:04 PM, Tomi Valkeinen wrote:
Hmm, maybe the above function, and the helper functions, should be
changed a bit more, so that this function would do:
ep
On Tuesday 27 May 2014 01:54 PM, Tomi Valkeinen wrote:
On 26/05/14 12:28, Archit Taneja wrote:
The init/uninit port functions are used to set up the DPI and SDI outputs under
the dss platform device. A 'reg' property is used to determine whether the node
is DPI or SDI for OMAP34xx DSS revision
Add an argument which describes which DPI instance we are referring to when
selecting it's overlay manager.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dpi.c | 2 +-
drivers/video/fbdev/omap2/dss/dss.c | 12 ++--
drivers/video/fbdev/omap2/dss/dss.h | 2
instance's dpi_data.
dpi_init_output/dpi_uninit_output are untouched and only used for non DT case,
dpi_init_output_port/dpi_uninit_output_port are used in the DT case, where DSS
configures the ports using dpi_init_port/dpi_uninit_port.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video
since we will have multiple DPI
ports to uninit in the future.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dpi.c | 2 +-
drivers/video/fbdev/omap2/dss/dss.c | 84 ++---
drivers/video/fbdev/omap2/dss/dss.h | 27 +++-
drivers
to 1 in the tfp410 and tpd12s015 encoder drivers.
Signed-off-by: Archit Taneja arc...@ti.com
---
.../fbdev/omap2/displays-new/encoder-tfp410.c | 1 +
.../fbdev/omap2/displays-new/encoder-tpd12s015.c | 1 +
drivers/video/fbdev/omap2/dss/dss-of.c | 51
To support multiple DPI instances, dpi_get_channel needs to take the DPI
instance's reg-id to get the corresponding channel. Make it take this
argument.
We just pass 0 in the non-DT path, since we don't support multiple instances
in the non-DT case.
Signed-off-by: Archit Taneja arc...@ti.com
, and not break things
when the DSS driver runs.
(Note: hdmi and rfbi hwmods still don't reset properly as they don't have their
mainclks as dss_dss_clk, this will be fixed later).
Signed-off-by: Archit Taneja arc...@ti.com
---
arch/arm/mach-omap2/omap_hwmod_54xx_data.c | 46 +-
1
to this instance.
omap_hwmod's soc enable_module and disable_module ops set the MODULEMODE
reigster bits only when the first module using it is enabled, or the last module
using it is disabled. We serialize accesses to the struct with a spinlock.
Signed-off-by: Archit Taneja arc...@ti.com
---
arch
On Tuesday 20 May 2014 01:34 PM, Tomi Valkeinen wrote:
On 08/05/14 12:15, Archit Taneja wrote:
The init/uninit port functions are used to set up the DPI and SDI outputs under
the dss platform device. A 'reg' property is used to determine whether the node
is DPI or SDI for OMAP34xx DSS revision
Hi,
On Friday 16 May 2014 01:44 PM, Tomi Valkeinen wrote:
On 12/05/14 18:48, Tony Lindgren wrote:
Also, I'm wondering why we still have .clk and .opt_clks entries in the
hwmod data for am43xx and omap5 which are both device tree based with
all the clocks coming from .dts files?
I think they
Hi,
On Friday 09 May 2014 06:02 PM, Tomi Valkeinen wrote:
Add binding documentation for HDMI connector's HPD GPIO.
Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
Cc: devicet...@vger.kernel.org
---
Documentation/devicetree/bindings/video/hdmi-connector.txt | 1 +
1 file changed, 1
Hi Paul,
On Thursday 08 May 2014 09:31 PM, Paul Walmsley wrote:
snip
The problem is that we have multiple hwmods which use the same MODULEMODE bit.
There is no use-counting done when it comes to enabling/disabling modulemode.
If there was such a thing, we could have provided MODULEMODE flags
On Tuesday 06 May 2014 07:56 PM, Tony Lindgren wrote:
* Archit Taneja arc...@ti.com [140505 22:24]:
Hi,
On Monday 21 April 2014 08:40 PM, Tony Lindgren wrote:
* Archit Taneja arc...@ti.com [140420 22:16]:
Hi,
On Friday 18 April 2014 10:48 PM, Tony Lindgren wrote:
* Archit Taneja arc
to add:
Tested-by: Darren Etheridge detheri...@ti.com
DMM doesn't get used by default since it's hwmod is missing in DRA7x's
hwmod data. I added the hwmod and tried using dmm with omapdrm, it works
fine.
So, for DSS and DMM:
Tested-by: Archit Taneja arc...@ti.com
arch/arm/boot/dts/dra7
On Thursday 08 May 2014 01:23 PM, Tero Kristo wrote:
On 05/08/2014 09:02 AM, Archit Taneja wrote:
On Tuesday 06 May 2014 07:56 PM, Tony Lindgren wrote:
* Archit Taneja arc...@ti.com [140505 22:24]:
Hi,
On Monday 21 April 2014 08:40 PM, Tony Lindgren wrote:
* Archit Taneja arc...@ti.com
The support for simpler port/endpoint binding was removed in the merged version
of omapdss DT. But dss_init_ports still tries to get to an endpoint even if no
port exists. Remove this as this doesn't work.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dss.c | 6
/sdi_uninit_port functions have to be removed from the exit
section as dss_features(which is initconst data) uses it, this prevents the
section mismatch.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dpi.c | 2 +-
drivers/video/fbdev/omap2/dss/dss.c | 88
-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dss-of.c | 41 ++
drivers/video/fbdev/omap2/dss/dss.c| 6 +
drivers/video/fbdev/omap2/dss/dss.h| 2 ++
drivers/video/fbdev/omap2/dss/output.c | 15 +
include/video
To support multiple DPI instances, dpi_get_channel needs to take the DPI
instance's reg-id to get the corresponding channel. Make it take this
argument.
We just pass 0 in the non-DT path, since we don't support multiple instances
in the non-DT case.
Signed-off-by: Archit Taneja arc...@ti.com
instance's dpi_data.
When an encoder/panel driver calls a dpi op, we get dpi_data using the function
dpi_get_data_from_dssdev. This function iterates through the ports under DSS
device node, and returns the port which has reg-id matching the reg-id specified
in omap_dss_device.
Signed-off-by: Archit
Add an argument which describes which DPI instance we are referring to when
selecting it's overlay manager. This will come into use when a DSS version
supports multiple DPI instances.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/video/fbdev/omap2/dss/dpi.c | 2 +-
drivers/video/fbdev
Hi Paul,
On Thursday 08 May 2014 10:07 AM, Paul Walmsley wrote:
Hi,
On Wed, 12 Mar 2014, Tomi Valkeinen wrote:
This patch adds hwmod data for omap5 display subsystem. I have tested this on
omap5-uevm with a DSI panel. I cannot test omap5-uevm's hdmi output yet, as the
mainline is missing
Hi,
On Monday 21 April 2014 08:40 PM, Tony Lindgren wrote:
* Archit Taneja arc...@ti.com [140420 22:16]:
Hi,
On Friday 18 April 2014 10:48 PM, Tony Lindgren wrote:
* Archit Taneja arc...@ti.com [140416 06:20]:
Add DT node for the ctrl-core sub module of the DRA7 control module. We map
this.
Reviewed-by: Archit Taneja arc...@ti.com?
Archit
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi,
On Thursday 24 April 2014 03:47 PM, Tomi Valkeinen wrote:
Add support to set OMAP5 DSI pin muxing.
Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
Cc: Tony Lindgren t...@atomide.com
---
arch/arm/mach-omap2/display.c | 35 ++-
1 file changed, 34
On Friday 25 April 2014 04:48 PM, Tomi Valkeinen wrote:
On 25/04/14 14:11, Archit Taneja wrote:
Hi,
On Thursday 24 April 2014 03:47 PM, Tomi Valkeinen wrote:
Add support to set OMAP5 DSI pin muxing.
Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
Cc: Tony Lindgren t...@atomide.com
Hi,
On Friday 18 April 2014 10:48 PM, Tony Lindgren wrote:
* Archit Taneja arc...@ti.com [140416 06:20]:
Add DT node for the ctrl-core sub module of the DRA7 control module. We map the
CTRL_MODULE_CORE address region up to 0x4a002d60, this region contains register
fields which configure clocks
in CTRL_CORE_CONTROL_IO_2.
Add dss_deshdcp gate-clock node under dra7-ctrl-core. There are separate bit
fields for enabling and disabling the clock. We just map the enable bit since
this clock doesn't cause any impact if left enabled.
Signed-off-by: Archit Taneja arc...@ti.com
---
arch/arm/boot/dts/dra7xx
or the system to suspend, and only
a very few flops get this clock all the time. So there is negligible impact.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/clk/ti/clk-7xx.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/ti/clk-7xx.c b/drivers/clk
aren't mapped.
Signed-off-by: Archit Taneja arc...@ti.com
---
arch/arm/boot/dts/dra7.dtsi | 13 +
1 file changed, 13 insertions(+)
diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index 1c0f8e1..58bbdf3 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm
the CORE sub module has clock related register fields. We ignore the WKUP sub
module.
Signed-off-by: Archit Taneja arc...@ti.com
---
arch/arm/mach-omap2/prm_common.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-omap2/prm_common.c b/arch/arm/mach-omap2/prm_common.c
index b4c4ab9
Hi Kamil,
On Thursday 13 March 2014 08:14 PM, Kamil Debski wrote:
Hi,
From: Archit Taneja [mailto:arc...@ti.com]
Sent: Thursday, March 13, 2014 12:44 PM
Rename the memory block resource vpe_csc to csc since it also
exists within the VIP IP block. This would make the name more generic
Hi,
On Thursday 13 March 2014 07:59 PM, Kamil Debski wrote:
Hi,
From: Archit Taneja [mailto:arc...@ti.com]
Sent: Thursday, March 13, 2014 1:09 PM
Hi Kamil,
On Thursday 13 March 2014 05:18 PM, Kamil Debski wrote:
Hi Archit,
From: Archit Taneja [mailto:arc...@ti.com]
Sent: Tuesday, March
of older cropping API.
- Typo fix.
- Some changes in patch 6/7 to support composing on the capture side of VPE.
Archit Taneja (14):
v4l: ti-vpe: Make sure in job_ready that we have the needed number of
dst_bufs
v4l: ti-vpe: register video device only when firmware is loaded
v4l: ti-vpe: Use
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
b/drivers/media/platform/ti-vpe/vpe.c
index 362d5be..972f43f 100644
--- a/drivers/media/platform/ti-vpe
-by: Hans Verkuil hans.verk...@cisco.com
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
b/drivers/media/platform/ti-vpe/vpe.c
index d7befb9..c0ae847 100644
--- a/drivers
device corresponding to it.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpdma.c | 8 +++--
drivers/media/platform/ti-vpe/vpdma.h | 7 +++--
drivers/media/platform/ti-vpe/vpe.c | 55 ---
3 files changed, 41 insertions(+), 29
The bus_info parameter in v4l2_capabilities expects a 'platform_' prefix. This
wasn't done in the driver and hence was breaking compliance. Update the bus_info
parameter accordingly.
Reviewed-by: Hans Verkuil hans.verk...@cisco.com
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media
to the vpdma_add_in_dtd() helper.
- Some minor changes are made to the vpdma_add_out_dtd() helper. The c_rect
param is used for specifying the 'composition' target, and 'width' is added
to calculate the line stride.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpdma.c
currently is the helper
video_device_release. This causes memory corruption, and leads to issues when
we try to re-insert the vpe module.
Use the video_device_release_empty helper function instead.
Reviewed-by: Hans Verkuil hans.verk...@cisco.com
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers
querycap currently returns V4L2_CAP_VIDEO_M2M as a capability, this should be
V4L2_CAP_VIDEO_M2M_MPLANE instead, as the driver supports multiplanar formats.
Reviewed-by: Hans Verkuil hans.verk...@cisco.com
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 2
or V4L2_FIELD_BOTTOM, the vb2 buf_prepare op returns
an error.
This ensures compliance, and that the dequeued output and captured buffers
contain the field type that the driver used internally.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 13 -
1 file
(G_FMT) != G_FMT'.
Fix the initial queue configuration such that it wouldn't need to be fixed by
try_fmt.
Reviewed-by: Hans Verkuil hans.verk...@cisco.com
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 6 --
1 file changed, 4 insertions(+), 2 deletions
Rename the memory block resource vpe_csc to csc since it also exists within
the VIP IP block. This would make the name more generic, and both VPE and VIP DT
nodes in the future can use it.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/csc.c | 2 +-
1 file changed
to the io_modes of the vb2 output and capture queues. This
allows the driver to import dma shared buffers.
Reviewed-by: Hans Verkuil hans.verk...@cisco.com
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
or
destination dimensions change, set_srcdst_params() is called for this purpose.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 150
1 file changed, 150 insertions(+)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
b/drivers
to
hit the condition where we don't have any free buffers for the VPE to write on.
Add the extra check in job_ready() to make sure we have the sufficient amount
of destination buffers.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 3 +++
1 file changed, 3
than 32x32 didn't make much sense, so stopped
at this.
Reviewed-by: Hans Verkuil hans.verk...@cisco.com
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
b
Hi Kamil,
On Thursday 13 March 2014 05:18 PM, Kamil Debski wrote:
Hi Archit,
From: Archit Taneja [mailto:arc...@ti.com]
Sent: Tuesday, March 11, 2014 9:34 AM
vpe fops(vpe_open in particular) should be called only when VPDMA
firmware is loaded. File operations on the video device are possible
On Tuesday 11 March 2014 12:45 PM, Tomi Valkeinen wrote:
Hi,
On 15/10/13 10:04, Archit Taneja wrote:
Add Dynamic Memory Manager (DMM) bindings for OMAP4 and OMAP5 devices. DMM
only requires address and irq information.
Add documentation for the DMM bindings.
Originally worked on by Andy
compatibility.
Changes in v2:
- selection API used instead of older cropping API.
- Typo fix.
- Some changes in patch 6/7 to support composing on the capture side of VPE.
Archit Taneja (14):
v4l: ti-vpe: Make sure in job_ready that we have the needed number of
dst_bufs
v4l: ti-vpe: register
to
hit the condition where we don't have any free buffers for the VPE to write on.
Add the extra check in job_ready() to make sure we have the sufficient amount
of destination buffers.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 3 +++
1 file changed, 3
The bus_info parameter in v4l2_capabilities expects a 'platform_' prefix. This
wasn't done in the driver and hence was breaking compliance. Update the bus_info
parameter accordingly.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 3 ++-
1 file changed, 2
than 32x32 didn't make much sense, so stopped
at this.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
b/drivers/media/platform/ti-vpe/vpe.c
index 0e7573a
device corresponding to it.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpdma.c | 8 +++--
drivers/media/platform/ti-vpe/vpdma.h | 7 +++--
drivers/media/platform/ti-vpe/vpe.c | 55 ---
3 files changed, 41 insertions(+), 29
-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
b/drivers/media/platform/ti-vpe/vpe.c
index 85d1122..970408a 100644
--- a/drivers/media/platform/ti-vpe/vpe.c
+++ b/drivers/media
or V4L2_FIELD_BOTTOM, the vb2 buf_prepare op returns
an error.
This ensures compliance, and that the dequeued output and captured buffers
contain the field type that the driver used internally.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 13 -
1 file
The dequed CAPTURE_MPLANE type buffers don't contain the flags that the
originally queued OUTPUT_MPLANE type buffers have. This breaks compliance.
Copy the source v4l2_buffer flags to the destination v4l2_buffer flags before
they are dequed.
Signed-off-by: Archit Taneja arc...@ti.com
(G_FMT) != G_FMT'.
Fix the initial queue configuration such that it wouldn't need to be fixed by
try_fmt.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
querycap currently returns V4L2_CAP_VIDEO_M2M as a capability, this should be
V4L2_CAP_VIDEO_M2M_MPLANE instead, as the driver supports multiplanar formats.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
or
destination dimensions change, set_srcdst_params() is called for this purpose.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 141
1 file changed, 141 insertions(+)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
b/drivers
Rename the memory block resource vpe_csc to csc since it also exists within
the VIP IP block. This would make the name more generic, and both VPE and VIP DT
nodes in the future can use it.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/csc.c | 2 +-
1 file changed
currently is the helper
video_device_release. This causes memory corruption, and leads to issues when
we try to re-insert the vpe module.
Use the video_device_release_empty helper function instead
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 2 +-
1 file changed
to the vpdma_add_in_dtd() helper.
- Some minor changes are made to the vpdma_add_out_dtd() helper. The c_rect
param is used for specifying the 'composition' target, and 'width' is added
to calculate the line stride.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpdma.c
to the io_modes of the vb2 output and capture queues. This
allows the driver to import dma shared buffers.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
b/drivers
On Tuesday 11 March 2014 05:51 PM, Hans Verkuil wrote:
Hi Archit,
A few small comments below...
On 03/11/14 09:33, Archit Taneja wrote:
Add selection ioctl ops. For VPE, cropping makes sense only for the input to
VPE(or V4L2_BUF_TYPE_VIDEO_OUTPUT/MPLANE buffers) and composing makes sense
only
On Tuesday 11 March 2014 06:19 PM, Hans Verkuil wrote:
On 03/11/14 13:46, Archit Taneja wrote:
On Tuesday 11 March 2014 05:51 PM, Hans Verkuil wrote:
Hi Archit,
A few small comments below...
On 03/11/14 09:33, Archit Taneja wrote:
snip
Yes. If for no other reason that I plan on adding
Hi Hans,
On Friday 07 March 2014 07:17 PM, Archit Taneja wrote:
On Friday 07 March 2014 07:02 PM, Hans Verkuil wrote:
On 03/07/2014 02:22 PM, Archit Taneja wrote:
Disregard what I said, it's OK to upstream it. But if you could just
spend
some hours fixing the problems, that would really
Hi Hans,
On Tuesday 04 March 2014 05:05 PM, Hans Verkuil wrote:
On 03/04/14 12:25, Archit Taneja wrote:
I had a minor question about the selection API:
Are the V4L2_SET_TGT_CROP/COMPOSE_DEFAULT and the corresponding
'BOUNDS' targets supposed to be used with VIDIOC_S_SELECTION? If so,
what's
Hi,
On Friday 07 March 2014 06:29 PM, Hans Verkuil wrote:
Do you think I can go ahead with posting the v3 patch set for 3.15, and
work on fixing the compliance issue for the -rc fixes?
It's fine to upstream this in staging, but while not all compliance errors
are fixed it can't go to
On Friday 07 March 2014 07:02 PM, Hans Verkuil wrote:
On 03/07/2014 02:22 PM, Archit Taneja wrote:
Hi,
On Friday 07 March 2014 06:29 PM, Hans Verkuil wrote:
Do you think I can go ahead with posting the v3 patch set for 3.15, and
work on fixing the compliance issue for the -rc fixes?
It's
Hi,
On Tuesday 04 March 2014 01:13 PM, Hans Verkuil wrote:
On 03/04/2014 08:38 AM, Archit Taneja wrote:
Hi Hans,
On Monday 03 March 2014 01:20 PM, Hans Verkuil wrote:
Hi Archit!
On 03/03/2014 08:33 AM, Archit Taneja wrote:
Add crop ioctl ops. For VPE, cropping only makes sense
.
Reference branch:
g...@github.com:boddob/linux.git vpe_for_315
Changes in v2:
- selection API used instead of older cropping API.
- Typo fix.
- Some changes in patch 6/7 to support composing on the capture side of VPE.
Archit Taneja (7):
v4l: ti-vpe: Make sure in job_ready that we have
device corresponding to it.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpdma.c | 8 +++--
drivers/media/platform/ti-vpe/vpdma.h | 7 +++--
drivers/media/platform/ti-vpe/vpe.c | 55 ---
3 files changed, 41 insertions(+), 29
than 32x32 didn't make much sense, so stopped
at this.
Signed-off-by: Archit Taneja arc...@ti.com
---
drivers/media/platform/ti-vpe/vpe.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/ti-vpe/vpe.c
b/drivers/media/platform/ti-vpe/vpe.c
index 915029b
1 - 100 of 1155 matches
Mail list logo