When generating the partition windows, operate directly on the partition
struct rather than copying and duplicating the processed data
Signed-off-by: Kieran Bingham
---
drivers/media/platform/vsp1/vsp1_video.c | 43 -
1 file
To improve image quality when scaling using the UDS we need to correctly
determine the start phase value for each partition window.
Provide helper functions for calculating the phase, and write this value
to the registers when used.
Signed-off-by: Kieran Bingham
Increase the overlap on the left edge to allow a margin to provide
better image scaling
Signed-off-by: Kieran Bingham
---
drivers/media/platform/vsp1/vsp1_rpf.c | 7 +-
drivers/media/platform/vsp1/vsp1_uds.c | 39 ---
2 files
The configuration of the pipeline, and entities directly affects the
inputs required to each entity for the partition algorithm. Thus it
makes sense to involve those entities in the decision making process.
Extend the entity ops API to provide an optional '.partition' call. This
allows entities
Hi Geert,
The following changes since commit 0c3b6ad6a559391f367879fd4be6d2d85625bd5a:
v4l: vsp1: Remove redundant context variables (Tue, 1 Nov 2016 15:01:02
+)
are available in the git repository at:
https://git.kernel.org/pub/scm/linux/kernel/git/kbingham/rcar.git
Hi Laurent,
Thanks for the review!
On 10/02/17 07:58, Laurent Pinchart wrote:
> Hi Kieran,
>
> Thank you for the patch.
>
> On Wednesday 08 Feb 2017 14:03:57 Kieran Bingham wrote:
>> From: Kieran Bingham
>>
>> Parameters can contain characters not suited to
Hi Kieran,
Thank you for the patch.
On Wednesday 08 Feb 2017 14:03:59 Kieran Bingham wrote:
> From: Kieran Bingham
>
> Pass the optional '--crop (X,Y)/WxH' parameter through reference_frame
> allowing the input to be cropped for comparison
>
> Signed-off-by:
Hi Kieran,
Thank you for the patch.
On Wednesday 08 Feb 2017 14:03:58 Kieran Bingham wrote:
> From: Kieran Bingham
>
> Allow the user to specify an input crop in the form (X,Y)/WxH
>
> Signed-off-by: Kieran Bingham
> ---
>
Hi Kieran,
Thank you for the patch.
On Wednesday 08 Feb 2017 14:04:00 Kieran Bingham wrote:
> From: Kieran Bingham
>
> Test both the input cropping size and position
>
> Signed-off-by: Kieran Bingham
> ---
>
Hi Kieran,
Thank you for the patch.
On Wednesday 08 Feb 2017 14:03:56 Kieran Bingham wrote:
> From: Kieran Bingham
>
> In the event of failed frames, or VSP_KEEP_FRAMES being set, the output
> file names do not sort such that the reference frame is next to the
Hi Laurent,
Thanks for the review,
On 10/02/17 08:19, Laurent Pinchart wrote:
> Hi Kieran,
>
> Thank you for the patch.
>
> On Wednesday 08 Feb 2017 14:03:58 Kieran Bingham wrote:
>> From: Kieran Bingham
>>
>> Allow the user to specify an input crop in the
From: Kieran Bingham
Pass the optional '--crop (X,Y)/WxH' parameter through reference_frame
allowing the input to be cropped for comparison
Signed-off-by: Kieran Bingham
---
scripts/vsp-lib.sh | 52
From: Kieran Bingham
Test both the input cropping size and position
Signed-off-by: Kieran Bingham
Reviewed-by: Laurent Pinchart
---
tests/vsp-unit-test-0021.sh | 36
On Wed, Jan 25, 2017 at 5:18 PM, Sergei Shtylyov
wrote:
> On 01/24/2017 09:21 PM, Simon Horman wrote:
>
>> From: Kazuya Mizuguchi
>>
>> "swiotlb buffer is full" errors occur after repeated initialisation of a
>> device - f.e.
From: Kieran Bingham
Parameters can contain characters not suited to use in filenames.
Add '=','(', and ')' to the filtering, and replace with '_'
This reduces extra escaping, and quoting when working with the output
filenames of failed tests and
From: Kieran Bingham
Update the gen-image utility such that it can crop the image at input,
and provide tests to ensure that the output of the RPF is as we expect.
v2:
- rebased on master branch
- review comments fixed
- vsp-unit-test-0021.sh description
From: Kieran Bingham
Allow the user to specify an input crop in the form (X,Y)/WxH
Signed-off-by: Kieran Bingham
---
src/gen-image.c | 132 +-
1 file changed, 132 insertions(+)
On Thursday, February 9, 2017 8:21:43 PM CET Chris Brandt wrote:
> On Thursday, February 09, 2017, Florian Fainelli worte:
> > > I think the closest I might have come was to purposely break the build
> > > if more then 1 was select, but that still didn't stop you from making
> > > the selection.
>
On Friday, February 10, 2017, Arnd Bergmann wrote:
> I also couldn't come up with something working when I looked at this, and
> it wouldn't solve the related problem of platforms that we want to be able
> to use with or without MMU: You can't make the decision of whether allow
> an MMU based on
Hi Geert,
On Friday, February 10, 2017, Geert Uytterhoeven wrote:
> > static const char *const r7s72100_boards_compat_dt[] __initconst = {
> > "renesas,r7s72100",
> > NULL,
> > @@ -29,4 +58,5 @@ DT_MACHINE_START(R7S72100_DT, "Generic R7S72100
> (Flattened Device Tree)")
> >
> > #1 and #3 look like they are the same thing (except #3 is enabled on power
> > on reset). The renesas_wdt.c uses the register names from #1.
> > Is the idea that you only use #3 to make sure your systems boots and get
> > into
> > Linux, then from there you use #1 and stop #3 (hence no
Hi Chris,
On Fri, Feb 10, 2017 at 3:59 PM, Chris Brandt wrote:
> On Friday, February 10, 2017, Geert Uytterhoeven wrote:
>> > static const char *const r7s72100_boards_compat_dt[] __initconst = {
>> > "renesas,r7s72100",
>> > NULL,
>> > @@ -29,4 +58,5 @@
Ooops, I left this on my screen, and have already sent the V2 anyway.
Hitting send for posterity.
--
Kieran
On 10/02/17 09:20, Laurent Pinchart wrote:
> Hi Kieran,
>
> Thank you for the patch.
>
> On Wednesday 08 Feb 2017 14:03:59 Kieran Bingham wrote:
>> From: Kieran Bingham
23 matches
Mail list logo