[PATCH net-next v2 09/10] can: ems_pci: Remove unnecessary (void*) conversions

[PATCH net-next v2 09/10] can: ems_pci: Remove unnecessary (void*) conversions

[Su Hui]

From: wuych <yunchuan@nfschina.com>

Pointer variables of void * type do not require type cast.

Signed-off-by: wuych <yunchuan@nfschina.com>
---
 drivers/net/can/sja1000/ems_pci.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/can/sja1000/ems_pci.c b/drivers/net/can/sja1000/ems_pci.c
index c56e27223e5f..ac86640998a8 100644
--- a/drivers/net/can/sja1000/ems_pci.c
+++ b/drivers/net/can/sja1000/ems_pci.c
@@ -148,7 +148,7 @@ static void ems_pci_v1_write_reg(const struct sja1000_priv *priv,
 
 static void ems_pci_v1_post_irq(const struct sja1000_priv *priv)
 {
-	struct ems_pci_card *card = (struct ems_pci_card *)priv->priv;
+	struct ems_pci_card *card = priv->priv;
 
 	/* reset int flag of pita */
 	writel(PITA2_ICR_INT0_EN | PITA2_ICR_INT0,
@@ -168,7 +168,7 @@ static void ems_pci_v2_write_reg(const struct sja1000_priv *priv,
 
 static void ems_pci_v2_post_irq(const struct sja1000_priv *priv)
 {
-	struct ems_pci_card *card = (struct ems_pci_card *)priv->priv;
+	struct ems_pci_card *card = priv->priv;
 
 	writel(PLX_ICSR_ENA_CLR, card->conf_addr + PLX_ICSR);
 }
@@ -186,7 +186,7 @@ static void ems_pci_v3_write_reg(const struct sja1000_priv *priv,
 
 static void ems_pci_v3_post_irq(const struct sja1000_priv *priv)
 {
-	struct ems_pci_card *card = (struct ems_pci_card *)priv->priv;
+	struct ems_pci_card *card = priv->priv;
 
 	writel(ASIX_LINTSR_INT0AC, card->conf_addr + ASIX_LINTSR);
 }
-- 
2.30.2

Re: [PATCH net-next v2 09/10] can: ems_pci: Remove unnecessary (void*) conversions

[Marc Kleine-Budde]

[-- Attachment #1: Type: text/plain, Size: 655 bytes --]

On 10.07.2023 14:41:38, Su Hui wrote:
> From: wuych <yunchuan@nfschina.com>
> 
> Pointer variables of void * type do not require type cast.

I like the idea. Please add my Acked-by: Marc Kleine-Budde
<mkl@pengutronix.de>, after you've fixed the issue:

> Signed-off-by: wuych <yunchuan@nfschina.com>

This patch is not Signed-off-by the contributing person.

regards,
Marc

-- 
Pengutronix e.K.                 | Marc Kleine-Budde          |
Embedded Linux                   | https://www.pengutronix.de |
Vertretung Nürnberg              | Phone: +49-5121-206917-129 |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-9   |

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH net-next v2 09/10] can: ems_pci: Remove unnecessary (void*) conversions

[yunchuan]

On 2023/7/10 14:49, Marc Kleine-Budde wrote:
> On 10.07.2023 14:41:38, Su Hui wrote:
>> From: wuych <yunchuan@nfschina.com>
>>
>> Pointer variables of void * type do not require type cast.
> I like the idea. Please add my Acked-by: Marc Kleine-Budde
> <mkl@pengutronix.de>, after you've fixed the issue:
that's great, I will do this.
thanks:)!

wuych

>> Signed-off-by: wuych <yunchuan@nfschina.com>
> This patch is not Signed-off-by the contributing person.
>
> regards,
> Marc
>

Re: [PATCH net-next v2 09/10] can: ems_pci: Remove unnecessary (void*) conversions

[yunchuan]

On 2023/7/10 14:53, yunchuan wrote:
> On 2023/7/10 14:49, Marc Kleine-Budde wrote:
>> On 10.07.2023 14:41:38, Su Hui wrote:
>>> From: wuych <yunchuan@nfschina.com>
>>>
>>> Pointer variables of void * type do not require type cast.
>> I like the idea. Please add my Acked-by: Marc Kleine-Budde
>> <mkl@pengutronix.de>, after you've fixed the issue:
> that's great, I will do this.
> thanks:)!
>
> wuych
>
>>> Signed-off-by: wuych <yunchuan@nfschina.com>
>> This patch is not Signed-off-by the contributing person.
>>

Oh, sorry for this, I just mixed the email because of some careless reasons.
The send email <suhui@nfschina.com> is my co-worker.
But the contributing person is me, sorry again!

wuych


>> regards,
>> Marc
>>