Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Sam Ravnborg <sam@...>
Cc: <linux-kernel@...>, <aoliva@...>, <alan@...>, Abhay Salunke <Abhay_Salunke@...>, <kay.sievers@...>, Takashi Iwai <tiwai@...>
Date: Saturday, May 24, 2008 - 11:34 am

On Sat, 2008-05-24 at 16:25 +0100, David Woodhouse wrote:

Here's what I have (on top of the git tree) now. I just need to sort out
the clean/mrproper behaviour -- and possibly should rename the korg
firmware to firmware/korg/k1212.c_shipped?

commit 728aa4212690701823ce3dcf22816f3953923530
Author: David Woodhouse <dwmw2@infradead.org>
Date:   Sat May 24 11:56:32 2008 +0100

    firmware: move firmware files into firmware/
    
    Signed-off-by: David Woodhouse <dwmw2@infradead.org>

diff --git a/firmware/Makefile b/firmware/Makefile
index f6b0c3c..62c2825 100644
--- a/firmware/Makefile
+++ b/firmware/Makefile
@@ -2,10 +2,15 @@
 # kbuild file for firmware/
 #
 
+firmware-$(CONFIG_SND_KORG1212_FIRMWARE_IN_KERNEL) += korg/k1212
+
 firmware_bins := $(subst ",,$(CONFIG_BUILTIN_FIRMWARE))
-firmware_objs := $(patsubst %,%.o, $(firmware_bins))
-firmware_srcs := $(patsubst %,$(obj)/%.c, $(firmware_bins))
+firmware_srcs_generated := $(patsubst %,$(obj)/%.c, $(firmware_bins))
+firmware_objs := $(patsubst %,%.o, $(firmware_bins) $(firmware-y)) 
+firmware_dirs := $(patsubst %,$(objtree)/$(obj)/%,$(dir $(firmware_objs)))
 
+quiet_cmd_mkdir = MKDIR   $@
+      cmd_mkdir = mkdir -p $@
 
 quiet_cmd_fwbin = MK_FW   $@
       cmd_fwbin = echo '/* File automatically generated */' > $@ ;	\
@@ -16,9 +21,14 @@ quiet_cmd_fwbin = MK_FW   $@
 		  echo '};' >> $@ ;					\
 		  echo 'DECLARE_BUILTIN_FIRMWARE("$(patsubst firmware/%.c,%,$@)",fw);' >> $@
 
-$(firmware_srcs): $(obj)/%.c: $(srctree)/$(obj)/%
+$(firmware_srcs_generated): $(obj)/%.c: $(srctree)/$(obj)/%
 	$(call cmd,fwbin)
 
+$(firmware_dirs):
+	$(call cmd,mkdir)
+
+$(patsubst %,$(obj)/%,$(firmware_objs)): $(firmware_dirs)
+
 obj-y := $(firmware_objs)
 
-targets := $(firmware_objs)
+targets := $(firmware_objs) $(patsubst $(obj)/%,%, $(firmware_srcs_generated))
diff --git a/sound/pci/korg1212/korg1212-firmware.c b/firmware/korg/k1212.c
similarity index 100%
rename from sound/pci/korg1212/korg1212-firmware.c
rename to firmware/korg/k1212.c
diff --git a/sound/pci/korg1212/Makefile b/sound/pci/korg1212/Makefile
index 7a5ebdf..f11ce1b 100644
--- a/sound/pci/korg1212/Makefile
+++ b/sound/pci/korg1212/Makefile
@@ -7,4 +7,3 @@ snd-korg1212-objs := korg1212.o
 
 # Toplevel Module Dependency
 obj-$(CONFIG_SND_KORG1212) += snd-korg1212.o
-obj-$(CONFIG_SND_KORG1212_FIRMWARE_IN_KERNEL) += korg1212-firmware.o


-- 
dwmw2

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH 1/3] firmware: allow firmware files to be built i..., Lennart Sorensen, (Fri May 23, 12:38 pm)
[PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Fri May 23, 9:46 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sat May 24, 10:46 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sat May 24, 11:25 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sat May 24, 11:34 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sat May 24, 2:18 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 7:49 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Alexandre Oliva, (Sun May 25, 1:17 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:49 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Alexandre Oliva, (Sun May 25, 11:30 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Alexandre Oliva, (Sun May 25, 11:13 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Alexandre Oliva, (Mon May 26, 1:09 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 3:01 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 9:19 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:15 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:34 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:23 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:46 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 3:03 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sun May 25, 10:18 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 9:12 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:12 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:28 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 7:54 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sun May 25, 9:46 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:07 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sat May 24, 3:23 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sat May 24, 3:31 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Fri May 23, 2:07 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Fri May 23, 1:13 pm)