From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751876AbcDPUPf (ORCPT ); Sat, 16 Apr 2016 16:15:35 -0400 Received: from mout.kundenserver.de ([217.72.192.73]:54606 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751614AbcDPUPe (ORCPT ); Sat, 16 Apr 2016 16:15:34 -0400 From: Arnd Bergmann To: Thor Thayer , Doug Thompson , Borislav Petkov , Mauro Carvalho Chehab Cc: Arnd Bergmann , Alan Tull , Dinh Nguyen , linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/2] EDAC, altera: remove useless casts Date: Sat, 16 Apr 2016 22:13:55 +0200 Message-Id: <1460837650-1237650-1-git-send-email-arnd@arndb.de> X-Mailer: git-send-email 2.7.0 X-Provags-ID: V03:K0:1OqIDYNzdnDpLq7gYHnDbVtkGUYNt/9BaSw+Aztiiyi2f4k6Inf whwrfgQRwOawFHrfmsYQJt9j1OnBGEQfk2ufxeVbpQHO5ALAG+OyCWdTSOQe4It2SGtQygB uM8vfQKJQcm8TgpVOKMB0xMxFZMeG93v+5PiFEpeNxdgSIPy3aQ1EQkCnoJS27B9doVN7P/ LHQzhjRbqHCMMnSK4is5g== X-UI-Out-Filterresults: notjunk:1;V01:K0:CO6RaDE6QCs=:iP0vDEkCZyogwWHiQbF3tB OyIi14MTbuc8SWtfQNBhRuG3psZLab5DEQlp18bsqVmTxdG6wiAP7STErXcBzpcrPMBMRLuqI o5SdLuvZIyDRqo50dO6DdpoxSodK0cGfbyfb3HDg+l26sXXpUo1EEzg7JOyQyUwf+z9NiCvwU Yc8lH10ZnQoYUnq7qSXo2xfv5xo5m37TQMw0UD6V3syHORooEa2QV4B6+Zt+WdpMzuo4uqS5f /Uh0IK9+ZgJZk9kjeGlrVB4izCoEHSdy51OXfCJALXv405SGvmik0NJR7I+cFuTDwh8s6jyaK o/xuOaHgDGgyVyJzZg3nFd6HFwzEbfSrg7bFG6Vo1ppVwYjTtGRdZfoZ272e3l0k8rCicVaav OSd5KnMFOpkvo5j8pOVk60MrIl9jBs8qhJFfvreR0FQN/FyB3mnJQhOdPhsSND6tJBY3Iy0bW bC4RnZnTqyjgZc0YOiaXT1b/wo64iy2WaLrF86S5QdFoPDrsNH7ntu8ekOaxCO9m+VvaG+h8w 5+L8zFa0rW0BtRQsnEf9RtkBGav+OUIjbGjkK+NV7UIw2lIegq5jhmku2YAai4gxR+2U4kicz 2n9F4slaHvhJc54YT3ENSKwZXHY8DTlzi0vkKgvJOOpjstiZDBkNjyeo/BPGuTOpIQeFCYo7W jXaoTcUFIpusK1FOXnk1d6o/QrKxtGDv+lybTz7wZ7Sf+tHZpDXPBZWdBIrcwCS0vuDE= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The altera EDAC driver refers to its per-device data using a cast to '(void *)', which makes the pointer non-const, though both the source and destination are actually const. Removing the annotation makes the reference (almost) fit into a single line for improved readability, and ensures that it is actually defined as const. Signed-off-by: Arnd Bergmann --- drivers/edac/altera_edac.c | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/drivers/edac/altera_edac.c b/drivers/edac/altera_edac.c index 11775dc0b139..cc987b4ce908 100644 --- a/drivers/edac/altera_edac.c +++ b/drivers/edac/altera_edac.c @@ -232,8 +232,8 @@ static unsigned long get_total_mem(void) } static const struct of_device_id altr_sdram_ctrl_of_match[] = { - { .compatible = "altr,sdram-edac", .data = (void *)&c5_data}, - { .compatible = "altr,sdram-edac-a10", .data = (void *)&a10_data}, + { .compatible = "altr,sdram-edac", .data = &c5_data}, + { .compatible = "altr,sdram-edac-a10", .data = &a10_data}, {}, }; MODULE_DEVICE_TABLE(of, altr_sdram_ctrl_of_match); @@ -705,15 +705,12 @@ static void altr_create_edacdev_dbgfs(struct edac_device_ctl_info *edac_dci, static const struct of_device_id altr_edac_device_of_match[] = { #ifdef CONFIG_EDAC_ALTERA_L2C - { .compatible = "altr,socfpga-l2-ecc", .data = (void *)&l2ecc_data }, - { .compatible = "altr,socfpga-a10-l2-ecc", - .data = (void *)&a10_l2ecc_data }, + { .compatible = "altr,socfpga-l2-ecc", .data = &l2ecc_data }, + { .compatible = "altr,socfpga-a10-l2-ecc", .data = &a10_l2ecc_data }, #endif #ifdef CONFIG_EDAC_ALTERA_OCRAM - { .compatible = "altr,socfpga-ocram-ecc", - .data = (void *)&ocramecc_data }, - { .compatible = "altr,socfpga-a10-ocram-ecc", - .data = (void *)&a10_ocramecc_data }, + { .compatible = "altr,socfpga-ocram-ecc", .data = &ocramecc_data }, + { .compatible = "altr,socfpga-a10-ocram-ecc", .data = &a10_ocramecc_data }, #endif {}, }; -- 2.7.0