On Fri, 10 Apr 2015, Ravi Kerur wrote: > > > On 4/9/2015 10:34 PM, Julia Lawall wrote: > > > > > > On Thu, 9 Apr 2015, Ravi Kerur wrote: > > > >> > >> > >> On 4/9/2015 1:48 AM, Dan Carpenter wrote: > >>> On Wed, Apr 08, 2015 at 05:36:01PM -0700, Ravi Kerur wrote: > >>>> > >>>> > >>>> On 4/7/2015 12:49 AM, Dan Carpenter wrote: > >>>>> On Fri, Apr 03, 2015 at 01:40:35PM -0700, Ravi Kerur wrote: > >>>>>> Team, > >>>>>> > >>>>>> I am planning to take up following items from TODO list. If it's already picked please let me know. > >>>>>> > >>>>>> * pci_set_dma_mask() and friends should use DMA_BIT_MASK(nn) instead of > >>>>>> > >>>>>> DMA_nnBIT_MASK or 0xffff... This is not 2.4 compatible, so beware of drivers with same code. [D: http://marc.theaimsgroup.com/?t=108001993000001] Don't forget to #include dma-mapping.h > >>>>>> > >>>>>> * check kmallocs for things like GFP_DMA without a memtype. > >>>>>> > >>>>> > >>>>> The TODO is desperately out of date. No one cares about 2.4 at all. > >>>>> These days we don't really allow drivers to have backwards compatability > >>>>> code so they compile on old kernels. That stuff has to be stored out of > >>>>> the main kernel tree. We also have the compat-wireless and other > >>>>> similar ways of backporting drivers. > >>>> > >>>> Thanks Dan. Do you recommend anything else for contribution? I am not a > >>>> newbie to kernel but never contributed to kernel before so any inputs > >>>> appreciated. > >>> > >>> There is always stuff to fix in staging. It's at all levels of > >>> difficulty. > >> > >> Can you please point me to the link or anything which lists what needs to be done. I can pick some of it. > > > > You can run checkpatch, or other tools and fix the things that they > > highlight. Once you start really looking at the code, you are likely to > > see other things that can be improved as well. Some staging drivers also > > have TODO lists. > > > > My question was which subsystem or specific area to start? Should I pick some random subsystem of my interest and start working on it? Dan suggested staging. In terms of learning, it doesn't matter what you choose to get started. You should avoid the following drivers, though, as they are on the way out of the kernel: i2o line6 media/parport media/tlg2300 media/vino julia -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
