ieee802154: ca8210: fix some kernel coding style errors
authorStefan Schmidt <stefan@osg.samsung.com>
Fri, 22 Sep 2017 12:14:00 +0000 (14:14 +0200)
committerStefan Schmidt <stefan@osg.samsung.com>
Mon, 6 Nov 2017 15:39:14 +0000 (16:39 +0100)
Remove unneeded parentheses and fix format for pointer style.

Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com>
Acked-by: Harry Morris <h.morris@cascoda.com>
Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com>
drivers/net/ieee802154/ca8210.c

index e6b8ce81a6c3206a99b8daa7d9a7ea52bcf9b8da..3da40a17e8cea7d9b74424efd66d5c05ef2c2fa6 100644 (file)
@@ -1303,7 +1303,7 @@ static u8 tdme_checkpibattribute(
                break;
        /* MAC */
        case MAC_BATT_LIFE_EXT_PERIODS:
-               if ((value < 6) || (value > 41))
+               if (value < 6 || value > 41)
                        status = MAC_INVALID_PARAMETER;
                break;
        case MAC_BEACON_PAYLOAD:
@@ -1319,7 +1319,7 @@ static u8 tdme_checkpibattribute(
                        status = MAC_INVALID_PARAMETER;
                break;
        case MAC_MAX_BE:
-               if ((value < 3) || (value > 8))
+               if (value < 3 || value > 8)
                        status = MAC_INVALID_PARAMETER;
                break;
        case MAC_MAX_CSMA_BACKOFFS:
@@ -1335,7 +1335,7 @@ static u8 tdme_checkpibattribute(
                        status = MAC_INVALID_PARAMETER;
                break;
        case MAC_RESPONSE_WAIT_TIME:
-               if ((value < 2) || (value > 64))
+               if (value < 2 || value > 64)
                        status = MAC_INVALID_PARAMETER;
                break;
        case MAC_SUPERFRAME_ORDER:
@@ -1511,7 +1511,7 @@ static u8 mcps_data_request(
        psec = (struct secspec *)(command.pdata.data_req.msdu + msdu_length);
        command.length = sizeof(struct mcps_data_request_pset) -
                MAX_DATA_SIZE + msdu_length;
-       if (!security || (security->security_level == 0)) {
+       if (!security || security->security_level == 0) {
                psec->security_level = 0;
                command.length += 1;
        } else {
@@ -1561,7 +1561,7 @@ static u8 mlme_reset_request_sync(
        status = response.pdata.status;
 
        /* reset COORD Bit for Channel Filtering as Coordinator */
-       if (CA8210_MAC_WORKAROUNDS && set_default_pib && (!status)) {
+       if (CA8210_MAC_WORKAROUNDS && set_default_pib && !status) {
                status = tdme_setsfr_request_sync(
                        0,
                        CA8210_SFR_MACCON,
@@ -2369,7 +2369,7 @@ static int ca8210_set_promiscuous_mode(struct ieee802154_hw *hw, const bool on)
                MAC_PROMISCUOUS_MODE,
                0,
                1,
-               (const void*)&on,
+               (const void *)&on,
                priv->spi
        );
        if (status) {