net: ipa: get rid of a useless line of code
authorAlex Elder <elder@linaro.org>
Mon, 9 Nov 2020 16:56:33 +0000 (10:56 -0600)
committerJakub Kicinski <kuba@kernel.org>
Wed, 11 Nov 2020 22:07:00 +0000 (14:07 -0800)
Delete a spurious line of code in ipa_hardware_config().  It reads a
register value then ignores the value, so is completely unnecessary.

Add a missing word in a comment.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ipa/ipa_main.c

index d1e582707800a8257ba2ae3fd2df380a33b67eae..bfe95a46acaf1a8766e010092dc57f0dfeb83961 100644 (file)
@@ -335,7 +335,6 @@ static void ipa_hardware_config(struct ipa *ipa)
        ipa_hardware_config_qsb(ipa);
 
        /* Configure aggregation granularity */
-       val = ioread32(ipa->reg_virt + IPA_REG_COUNTER_CFG_OFFSET);
        granularity = ipa_aggr_granularity_val(IPA_AGGR_GRANULARITY);
        val = u32_encode_bits(granularity, AGGR_GRANULARITY);
        iowrite32(val, ipa->reg_virt + IPA_REG_COUNTER_CFG_OFFSET);
@@ -787,7 +786,7 @@ static int ipa_probe(struct platform_device *pdev)
        if (ret)
                goto err_mem_exit;
 
-       /* Result is a non-zero mask endpoints that support filtering */
+       /* Result is a non-zero mask of endpoints that support filtering */
        ipa->filter_map = ipa_endpoint_init(ipa, data->endpoint_count,
                                            data->endpoint_data);
        if (!ipa->filter_map) {