Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <fcf78a88-ab27-3a6a-db06-66504fc18e10@collabora.com>
Date:   Tue, 25 Oct 2022 13:36:03 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.3.1
Subject: Re: [PATCH v1] media: cedrus: Propagate error code from
 cedrus_h265_skip_bits()
Content-Language: en-US
To:     Hans Verkuil <hverkuil@xs4all.nl>,
        =?UTF-8?Q?Jernej_=c5=a0krabec?= <jernej.skrabec@gmail.com>,
        Maxime Ripard <mripard@kernel.org>,
        Paul Kocialkowski <paul.kocialkowski@bootlin.com>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Chen-Yu Tsai <wens@csie.org>,
        Samuel Holland <samuel@sholland.org>,
        Nicolas Dufresne <nicolas.dufresne@collabora.com>
Cc:     linux-media@vger.kernel.org, linux-staging@lists.linux.dev,
        linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org,
        kernel@collabora.com
References: <20220914150105.209484-1-dmitry.osipenko@collabora.com>
 <d75c0597-2323-27f2-a7e2-b319667bdcf6@xs4all.nl>
 <12078224.O9o76ZdvQC@jernej-laptop>
 <3a26e105-9c50-8cdd-b4be-ba97b3306261@collabora.com>
 <1103d833-6c29-6ccb-05dd-a2883eaa2281@xs4all.nl>
From:   Dmitry Osipenko <dmitry.osipenko@collabora.com>
In-Reply-To: <1103d833-6c29-6ccb-05dd-a2883eaa2281@xs4all.nl>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 10/25/22 13:23, Hans Verkuil wrote:
> On 10/25/22 11:34, Dmitry Osipenko wrote:
>> On 10/25/22 07:59, Jernej Škrabec wrote:
>>> Dne ponedeljek, 24. oktober 2022 ob 13:38:36 CEST je Hans Verkuil napisal(a):
>>>> Hi Dmitry,
>>>>
>>>> This patch has a conflict with this patch from Jernej:
>>>>
>>>> https://patchwork.linuxtv.org/project/linux-media/patch/20221017194413.11983
>>>> 01-1-jernej.skrabec@gmail.com/
>>>>
>>>> I decided to take Jernej's patch first. Can you make a v2 that sits on top
>>>> of that patch?
>>> I believe you already merged first version of the patch for 6.1. No need for 
>>> this version though, first version already solves main issue.
>>
>> That was a followup to the first version.
>>
> 
> I have no idea what you are both talking about :-)
> 
> "I believe you already merged first version of the patch for 6.1" Which patch
> are you referring to?

This is about [1] that was already merged. This patch implements the
error propagation that Jernej asked [2] when he reviewed [1].

[1]
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=91db7a3fc7fe670cf1770a398a43bb4a1f776bf1

[2] https://lore.kernel.org/lkml/1733932.VLH7GnMWUR@kista/

I'll prepare the v2 if this change is still desired by Jernej.

-- 
Best regards,
Dmitry