From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from fgw23-7.mail.saunalahti.fi (fgw23-7.mail.saunalahti.fi [62.142.5.84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9D4C023770 for ; Sun, 4 Feb 2024 17:00:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=62.142.5.84 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707066057; cv=none; b=VZ9b2JJTs5VTKKlARs+w6PVbkQ/ImUewYIKnesAiymI9ak0i60R1oVmMB0UM2A+b5KU3h59l7pGSHD2HBXsfXsvYqfJqI+QELm+lKTkF9rhVrEwzSs/QBwXNSh4O+DOOTFdz8fgns9AjvsuU8VQKRUHUlio+Y2zk7/wTPWWZYsk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707066057; c=relaxed/simple; bh=L4N+w+Xl9amelgcnkkk1AsPQ5omDQKGC5rGF9ag5Cj8=; h=From:Date:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=TdeOF37f7fzSR9LNp6XyU1bJEQIhdIm17fZgyT1KWolEKsc4Kq2UF1tXtV7oOJgxV4spWd2Gx+Ptclw4WG+6sMMMqYzBb4vYu03bHcabJYxqar3jjxS/9dNGm8iPEVtZeea8ICH3HEROtWXwiHNClAdcY3mXMR4RVxLUV+TIWlI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com; spf=fail smtp.mailfrom=gmail.com; arc=none smtp.client-ip=62.142.5.84 Authentication-Results: smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=gmail.com Received: from localhost (88-113-26-30.elisa-laajakaista.fi [88.113.26.30]) by fgw23.mail.saunalahti.fi (Halon) with ESMTP id f4473253-c37e-11ee-b972-005056bdfda7; Sun, 04 Feb 2024 19:00:53 +0200 (EET) From: andy.shevchenko@gmail.com Date: Sun, 4 Feb 2024 19:00:52 +0200 To: Nikita Shubin Cc: Sergey Shtylyov , Damien Le Moal , Niklas Cassel , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, Arnd Bergmann Subject: Re: [PATCH v7 22/39] ata: pata_ep93xx: add device tree support Message-ID: References: <20240118-ep93xx-v7-0-d953846ae771@maquefel.me> <20240118-ep93xx-v7-22-d953846ae771@maquefel.me> Precedence: bulk X-Mailing-List: linux-ide@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240118-ep93xx-v7-22-d953846ae771@maquefel.me> Thu, Jan 18, 2024 at 11:21:05AM +0300, Nikita Shubin kirjoitti: > - add OF ID match table > - drop platform DMA and filters > - change DMA setup to OF, so we can defer probe ... > + drv_data->dma_rx_channel = dma_request_chan(dev, "rx"); > + if (IS_ERR(drv_data->dma_rx_channel)) { > + ret = PTR_ERR(drv_data->dma_rx_channel); > + return dev_err_probe(dev, ret, "rx DMA setup failed\n"); It can be done in one statement: return dev_err_probe(dev, PTR_ERR(drv_data->dma_rx_channel), "rx DMA setup failed\n"); > + } ... > + drv_data->dma_tx_channel = dma_request_chan(&pdev->dev, "tx"); > + if (IS_ERR(drv_data->dma_tx_channel)) { > + ret = PTR_ERR(drv_data->dma_tx_channel); > + dev_err_probe(dev, ret, "tx DMA setup failed\n"); In the similar way ret = dev_err_probe(...); > + goto fail_release_rx; > } -- With Best Regards, Andy Shevchenko