Greg KH wrote on July 9th at 1:19 P.M. Ok, this has been fun to watch on lkml for a while now, but really, please, just stop doing this. Randomly searching the kernel source for FIXME lines and just commenting them out, isn't ok. Almost always, those lines are there because the original developer really doesn't know how else to resolve the issue. So, if the domain-specific-author doesn't have an idea of what to do, how does someone who is brand-new to the code know better? If you are looking for a task to do in the kernel, try drivers/staging/*/TODO for a list. Or look at the kernel janitor's list on kernelnewbies.org. Or try running the kernel and finding something that is broken for you and fixing that. Any of those would be better than randomly deleting FIXME lines. By doing that, you are just wasting maintainer's time. Which is the resource we have the least of at the moment. thanks, greg k-h Greg , I sent in another patch which enables omap_cfg_reg(USB2_SPEED) as stated in the fix me. The maintainers of this file stated to me this was dangerous due to the wires perhaps being shared with other parts of the board and we should depend on the boot loader to set this. The maintainers decided to remove this fix me not me based on their better understanding of this file. Cheers Nick On Wed, Jul 9, 2014 at 1:19 PM, Greg KH <greg@xxxxxxxxx> wrote: > Ok, this has been fun to watch on lkml for a while now, but really, > please, just stop doing this. Randomly searching the kernel source for > FIXME lines and just commenting them out, isn't ok. Almost always, > those lines are there because the original developer really doesn't know > how else to resolve the issue. > > So, if the domain-specific-author doesn't have an idea of what to do, > how does someone who is brand-new to the code know better? > > If you are looking for a task to do in the kernel, try > drivers/staging/*/TODO for a list. Or look at the kernel janitor's list > on kernelnewbies.org. Or try running the kernel and finding something > that is broken for you and fixing that. Any of those would be better > than randomly deleting FIXME lines. By doing that, you are just wasting > maintainer's time. Which is the resource we have the least of at the > moment. > > thanks, > > greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
