linux-kernel - Re: mmotm 2008-12-03-01-50 uploaded (sdhci)

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20081214184018.157062da@mjolnir.drzeus.cx>
Date:	Sun, 14 Dec 2008 18:40:18 +0100
From:	Pierre Ossman <drzeus@...eus.cx>
To:	Randy Dunlap <randy.dunlap@...cle.com>
Cc:	linux-kernel@...r.kernel.org, akpm <akpm@...ux-foundation.org>,
	drzeus-sdhci@...eus.cx, sdhci-devel@...t.drzeus.cx
Subject: Re: mmotm 2008-12-03-01-50 uploaded (sdhci)

On Wed, 03 Dec 2008 12:53:24 -0800
Randy Dunlap <randy.dunlap@...cle.com> wrote:

> akpm@...ux-foundation.org wrote:
> > The mm-of-the-moment snapshot 2008-12-03-01-50 has been uploaded to
> > 
> >    http://userweb.kernel.org/~akpm/mmotm/
> > 
> > It contains the following patches against 2.6.28-rc7:
> 
> 
> drivers/built-in.o: In function `sdhci_remove_host':
> (.text+0xd246a): undefined reference to `led_classdev_unregister'
> drivers/built-in.o: In function `sdhci_add_host':
> (.text+0xd5378): undefined reference to `led_classdev_register'
> make[1]: *** [.tmp_vmlinux1] Error 1
> 
> sdhci.c tries to handle the availability of led_()* without using
> Kconfig attributes.  In this case, MMC_SDHCI=y and LEDS_CLASS=m,
> which sdhci.c doesn't handle.
> 

I'm not sure how to properly solve this. I'm not a big fan of adding a
specific Kconfig option for each and every optional feature. I'd prefer
if we get the correct thing<tm> based on just the MMC_SDHCI and
LEDS_CLASS options.

-#if defined(CONFIG_LEDS_CLASS) || defined(CONFIG_LEDS_CLASS_MODULE)
+#if defined(CONFIG_LEDS_CLASS) || (defined(CONFIG_LEDS_CLASS_MODULE)
 	&& !defined(CONFIG_MMC_SDHCI_MODULE))

perhaps?

Rgds
-- 
     -- Pierre Ossman

  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.

Download attachment "signature.asc" of type "application/pgp-signature" (198 bytes)