lkml.org 
[lkml]   [2007]   [Jun]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Documentation/firmware_class/firmware_sample_driver.c
Hi,

[ It's good that you're trying to fix this, code in documentation should
be setting standards, clearly. ]

On 6/27/07, Anders Blomdell <anders.blomdell@control.lth.se> wrote:
> [...]
> Minor modifications to make the example load and unload without Oops
> [...]
> static int sample_init(void)
> {
> - device_initialize(&ghost_device);
> + device_register(&ghost_device);
> /* since there is no real hardware insertion I just call the
> * sample probe functions here */
> - sample_probe_specific();
> + /* sample_probe_specific(); */
> sample_probe_default();
> - sample_probe_async();
> + /*sample_probe_async();*/
> return 0;
> }

But IMO the above functions should be *fixed* to work properly instead
of simply commenting them out. If they are un-fixable, why even keep
the broken code in that file (only to mislead readers in future?). You
might as well remove those calls (and the function definitions) completely.
Best would be to fix them, of course.

My Rs. 0.02,
Satyam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-06-28 12:39    [W:0.046 / U:0.132 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site