linux-kernel - RE: [PATCH] drivers:staging:ti-st: mv ti_wilink

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <19F8576C6E063C45BE387C64729E739404A9D8AEA7@dbde02.ent.ti.com>
Date:	Thu, 30 Sep 2010 23:03:02 +0530
From:	"Savoy, Pavan" <pavan_savoy@...com>
To:	Greg KH <greg@...ah.com>
CC:	"gregkh@...e.de" <gregkh@...e.de>,
	"alan@...rguk.ukuu.org.uk" <alan@...rguk.ukuu.org.uk>,
	"devel@...verdev.osuosl.org" <devel@...verdev.osuosl.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] drivers:staging:ti-st: mv ti_wilink_st header


Greg,

 
> -----Original Message-----
> From: Greg KH [mailto:greg@...ah.com]
> Sent: Thursday, September 30, 2010 6:51 AM
> To: Savoy, Pavan
> Cc: gregkh@...e.de; alan@...rguk.ukuu.org.uk; devel@...verdev.osuosl.org;
> linux-kernel@...r.kernel.org
> Subject: Re: [PATCH] drivers:staging:ti-st: mv ti_wilink_st header
> 
> On Wed, Sep 29, 2010 at 12:25:22PM -0400, pavan_savoy@...com wrote:
> > From: Pavan Savoy <pavan_savoy@...com>
> >
> > Greg,
> >
> > How is the movement of drivers done out of staging?
> > Should I submit patches like these, which move individual files without
> > breaking the build during anytime of movement?
> 
> Yes.
> 
> > In few following days I plan to submit patches to move the ST headers
> sources
> > out.
> > ti_wilink_st.h -> include/linux/
> > st_core.c, st_kim.c and st_ll.c -> drivers/misc/
> > Makefile and Kconfig changes to drivers/misc/Kconfig and
> drivers/misc/Makefile
> > bt_drv.c -> drivers/bluetooth/
> 
> Yes, that sounds great.
> 
> But, the bluetooth patch should go to the bluetooth maintainer for his
> review, I'll need the ack from him before I can apply it.

Yes, I will loop in linux-bluetooth and Marcel when I post the Bluetooth driver patch, he has been already introduced to ST, so hopefully the transition is smoother.

> Oh, this patch looks nice, but I found one problem:
> 
> > --- a/drivers/staging/ti-st/ti_wilink_st.h
> > +++ /dev/null
> > -#ifndef TI_WILINK_ST_H
> > -#define TI_WILINK_ST_H
> > -
> > -/* TODO:
> > - * Move the following to tty.h upon acceptance
> > - */
> > -#define N_TI_WL	22	/* Ldisc for TI's WL BT, FM, GPS combo chips */
> 
> We need to do that now, before you move this file.
> 
> Then, you can drop this:
> 
> > -#ifdef __KERNEL__
> 
> As it will not be needed, right?
> 
> Care to send me a patch that does this first?

Yes, sending it right away.

> thanks,
> 
> greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/