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: <506d3ae3-f06c-1243-877b-d1378bf435c6@i2se.com>
Date:   Thu, 22 Dec 2022 12:12:21 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Subject: Re: [PATCH v2 0/7] staging: vc04_services: Remove custom return
 values
Content-Language: en-US
To:     Umang Jain <umang.jain@ideasonboard.com>,
        linux-staging@lists.linux.dev,
        linux-rpi-kernel@lists.infradead.org,
        linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org,
        linux-kernel@vger.kernel.org
Cc:     Kees Cook <keescook@chromium.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Adrien Thierry <athierry@redhat.com>,
        Dan Carpenter <error27@gmail.com>,
        Dave Stevenson <dave.stevenson@raspberrypi.com>,
        Kieran Bingham <kieran.bingham@ideasonboard.com>,
        Laurent Pinchart <laurent.pinchart@ideasonboard.com>
References: <20221219115725.11992-1-umang.jain@ideasonboard.com>
From:   Stefan Wahren <stefan.wahren@i2se.com>
In-Reply-To: <20221219115725.11992-1-umang.jain@ideasonboard.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
UI-OutboundReport: notjunk:1;M01:P0:yhPsEOh5kHU=;s9Cz3HF4r3OSITv04JbS2Gsbgd9
 vUe/JjBud2Y9iqXus/8qJOT4oZzGoSjHWFrBdszLdqkOiPuR+lGrabAQZvTc0f23Np1VNP4wF
 +Ct82RjTDL93iyOYk6RfIF8jrtoI0hAPJv1Cmw7lONj1rd3asNtfCzxTNjgsvoMXzBZ0ZR+Ki
 i+1Uy2aB474C6RWnRVnhiNTUDruGL8jSf51jLhfUyvMZUGjXUYOjDRhSw8nIxcVCth+gdkLK5
 zhAcrbX5T8ZVr9JXqq+mFc14/BRKOac4waZpWbUsUL/2zIdRcqYkzibU9V3y8NP5aDGM7FH7R
 JY9+Nf339NZaGT/DYq7xb4K694Tnu3LvXWJJRfemQ5ur1XvLhgncks9l678OHxxIVjn7wYf77
 Epx8t1+YjLAZIsbX/aTob2l3/OfDTODxL0slqS9fWoNLInv7baIXGk7PLQYr2pjnW657y+11u
 fmeG+hs8BrtU3Tgf/LP6Aind1rwaOKLhCGxGL+AxR9DBWLhufrG50JlNiG89blHTVsBRwGDcc
 gvSjYG6dPpx2VlOfmzy+q3GmNtHtnf2i7Rz/1OauMydWhO5TVeO4gpr4gPqdCIj1VoahJkxoh
 sjyRROX87l/GCn3DwqXz+HjXNlzukl/KGGD/f0UNwPUXoCY/DESl4sSdVvbA1tITFhdprB+P0
 TrjP72byrtGEVpMS+LYuoC82Il4oe8jRA6JmZLZwow==
Precedence: bulk

Am 19.12.22 um 12:57 schrieb Umang Jain:
> The series removes the custom return values from functions
> and replaces them with linux error codes. This address the TODO
>   vchiq interface:
> * Get rid of custom function return values
>
> Changes in V2:
> - Patch 3/7 now reports for specific errors like -ENOMEM, -EHOSTDOWN
> - Patch 5/7 reports -ENOTCONN instead of -EINVAL and adds a "Fixes" tag
>
> Umang Jain (7):
>    staging: vc04_services: Replace vchiq_status return type to int
>    staging: vc04_services: Drop VCHIQ_SUCCESS usage
>    staging: vc04_services: Drop VCHIQ_ERROR usage
>    staging: vc04_services: Drop VCHIQ_RETRY usage
>    vc04_services: vchiq_arm: Drop VCHIQ_RETRY usage on disconnect
>    staging: vc04_services: Drop enum vchiq_status remnants
>    staging: vc04_services: vchiq: Drop custom return values from TODO

The whole series is:

Tested-by: Stefan Wahren <stefan.wahren@i2se.com>