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]