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;
Date:   Wed, 31 May 2023 13:52:50 +0300
From:   Serge Semin <fancer.lancer@gmail.com>
To:     Mark Brown <broonie@kernel.org>
Cc:     Osama Muhammad <osmtendev@gmail.com>, linux-spi@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1] spi-dw-core.c: Fix error checking for
 debugfs_create_dir
Message-ID: <20230531105250.6pd5jpngwbnbnbbc@mobilestation>
References: <20230520224025.14928-1-osmtendev@gmail.com>
 <168546845148.691057.9965315836245052581.b4-ty@kernel.org>
 <20230530211446.foqpcfha6hjruhow@mobilestation>
 <30bcf77a-3e7c-4f13-94ab-f4efc52193dc@sirena.org.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <30bcf77a-3e7c-4f13-94ab-f4efc52193dc@sirena.org.uk>
Precedence: bulk

On Tue, May 30, 2023 at 10:16:00PM +0100, Mark Brown wrote:
> On Wed, May 31, 2023 at 12:14:46AM +0300, Serge Semin wrote:
> 
> > Oh, thanks. I've absolutely missed the respinned version of the patch
> > (it should have been marked as v2 though). Anyway the change looks
> > good except it introduces a redundant empty line at the tail of the
> > dw_spi_debugfs_init() function. Is it possible to rebase the branch
> > and drop the line it? If it's not I'll send an incremental cleanup
> > patch then.
> 
> I'd rather just take the incremental patch here.

Done. Please find the cleanup patch here:
https://lore.kernel.org/linux-spi/20230530221725.26319-1-Sergey.Semin@baikalelectronics.ru

-Serge(y)