[PATCH 1/2] misc: add CARMA DATA-FPGA Access Driver

Dave Jones davej at redhat.com
Wed Feb 9 04:29:11 EST 2011


On Tue, Feb 08, 2011 at 09:20:46AM -0800, Ira W. Snyder wrote:

 > > > +static DEVICE_ATTR(enable, S_IWUGO | S_IRUGO, data_en_show, data_en_set);
 > > 
 > > Are all of these really needed or most of them are for debug?
 > > 
 > 
 > Most are for debugging. They have proved useful a few times in
 > production to track down bugs.
 
File mode should probably not be world writable.
(checkpatch.pl should warn you about this now btw)

	Dave



More information about the Linuxppc-dev mailing list