crypto: octeontx2 - Simplify multiple return statements
Subsystems (2)
PATCH 1/1 crypto: octeontx2 - Simplify multiple return statements
Simplify multiple return statements by directly returning the boolean expressions.
Signed-off-by Thorsten Blum <[email protected]>
--- .../marvell/octeontx2/otx2_cpt_common.h | 34 ++++++------------- 1 file changed, 10 insertions(+), 24 deletions(-)
drivers/crypto/marvell/octeontx2/otx2_cpt_common.h +10 -24
@@ -145,11 +145,8 @@ static inline u64 otx2_cpt_read64(void __iomem *reg_base, u64 blk, u64 slot,static inline bool is_dev_otx2 (struct pci_dev *pdev) { -if (pdev->device ==OTX2_CPT_PCI_PF_DEVICE_ID || - pdev->device ==OTX2_CPT_PCI_VF_DEVICE_ID ) -return true; - -return false; +return pdev->device ==OTX2_CPT_PCI_PF_DEVICE_ID || + pdev->device ==OTX2_CPT_PCI_VF_DEVICE_ID ; }static inline bool is_dev_cn10ka (struct pci_dev *pdev)
@@ -159,12 +156,10 @@ static inline bool is_dev_cn10ka(struct pci_dev *pdev)static inline bool is_dev_cn10ka_ax (struct pci_dev *pdev) { -if (pdev->subsystem_device ==CPT_PCI_SUBSYS_DEVID_CN10K_A && - ((pdev->revision &0xFF ) ==4 || (pdev->revision &0xFF ) ==0x50 || - (pdev->revision &0xff ) ==0x51 )) -return true; - -return false; +return pdev->subsystem_device ==CPT_PCI_SUBSYS_DEVID_CN10K_A && + ((pdev->revision &0xFF ) ==4 || + (pdev->revision &0xFF ) ==0x50 || + (pdev->revision &0xFF ) ==0x51 ); }static inline bool is_dev_cn10kb (struct pci_dev *pdev)
@@ -174,11 +169,8 @@ static inline bool is_dev_cn10kb(struct pci_dev *pdev)static inline bool is_dev_cn10ka_b0 (struct pci_dev *pdev) { -if (pdev->subsystem_device ==CPT_PCI_SUBSYS_DEVID_CN10K_A && - (pdev->revision &0xFF ) ==0x54 ) -return true; - -return false; +return pdev->subsystem_device ==CPT_PCI_SUBSYS_DEVID_CN10K_A && + (pdev->revision &0xFF ) ==0x54 ; } static inline void otx2_cpt_set_hw_caps(struct pci_dev *pdev,
@@ -192,18 +184,12 @@ static inline void otx2_cpt_set_hw_caps(struct pci_dev *pdev,static inline bool cpt_is_errata_38550_exists (struct pci_dev *pdev) { -if (is_dev_otx2 (pdev) ||is_dev_cn10ka_ax (pdev)) -return true; - -return false; +return is_dev_otx2 (pdev) ||is_dev_cn10ka_ax (pdev); }static inline bool cpt_feature_sgv2 (struct pci_dev *pdev) { -if (!is_dev_otx2 (pdev) && !is_dev_cn10ka_ax (pdev)) -return true; - -return false; +return !is_dev_otx2 (pdev) && !is_dev_cn10ka_ax (pdev); }int otx2_cpt_send_ready_msg (struct otx2_mbox *mbox,struct pci_dev *pdev);
Signature · 1 line
2.49.0
Herbert Xu
maintainerQuoted 7 lines · Thorsten Blum
Simplify multiple return statements by directly returning the boolean expressions.
Signed-off-by Thorsten Blum <[email protected]>--- .../marvell/octeontx2/otx2_cpt_common.h | 34 ++++++------------- 1 file changed, 10 insertions(+), 24 deletions(-)
Patch applied. Thanks.
Signature · 3 lines
Email: Herbert Xu <[email protected]> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt