Re: staging: rts5208: add support for rts5208 and rts5288

2014-04-29 Thread micky
gh to submit that patch myself. regards, dan carpenter On Fri, Mar 28, 2014 at 02:27:56AM +0300, Dan Carpenter wrote: Hello Micky Ching, The patch fa590c222fba: "staging: rts5208: add support for rts5208 and rts5288" from Nov 12, 2013, leads to the following static checker warning:

Re: staging: rts5208: add support for rts5208 and rts5288

2014-04-29 Thread Dan Carpenter
2fba: "staging: rts5208: add support for rts5208 > and rts5288" from Nov 12, 2013, leads to the following static checker > warning: > > drivers/staging/rts5208/rtsx_chip.c:107 rtsx_enable_bus_int() > warn: add curly braces? > > drivers/staging/rts5208/r

re: staging: rts5208: add support for rts5208 and rts5288

2014-03-27 Thread Dan Carpenter
Hello Micky Ching, The patch fa590c222fba: "staging: rts5208: add support for rts5208 and rts5288" from Nov 12, 2013, leads to the following static checker warning: drivers/staging/rts5208/rtsx_chip.c:107 rtsx_enable_bus_int() warn: add curly braces? drivers/stagi

Re: [PATCH] staging: rts5208: add support for rts5208 and rts5288

2013-11-11 Thread Greg KH
On Fri, Nov 08, 2013 at 05:00:43PM +0800, micky_ch...@realsil.com.cn wrote: > --- /dev/null > +++ b/drivers/staging/rts5208/TODO > @@ -0,0 +1,5 @@ > +TODO: > +- use kernel coding style > +- checkpatch.pl fixes > +- We will use the stack in drivers/mmc to implement > + rts5208/5288 in the future I