From patchwork Tue Dec 12 14:52:55 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gilad Ben-Yossef X-Patchwork-Id: 121530 Delivered-To: patch@linaro.org Received: by 10.140.22.227 with SMTP id 90csp4222699qgn; Tue, 12 Dec 2017 06:54:47 -0800 (PST) X-Google-Smtp-Source: ACJfBov2XOVBIxJAApY7VjGjiCLRNTkI6+Ah1/yiMlTh+dE8Bgejz6p6jfA01VM8YkbyW/JhkY/f X-Received: by 10.159.235.132 with SMTP id f4mr2540069plr.122.1513090487506; Tue, 12 Dec 2017 06:54:47 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1513090487; cv=none; d=google.com; s=arc-20160816; b=GHFlPhamA8MFLRcpApeayn9KO+qkXcn9YYanToPwnbUH7dVk8gKtmhjj+2IzyUXXPS UJK2Iw1a6Z+KXKrgeAUVHNjh7qCt1Ki1x+4Jj7GKCy0mvLXIezeaRLgxVILO4OF26+AV sX9kI426QX6QPLA2chtZkwh2B0eBtPB4EmFQlUUdZ4AZa87/S0wlbxV5i3xfuUIl4w5h ElGzTF8kURF1P0KEvE++htkFJhQm7FAdWeImzy1sQjuE3XMZtLeT/KSeLn55olXqVzbA Vy83AbhUvLVlI9Zq3SFjDhb/F4t9LTSBXMTT1VpEk9TxB3+onNsxXvz3z2vvGlLOzV+U /Fsg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:arc-authentication-results; bh=nKn08GIXZJTsrciU74ssdPpkQ7PiU+gTeLIcon7LC/Y=; b=qe6TDze3FohOzoA2HgrxckKVv0By7sSfQcy418OYzHugRG0gc1fWBvTn9JAUIpxAa2 PQn2xp9MCqcGP0PP+yRFRi7VH1kEjsx3gNatpF928B2L01FLFt1jD5LdOAeSTILbDshx CH2Ar0cbrb/ceivBtFXRUMMp05zMbfi9Ng0PK3U5isdDgkEtUfLCowikm51ACoQ77GD0 Wigfr2/+LeaJucayMmNZo3MjIkd+kVLs2pWWMdlukR5OehHEm4DS6JA4G0fL/7JBGnpu S/U9TRpVrCSL9R6RhiTR/o4qxFexEut/GlrGTpBpsu84Lo1+PuBgQulgVXLupBExsJ/b YVYg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-crypto-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-crypto-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v9si10144691plo.199.2017.12.12.06.54.47; Tue, 12 Dec 2017 06:54:47 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-crypto-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-crypto-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-crypto-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752405AbdLLOyo (ORCPT + 1 other); Tue, 12 Dec 2017 09:54:44 -0500 Received: from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]:45102 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752467AbdLLOyl (ORCPT ); Tue, 12 Dec 2017 09:54:41 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 560301529; Tue, 12 Dec 2017 06:54:41 -0800 (PST) Received: from sugar.kfn.arm.com (unknown [10.45.48.196]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 677B73F577; Tue, 12 Dec 2017 06:54:39 -0800 (PST) From: Gilad Ben-Yossef To: Greg Kroah-Hartman Cc: Ofir Drang , linux-crypto@vger.kernel.org, driverdev-devel@linuxdriverproject.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCH 09/24] staging: ccree: fix reg mgr naming convention Date: Tue, 12 Dec 2017 14:52:55 +0000 Message-Id: <1513090395-7938-10-git-send-email-gilad@benyossef.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1513090395-7938-1-git-send-email-gilad@benyossef.com> References: <1513090395-7938-1-git-send-email-gilad@benyossef.com> Sender: linux-crypto-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org The request manager files were using a func naming convention which was inconsistent (ssi vs. cc), included a useless prefix (ssi_request_mgr) and often too long. Make the code more readable by switching to a simpler, consistent naming convention. Signed-off-by: Gilad Ben-Yossef --- drivers/staging/ccree/ssi_driver.c | 8 +++---- drivers/staging/ccree/ssi_request_mgr.c | 40 ++++++++++++++++----------------- drivers/staging/ccree/ssi_request_mgr.h | 4 ++-- 3 files changed, 25 insertions(+), 27 deletions(-) -- 2.7.4 diff --git a/drivers/staging/ccree/ssi_driver.c b/drivers/staging/ccree/ssi_driver.c index 513c5e4..491e2b9 100644 --- a/drivers/staging/ccree/ssi_driver.c +++ b/drivers/staging/ccree/ssi_driver.c @@ -326,9 +326,9 @@ static int init_cc_resources(struct platform_device *plat_dev) goto post_sram_mgr_err; } - rc = request_mgr_init(new_drvdata); + rc = cc_req_mgr_init(new_drvdata); if (rc) { - dev_err(dev, "request_mgr_init failed\n"); + dev_err(dev, "cc_req_mgr_init failed\n"); goto post_sram_mgr_err; } @@ -389,7 +389,7 @@ static int init_cc_resources(struct platform_device *plat_dev) post_buf_mgr_err: cc_buffer_mgr_fini(new_drvdata); post_req_mgr_err: - request_mgr_fini(new_drvdata); + cc_req_mgr_fini(new_drvdata); post_sram_mgr_err: ssi_sram_mgr_fini(new_drvdata); post_fips_init_err: @@ -422,7 +422,7 @@ static void cleanup_cc_resources(struct platform_device *plat_dev) ssi_ivgen_fini(drvdata); cc_pm_fini(drvdata); cc_buffer_mgr_fini(drvdata); - request_mgr_fini(drvdata); + cc_req_mgr_fini(drvdata); ssi_sram_mgr_fini(drvdata); ssi_fips_fini(drvdata); #ifdef ENABLE_CC_SYSFS diff --git a/drivers/staging/ccree/ssi_request_mgr.c b/drivers/staging/ccree/ssi_request_mgr.c index 5f34336..dbdfd0c 100644 --- a/drivers/staging/ccree/ssi_request_mgr.c +++ b/drivers/staging/ccree/ssi_request_mgr.c @@ -33,7 +33,7 @@ #define SSI_MAX_POLL_ITER 10 -struct ssi_request_mgr_handle { +struct cc_req_mgr_handle { /* Request manager resources */ unsigned int hw_queue_size; /* HW capability */ unsigned int min_free_hw_slots; @@ -68,9 +68,9 @@ static void comp_handler(unsigned long devarg); static void comp_work_handler(struct work_struct *work); #endif -void request_mgr_fini(struct ssi_drvdata *drvdata) +void cc_req_mgr_fini(struct ssi_drvdata *drvdata) { - struct ssi_request_mgr_handle *req_mgr_h = drvdata->request_mgr_handle; + struct cc_req_mgr_handle *req_mgr_h = drvdata->request_mgr_handle; struct device *dev = drvdata_to_dev(drvdata); if (!req_mgr_h) @@ -92,14 +92,14 @@ void request_mgr_fini(struct ssi_drvdata *drvdata) /* Kill tasklet */ tasklet_kill(&req_mgr_h->comptask); #endif - memset(req_mgr_h, 0, sizeof(struct ssi_request_mgr_handle)); + memset(req_mgr_h, 0, sizeof(struct cc_req_mgr_handle)); kfree(req_mgr_h); drvdata->request_mgr_handle = NULL; } -int request_mgr_init(struct ssi_drvdata *drvdata) +int cc_req_mgr_init(struct ssi_drvdata *drvdata) { - struct ssi_request_mgr_handle *req_mgr_h; + struct cc_req_mgr_handle *req_mgr_h; struct device *dev = drvdata_to_dev(drvdata); int rc = 0; @@ -161,7 +161,7 @@ int request_mgr_init(struct ssi_drvdata *drvdata) return 0; req_mgr_init_err: - request_mgr_fini(drvdata); + cc_req_mgr_fini(drvdata); return rc; } @@ -202,9 +202,9 @@ static void request_mgr_complete(struct device *dev, void *dx_compl_h) complete(this_compl); } -static int request_mgr_queues_status_check( +static int cc_queues_status( struct ssi_drvdata *drvdata, - struct ssi_request_mgr_handle *req_mgr_h, + struct cc_req_mgr_handle *req_mgr_h, unsigned int total_seq_len) { unsigned long poll_queue; @@ -264,7 +264,7 @@ int send_request( struct cc_hw_desc *desc, unsigned int len, bool is_dout) { void __iomem *cc_base = drvdata->cc_base; - struct ssi_request_mgr_handle *req_mgr_h = drvdata->request_mgr_handle; + struct cc_req_mgr_handle *req_mgr_h = drvdata->request_mgr_handle; unsigned int used_sw_slots; unsigned int iv_seq_len = 0; unsigned int total_seq_len = len; /*initial sequence length*/ @@ -291,8 +291,7 @@ int send_request( * in case iv gen add the max size and in case of no dout add 1 * for the internal completion descriptor */ - rc = request_mgr_queues_status_check(drvdata, req_mgr_h, - max_required_seq_len); + rc = cc_queues_status(drvdata, req_mgr_h, max_required_seq_len); if (rc == 0) /* There is enough place in the queue */ break; @@ -418,14 +417,13 @@ int send_request_init( struct ssi_drvdata *drvdata, struct cc_hw_desc *desc, unsigned int len) { void __iomem *cc_base = drvdata->cc_base; - struct ssi_request_mgr_handle *req_mgr_h = drvdata->request_mgr_handle; + struct cc_req_mgr_handle *req_mgr_h = drvdata->request_mgr_handle; unsigned int total_seq_len = len; /*initial sequence length*/ int rc = 0; /* Wait for space in HW and SW FIFO. Poll for as much as FIFO_TIMEOUT. */ - rc = request_mgr_queues_status_check(drvdata, req_mgr_h, - total_seq_len); + rc = cc_queues_status(drvdata, req_mgr_h, total_seq_len); if (rc) return rc; @@ -448,7 +446,7 @@ int send_request_init( void complete_request(struct ssi_drvdata *drvdata) { - struct ssi_request_mgr_handle *request_mgr_handle = + struct cc_req_mgr_handle *request_mgr_handle = drvdata->request_mgr_handle; complete(&drvdata->hw_queue_avail); @@ -474,7 +472,7 @@ static void proc_completions(struct ssi_drvdata *drvdata) { struct ssi_crypto_req *ssi_req; struct device *dev = drvdata_to_dev(drvdata); - struct ssi_request_mgr_handle *request_mgr_handle = + struct cc_req_mgr_handle *request_mgr_handle = drvdata->request_mgr_handle; unsigned int *tail = &request_mgr_handle->req_queue_tail; unsigned int *head = &request_mgr_handle->req_queue_head; @@ -540,7 +538,7 @@ static inline u32 cc_axi_comp_count(struct ssi_drvdata *drvdata) static void comp_handler(unsigned long devarg) { struct ssi_drvdata *drvdata = (struct ssi_drvdata *)devarg; - struct ssi_request_mgr_handle *request_mgr_handle = + struct cc_req_mgr_handle *request_mgr_handle = drvdata->request_mgr_handle; u32 irq; @@ -590,7 +588,7 @@ static void comp_handler(unsigned long devarg) #if defined(CONFIG_PM) int cc_resume_req_queue(struct ssi_drvdata *drvdata) { - struct ssi_request_mgr_handle *request_mgr_handle = + struct cc_req_mgr_handle *request_mgr_handle = drvdata->request_mgr_handle; spin_lock_bh(&request_mgr_handle->hw_lock); @@ -606,7 +604,7 @@ int cc_resume_req_queue(struct ssi_drvdata *drvdata) */ int cc_suspend_req_queue(struct ssi_drvdata *drvdata) { - struct ssi_request_mgr_handle *request_mgr_handle = + struct cc_req_mgr_handle *request_mgr_handle = drvdata->request_mgr_handle; /* lock the send_request */ @@ -624,7 +622,7 @@ int cc_suspend_req_queue(struct ssi_drvdata *drvdata) bool cc_req_queue_suspended(struct ssi_drvdata *drvdata) { - struct ssi_request_mgr_handle *request_mgr_handle = + struct cc_req_mgr_handle *request_mgr_handle = drvdata->request_mgr_handle; return request_mgr_handle->is_runtime_suspended; diff --git a/drivers/staging/ccree/ssi_request_mgr.h b/drivers/staging/ccree/ssi_request_mgr.h index 53eed5f..d018f51 100644 --- a/drivers/staging/ccree/ssi_request_mgr.h +++ b/drivers/staging/ccree/ssi_request_mgr.h @@ -23,7 +23,7 @@ #include "cc_hw_queue_defs.h" -int request_mgr_init(struct ssi_drvdata *drvdata); +int cc_req_mgr_init(struct ssi_drvdata *drvdata); /*! * Enqueue caller request to crypto hardware. @@ -47,7 +47,7 @@ int send_request_init( void complete_request(struct ssi_drvdata *drvdata); -void request_mgr_fini(struct ssi_drvdata *drvdata); +void cc_req_mgr_fini(struct ssi_drvdata *drvdata); #if defined(CONFIG_PM) int cc_resume_req_queue(struct ssi_drvdata *drvdata);