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  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Wed, 2 Apr 2014 20:52:18 +0000
From:	Insop Song <Insop.Song@...nspeed.com>
To:	Greg KH <gregkh@...uxfoundation.org>,
	Paul Bolle <pebolle@...cali.nl>
CC:	"devel@...uxdriverproject.org" <devel@...uxdriverproject.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"'insop.song@...il.com'" <insop.song@...il.com>
Subject: RE: [GIT PULL] Staging driver patches for 3.15-rc1

On Wed, April 02, 2014 1:04 PM, Greg KH wrote:
> On Wed, Apr 02, 2014 at 10:24:03AM +0200, Paul Bolle wrote:
> > On Tue, 2014-04-01 at 11:48 -0700, Greg KH wrote:
> > > Staging driver pull request for 3.15-rc1
> > >
> > > Here's the huge drivers/staging/ update for 3.15-rc1.
> > >
> > > Loads of cleanup fixes, a few drivers removed, and some new ones
> added.
> > >
> > > All have been in linux-next for a while.
> > >
> > >[...]
> > >
> > > Insop Song (1):
> > >       staging: fpgaboot: Xilinx FPGA firmware download driver
> >
> > This commit adds checks for CONFIG_B4860G100. Is a patch to add a
> > Kconfig symbol B4860G100 perhaps queued somewhere?
> 
> Insop, I thought this config option was coming from some other place, right?
> 

Paul,
I didn't include CONFIG_B4860G100 in Kconfig in original patch set, since programming FPGA method can vary in different system, and this was discussed during the review with Greg as well.

However, actual fpga programming method is well contained in io.c with ifdef-ed CONFIG_B4860G100, now I think I might better to update Kconfig to include CONFIG_B4860G100.

Greg, what do you think? Any harm to add custom board CONFIG* to staging Kconfig? Let me know.


> > It also adds this Makefile line:
> >     ccflags-$(CONFIG_GS_FPGA_DEBUG)    := -DDEBUG
> >
> > But there's no Kconfig symbol GS_FGPA_DEBUG. On the other hand, the
> > code involved doesn't seem to use either DEBUG or dev_dbg() so perhaps
> > it isn't actually needed. (But note that the code does check for
> > DEBUG_FPGA, which apparently has to be set manually.)
> 
> Odds are we can just drop this, patches gladly accepted :)
> 

I looked at it, and thank you for pointing that out.
They can be removed, as Greg said, patches are welcome :), or let me know if you are busy, then I can do that.

Thank you,

ISS
--
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/

Powered by blists - more mailing lists