Re: [PATCH] dev->priv to netdev_priv(dev), drivers/net/tokenring/

Jan Engelhardt <[email protected]> writes:

> On Jul 23 2007 15:18, Yoann Padioleau wrote:
>>
>>Here is an excerpt of the semantic patch that performs the transformation
>>
>>@@
>>
>>- (T*) dev->priv
>>+ netdev_priv(dev)
>
> Note that dev->priv is a void*, and hence does not need casting.
> So dev->priv may also appear without one.

Yes you are right. I have shown only an excerpt of the semantic patch.
Sorry. There is another rule:

@ rule3 depends on rule1 && !rule1bis @
struct net_device *dev;
@@

- dev->priv
+ netdev_priv(dev)


If you look at my patch there is one such case

-	struct tok_info *ti = dev->priv;
+	struct tok_info *ti = netdev_priv(dev);



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH] dev->priv to netdev_priv(dev), drivers/net/tokenring/
    • From: Yoann Padioleau <[email protected]>
  • Re: [PATCH] dev->priv to netdev_priv(dev), drivers/net/tokenring/
    • From: Jan Engelhardt <[email protected]>

• Prev by Date: 2.6.23rc1 git: EIP is at acpi_processor_throttling_seq_show+0x8b/0xdd [processor]
• Next by Date: Re: [RFC] what should 'uptime' be on suspend?
• Previous by thread: Re: [PATCH] dev->priv to netdev_priv(dev), drivers/net/tokenring/
• Next by thread: Re: [PATCH] dev->priv to netdev_priv(dev), drivers/net/tokenring/
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]