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 76E89EB64DA for ; Fri, 7 Jul 2023 07:52:20 +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-Type: Content-Transfer-Encoding:Reply-To:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:References:Cc:To:Subject: From:MIME-Version:Date:Message-ID:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=d2p3aLXSI0qX86uCO1KzXBOxWdL0FZET0OtkFh2a1As=; b=ASig92ZvpGEmo69M9sRiZ3LGXZ p60ySvIYtFo6TEznGqZhrVAeYwgVsFpq9ypNwPP4J9IXpyeJqve0fUQWRdooAv/mV1A4iREmsLYij 6I1apFPfYz4E4qTQNmrRvxXCggOyvuA+IoXrs6zGi2hCd+EvkTfCrSdZQqQLrsR/z8n/CpqW5Dj5N 0fM+zXnon/KUBDjlzjU7tONlKcwO4YPbbuhWZ7+WhwLinhCDxRdAsv9mwgKgyPNS9aia8hNdRGxrt cttZxXp2OlQiV659bYJ3l9rzjFwqHj4hU9phaeWLjquA4WkHm4NAXOdTB6cLfhLqyXaI4YGPYDdlf PFgJnnUw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qHgG0-003uqq-21; Fri, 07 Jul 2023 07:52:00 +0000 Received: from mail-wr1-x435.google.com ([2a00:1450:4864:20::435]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qHgFx-003uoh-0P for linux-amlogic@lists.infradead.org; Fri, 07 Jul 2023 07:51:58 +0000 Received: by mail-wr1-x435.google.com with SMTP id ffacd0b85a97d-3143b72c5ffso1786750f8f.3 for ; Fri, 07 Jul 2023 00:51:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1688716311; x=1691308311; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to; bh=UC5F0FA71mhMEgXpkNWdObl4xUHsYLbWZcNLdSixP60=; b=wZxj4FwJnEuUXQUynRvoqIwtLtG3qgMtWZ6/8COXXEN8Qhl3ctiOhEBl1ncuaOu8/G la2v/U/9a0RqxDjSIv5VQD9RN+VzuHF1HICjL2ZPWdPDkmIcPN9KAYpMYNP7S2eIgA1I 4735860OkAZEY51MYcEfjICwmCON0rBlBs+itbxcWn+kc+kfoYD891WVYnlQmFTkZ26w JwkD9/HjxhTkKqHoTMBG7UcTXF1iYMALia43F1ui75i7EcUXko1kho5xriFTKk8ym8Jd qgaGQvcHVmB+0ZgbG+ACVF5aPoYnhqZrUV9vGo7tXc6ViIqcQKvGBNeGlTEkgR7J8gTd I+tg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1688716311; x=1691308311; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=UC5F0FA71mhMEgXpkNWdObl4xUHsYLbWZcNLdSixP60=; b=GKMg9f8hDjr6VxQ576e4oJXFx+ibbWXZB0iQiHzNKIk94nVbZ3okUz27sNvtNcYTsa +1tJIhV30V161C+R+6AD5+T7clkKMnNL4+Rtaosz7YGO7qU/D1db6PiLep22Fymk6QtN t/OdV75UFEHSje57RKA0QeClTzTHJeJ6ZvlT28n0GH7A0BdjE6bpAJE0BFSCBDsS9JBo dpKzmkw5I9FwIr92LBJKJQg20X3KhiV60eYOXsWYR5tLWQMqDFhxqu4QoTqOOpIofUg1 hjqSJxeSaQaeduMY8M5ubCJ7SpAbliYTHMkQmEcOy0uGYO0+/rWuLqUip4Tgt3v4xUuy hwaA== X-Gm-Message-State: ABy/qLZzB9oYr/s0nqdqE2pVaXQuOYwvuMklDVBYLKt9UbwXJLXPyvFL dq6kx0uIkWCRiQwiAfFuJxKipA== X-Google-Smtp-Source: APBJJlFs77aJEcKl47rs6zmBGWRedkrw2mAlemkoqQjXu7RZz1IOFErpMhKmP0OOEG2NyGaZi6PVeQ== X-Received: by 2002:a5d:6045:0:b0:314:52b4:cc38 with SMTP id j5-20020a5d6045000000b0031452b4cc38mr5275076wrt.52.1688716310914; Fri, 07 Jul 2023 00:51:50 -0700 (PDT) Received: from [192.168.27.65] (home.beaume.starnux.net. [82.66.176.246]) by smtp.gmail.com with ESMTPSA id f18-20020adff992000000b003143c532431sm3775318wrr.27.2023.07.07.00.51.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 07 Jul 2023 00:51:50 -0700 (PDT) Message-ID: <1f962997-82f9-18e3-f95d-e1b723e636f8@linaro.org> Date: Fri, 7 Jul 2023 09:51:49 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 From: Neil Armstrong Subject: Re: [PATCH v2 1/2] spi: amlogic-spifc-a1: implement adjust_op_size() Content-Language: en-US To: Martin Kurbanov , Mark Brown , Andy Shevchenko Cc: linux-spi@vger.kernel.org, linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org, kernel@sberdevices.ru References: <20230706110331.19794-1-mmkurbanov@sberdevices.ru> <20230706110331.19794-2-mmkurbanov@sberdevices.ru> Organization: Linaro Developer Services In-Reply-To: <20230706110331.19794-2-mmkurbanov@sberdevices.ru> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230707_005157_184251_363C89B2 X-CRM114-Status: GOOD ( 20.93 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: neil.armstrong@linaro.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org On 06/07/2023 13:03, Martin Kurbanov wrote: > This enhancement eliminates the need for a loop in the > amlogic_spifc_a1_exec_op() function and allows the SPI core to > dynamically divide transactions into appropriately sized chunks. > > Signed-off-by: Martin Kurbanov > --- > drivers/spi/spi-amlogic-spifc-a1.c | 66 +++++++++++------------------- > 1 file changed, 25 insertions(+), 41 deletions(-) > > diff --git a/drivers/spi/spi-amlogic-spifc-a1.c b/drivers/spi/spi-amlogic-spifc-a1.c > index 3c4224c38399..a92e4fc23396 100644 > --- a/drivers/spi/spi-amlogic-spifc-a1.c > +++ b/drivers/spi/spi-amlogic-spifc-a1.c > @@ -72,7 +72,7 @@ > > #define SPIFC_A1_USER_DBUF_ADDR_REG 0x248 > > -#define SPIFC_A1_BUFFER_SIZE 512 > +#define SPIFC_A1_BUFFER_SIZE 512U > > #define SPIFC_A1_MAX_HZ 200000000 > #define SPIFC_A1_MIN_HZ 1000000 > @@ -240,61 +240,44 @@ static int amlogic_spifc_a1_exec_op(struct spi_mem *mem, > { > struct amlogic_spifc_a1 *spifc = > spi_controller_get_devdata(mem->spi->controller); > - size_t off, nbytes = op->data.nbytes; > - u32 cmd_cfg, addr_cfg, dummy_cfg, dmode; > + size_t data_size = op->data.nbytes; > int ret; > > amlogic_spifc_a1_user_init(spifc); > + amlogic_spifc_a1_set_cmd(spifc, SPIFC_A1_USER_CMD(op)); > > - cmd_cfg = SPIFC_A1_USER_CMD(op); > - amlogic_spifc_a1_set_cmd(spifc, cmd_cfg); > + if (op->addr.nbytes) > + amlogic_spifc_a1_set_addr(spifc, op->addr.val, > + SPIFC_A1_USER_ADDR(op)); > > - if (op->addr.nbytes) { > - addr_cfg = SPIFC_A1_USER_ADDR(op); > - amlogic_spifc_a1_set_addr(spifc, op->addr.val, addr_cfg); > - } > - > - if (op->dummy.nbytes) { > - dummy_cfg = SPIFC_A1_USER_DUMMY(op); > - amlogic_spifc_a1_set_dummy(spifc, dummy_cfg); > - } > - > - if (!op->data.nbytes) > - return amlogic_spifc_a1_request(spifc, false); > - > - dmode = ilog2(op->data.buswidth); > - off = 0; > - > - do { > - size_t block_size = min_t(size_t, nbytes, SPIFC_A1_BUFFER_SIZE); > - > - amlogic_spifc_a1_set_cmd(spifc, cmd_cfg); > + if (op->dummy.nbytes) > + amlogic_spifc_a1_set_dummy(spifc, SPIFC_A1_USER_DUMMY(op)); > > - if (op->addr.nbytes) > - amlogic_spifc_a1_set_addr(spifc, op->addr.val + off, > - addr_cfg); > - > - if (op->dummy.nbytes) > - amlogic_spifc_a1_set_dummy(spifc, dummy_cfg); > + if (data_size) { > + u32 mode = ilog2(op->data.buswidth); > > writel(0, spifc->base + SPIFC_A1_USER_DBUF_ADDR_REG); > > if (op->data.dir == SPI_MEM_DATA_IN) > - ret = amlogic_spifc_a1_read(spifc, > - op->data.buf.in + off, > - block_size, dmode); > + ret = amlogic_spifc_a1_read(spifc, op->data.buf.in, > + data_size, mode); > else > - ret = amlogic_spifc_a1_write(spifc, > - op->data.buf.out + off, > - block_size, dmode); > - > - nbytes -= block_size; > - off += block_size; > - } while (nbytes != 0 && !ret); > + ret = amlogic_spifc_a1_write(spifc, op->data.buf.out, > + data_size, mode); > + } else { > + ret = amlogic_spifc_a1_request(spifc, false); > + } > > return ret; > } > > +static int amlogic_spifc_a1_adjust_op_size(struct spi_mem *mem, > + struct spi_mem_op *op) > +{ > + op->data.nbytes = min(op->data.nbytes, SPIFC_A1_BUFFER_SIZE); > + return 0; > +} > + > static void amlogic_spifc_a1_hw_init(struct amlogic_spifc_a1 *spifc) > { > u32 regv; > @@ -314,6 +297,7 @@ static void amlogic_spifc_a1_hw_init(struct amlogic_spifc_a1 *spifc) > > static const struct spi_controller_mem_ops amlogic_spifc_a1_mem_ops = { > .exec_op = amlogic_spifc_a1_exec_op, > + .adjust_op_size = amlogic_spifc_a1_adjust_op_size, > }; > > static int amlogic_spifc_a1_probe(struct platform_device *pdev) Reviewed-by: Neil Armstrong _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic