mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Gene Heskett <gene.heskett@gmail.com>
To: Jaswinder Singh <jaswinder@infradead.org>
Cc: martin@valinux.com, gareth@valinux.com, airlied@linux.ie,
	dri-devel@lists.sourceforge.net,
	LKML <linux-kernel@vger.kernel.org>,
	David Woodhouse <dwmw2@infradead.org>
Subject: Re: [PATCH] radeon_cp: use request_firmware
Date: Thu, 07 Aug 2008 06:24:11 -0400	[thread overview]
Message-ID: <200808070624.12214.gene.heskett@gmail.com> (raw)
In-Reply-To: <1218095497.4016.3.camel@jaswinder.satnam>

On Thursday 07 August 2008, Jaswinder Singh wrote:
>Moved datah before datal because datah is required before datal.
>
>Firmware blob looks like this...
>	__be32 datah
>	__be32 datal
>
>Signed-off-by: Jaswinder Singh <jaswinder@infradead.org>

When applied to 2.6.27-rc2, firmware/Makefile was rejected, so I manually added 
the blob list to that Makefile in what looked to be the correct place.  And added a
"make firmware_install" to my kernel builder script.

Unforch:
  MK_FW   firmware/R100_cp.bin.ihex.gen.S
make[1]: *** No rule to make target `firmware/R100_cp.bin.ihex', needed by `firmware/R100_cp.bin.ihex.gen.o'.  Stop.
make[1]: *** Waiting for unfinished jobs....
make: *** [firmware] Error 2

I suspect I need to go back and hand edit the .config to add the "radeon/"
in front of each of those, so its looking for
 "firmware/radeon/R100_cp.bin.ihex" in each of those cases.  Correct?  Or can
I use "radeon/*" ?

[...]

-- 
Cheers, Gene
"There are four boxes to be used in defense of liberty:
 soap, ballot, jury, and ammo. Please use in that order."
-Ed Howdershelt (Author)
Fiber optics caused gas main leak

  reply	other threads:[~2008-08-07 13:29 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-07  7:51 Jaswinder Singh
2008-08-07 10:24 ` Gene Heskett [this message]
2008-08-07 17:04   ` Jaswinder Singh
2008-12-19 20:23 ` Jaswinder Singh
2008-12-21 20:58   ` Dave Airlie
2008-12-22 10:39     ` Julien Cristau
2008-12-23 14:53       ` Ben Hutchings
2008-12-30  3:02         ` Jaswinder Singh Rajput
2008-12-30  4:04           ` [PATCH] radeon: Use request_firmware() to load CP microcode Ben Hutchings
2008-12-22 11:27     ` [PATCH] radeon_cp: use request_firmware Jaswinder Singh
2009-04-13 16:27 Jaswinder Singh Rajput
2009-04-13 18:26 ` Ben Hutchings
2009-04-14  4:54   ` Jaswinder Singh Rajput
2009-04-14 14:26     ` Ben Hutchings
2009-04-14 14:50       ` Krzysztof Halasa
2009-04-14 21:06         ` Bill Davidsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200808070624.12214.gene.heskett@gmail.com \
    --to=gene.heskett@gmail.com \
    --cc=airlied@linux.ie \
    --cc=dri-devel@lists.sourceforge.net \
    --cc=dwmw2@infradead.org \
    --cc=gareth@valinux.com \
    --cc=jaswinder@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=martin@valinux.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome