Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Message-ID: <c7a7798b65e38355230d9ee235a3f8aa331123ca.camel@collabora.com>
Subject: Re: [PATCH v3] media: dvb: Provide 4 devm variants
From:   Ezequiel Garcia <ezequiel@collabora.com>
To:     Marc Gonzalez <marc.w.gonzalez@free.fr>,
        Sean Young <sean@mess.org>, Brad Love <brad@nextdimension.cc>,
        Mauro Carvalho Chehab <mchehab@kernel.org>,
        Jonathan =?ISO-8859-1?Q?Neusch=E4fer?= <j.neuschaefer@gmx.net>
Cc:     linux-media <linux-media@vger.kernel.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Bjorn Andersson <bjorn.andersson@linaro.org>,
        Guenter Roeck <linux@roeck-us.net>
Date:   Sun, 21 Jul 2019 12:02:44 -0300
In-Reply-To: <068836c7-a116-7e6f-cbb2-f2176fbb9a38@free.fr>
References: <068836c7-a116-7e6f-cbb2-f2176fbb9a38@free.fr>
Organization: Collabora
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.30.5-1.1 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi Marc,

Thanks for your patch.

On Thu, 2019-07-18 at 15:03 +0200, Marc Gonzalez wrote:
> Provide devm variants for automatic resource release on device removal.
> Makes error-handling in probe() simpler, thus less error-prone.
> Once all resources are devmanaged, remove() is no longer needed.
> 

I think it would be better to add this API as part of a series
that also uses it.

Regards,
Eze