On Fri, May 31, 2013 at 09:48:09PM -0400, Sean MacLennan wrote:
> > > +#if 0
> > >  struct phy_ofdm_rx_status_rxsc_sgien_exintfflag {
> > >   u8                      reserved:4;
> > >   u8                      rxsc:2;
> > >   u8                      sgi_en:1;
> > >   u8                      ex_intf_flag:1;
> > >  };
> > > +#endif
> > 
> > Didn't you just want to delete this?  The patch ifdefs out some
> > other code below as well.
> 
> No. I wanted to keep it as documentation just in case. But I wanted to
> be sure nobody was using it. What is normal practice here, delete and
> say you can always look it up in the older code?

Yeah.  Delete and look in git history as needed.

regards,
dan carpenter

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to