From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 22D36D68BCF for ; Fri, 15 Nov 2024 19:56:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=e2eICgLxoCF/r9xjNCZUQm6PeFz/qxiXCGHK+JS28BM=; b=wwl2eseI01mDYE u35/FFtNwx4Eve1OjKp0mkGJ9fvPHWnygDE5Fw84/OeXeq0f5086X5Ri8aP69OOeYFm4V60eznNmC JrkqIGJHSqTeObeIVl4esrrhf8k24YSCVKHli57x3/kmCLd615+dlm/bb7SF0EixGYFqUznov4qsM 6GFKKWaXSYAASuoH5yCcF3TVNhUEwVIG+S9CwEuzj6XQyYh/SAkMGT3nrAaAp/23JBAxEKFMaWvww 3+7jrFy//Q6x9XJNL6XdURs2kMrDUJIoV7lgq7fWUPDTwR+L6sNg8JdXxKX4oOmcZoj9U8fI2Nz7v +jEGfn463WUQiUCvPW3Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tC2Qk-00000003wAL-34Wl; Fri, 15 Nov 2024 19:56:34 +0000 Received: from relay1-d.mail.gandi.net ([217.70.183.193]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tC2Qi-00000003w9X-0jSb for linux-mtd@lists.infradead.org; Fri, 15 Nov 2024 19:56:33 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id 01596240005; Fri, 15 Nov 2024 19:56:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1731700589; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=7mNYmj6ni23Jqko1xtDRAMtam5EhBsgqdEbqnjiENf0=; b=XDbWp8i8JF47NcJG/C7twNY8eXASi6ybsjpo1jkIg6zozBQyheFJ+4vmRR5rGbrnNqbtDt cq6NoEdhqw0Ty1N6aSuPSwbb59OqxoFhQUNZNF1HsCKcaHMIC3B3SV0ijuXYxeIjExHJBy mSQP+RyJz9AIxLKqOMLeUYfT5qNVx414L3hSCBaA0zj6MV5dfpSKz8z52mCjWhdgT9iB+v 0Xi1Uafy925rS23lfO/vn+mPAKKu6Qq03Lwc1/EIsMSstyeTtNtBkhQwnFV8PC6k0KIMeW pU208kbhVfSJT50YH+FUfwQT/FVsmwD1WF60YM+oC1l9a/z96LGoFVw5RlLOWA== From: Miquel Raynal To: Bastien Curutchet Cc: Santosh Shilimkar , Krzysztof Kozlowski , Richard Weinberger , Vignesh Raghavendra , linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org, Thomas Petazzoni , Herve Codina , Christopher Cordahi Subject: Re: [PATCH v4 03/10] memory: ti-aemif: Wrap CS timings into a struct In-Reply-To: <20241115132631.264609-4-bastien.curutchet@bootlin.com> (Bastien Curutchet's message of "Fri, 15 Nov 2024 14:26:24 +0100") References: <20241115132631.264609-1-bastien.curutchet@bootlin.com> <20241115132631.264609-4-bastien.curutchet@bootlin.com> User-Agent: mu4e 1.12.1; emacs 29.4 Date: Fri, 15 Nov 2024 20:56:27 +0100 Message-ID: <87jzd4mftg.fsf@bootlin.com> MIME-Version: 1.0 X-GND-Sasl: miquel.raynal@bootlin.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241115_115632_342701_98201CD9 X-CRM114-Status: UNSURE ( 6.43 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org On 15/11/2024 at 14:26:24 +01, Bastien Curutchet wrote: > CS timings are store in the struct aemif_cs_data along with other CS > parameters. It isn't convenient for exposing CS timings to other drivers > without also exposing the other parameters. > > Wrap the CS timings in a new the struct aemif_cs_timings to simplify s/the// ^^^ > their export in upcoming patches. > > Signed-off-by: Bastien Curutchet But otherwise this feels very sensible. Reviewed-by: Miquel Raynal ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/