Re: [PATCH] net: typhoon: Remove redundant casts
From: Rasmus Villemoes Date: Tue, 21 Oct 2014 16:51:43 +0200 > Both image_data and typhoon_fw->data are const u8*, so the cast to u8* > is unnecessary and confusing. > > Signed-off-by: Rasmus Villemoes Applied. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] net: typhoon: Remove redundant casts
From: Rasmus Villemoes li...@rasmusvillemoes.dk Date: Tue, 21 Oct 2014 16:51:43 +0200 Both image_data and typhoon_fw-data are const u8*, so the cast to u8* is unnecessary and confusing. Signed-off-by: Rasmus Villemoes li...@rasmusvillemoes.dk Applied. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] net: typhoon: Remove redundant casts
On Tue, 2014-10-21 at 16:51 +0200, Rasmus Villemoes wrote: > Both image_data and typhoon_fw->data are const u8*, so the cast to u8* > is unnecessary and confusing. > > Signed-off-by: Rasmus Villemoes Acked-by: David Dillow -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] net: typhoon: Remove redundant casts
Both image_data and typhoon_fw->data are const u8*, so the cast to u8* is unnecessary and confusing. Signed-off-by: Rasmus Villemoes --- drivers/net/ethernet/3com/typhoon.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/3com/typhoon.c b/drivers/net/ethernet/3com/typhoon.c index 48775b8..dede43f 100644 --- a/drivers/net/ethernet/3com/typhoon.c +++ b/drivers/net/ethernet/3com/typhoon.c @@ -1285,7 +1285,7 @@ typhoon_request_firmware(struct typhoon *tp) return err; } - image_data = (u8 *) typhoon_fw->data; + image_data = typhoon_fw->data; remaining = typhoon_fw->size; if (remaining < sizeof(struct typhoon_file_header)) goto invalid_fw; @@ -1343,7 +1343,7 @@ typhoon_download_firmware(struct typhoon *tp) int i; int err; - image_data = (u8 *) typhoon_fw->data; + image_data = typhoon_fw->data; fHdr = (struct typhoon_file_header *) image_data; /* Cannot just map the firmware image using pci_map_single() as -- 2.0.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] net: typhoon: Remove redundant casts
Both image_data and typhoon_fw-data are const u8*, so the cast to u8* is unnecessary and confusing. Signed-off-by: Rasmus Villemoes li...@rasmusvillemoes.dk --- drivers/net/ethernet/3com/typhoon.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/3com/typhoon.c b/drivers/net/ethernet/3com/typhoon.c index 48775b8..dede43f 100644 --- a/drivers/net/ethernet/3com/typhoon.c +++ b/drivers/net/ethernet/3com/typhoon.c @@ -1285,7 +1285,7 @@ typhoon_request_firmware(struct typhoon *tp) return err; } - image_data = (u8 *) typhoon_fw-data; + image_data = typhoon_fw-data; remaining = typhoon_fw-size; if (remaining sizeof(struct typhoon_file_header)) goto invalid_fw; @@ -1343,7 +1343,7 @@ typhoon_download_firmware(struct typhoon *tp) int i; int err; - image_data = (u8 *) typhoon_fw-data; + image_data = typhoon_fw-data; fHdr = (struct typhoon_file_header *) image_data; /* Cannot just map the firmware image using pci_map_single() as -- 2.0.4 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] net: typhoon: Remove redundant casts
On Tue, 2014-10-21 at 16:51 +0200, Rasmus Villemoes wrote: Both image_data and typhoon_fw-data are const u8*, so the cast to u8* is unnecessary and confusing. Signed-off-by: Rasmus Villemoes li...@rasmusvillemoes.dk Acked-by: David Dillow d...@thedillows.org -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/