Re: [PATCH] drm/msm: update adreno firmware path in MODULE_FIRMWARE

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2017-11-23 at 16:09 +0100, Nicolas Dechesne wrote:
> On Thu, Nov 23, 2017 at 3:54 PM, Rob Clark <robdclark@xxxxxxxxx> wrote:
> > On Thu, Nov 23, 2017 at 5:32 AM, Nicolas Dechesne
> > <nicolas.dechesne@xxxxxxxxxx> wrote:
> > > The preferred location for Adreno firmware files is now in qcom/ subfolder,
> > > especially now that we are adding some of them in linux-firmware.
> > > 
> > > Reported-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx>
> > > Signed-off-by: Nicolas Dechesne <nicolas.dechesne@xxxxxxxxxx>
> > 
> > Thanks, I was wondering if we should perhaps list both old and new
> > paths?  I'm not sure, maybe we don't need to care about dracut or
> > initrd generation for the legacy case (since mostly there you are
> > using fastboot).
> 
> I've been going back and forth on that too. and i decided to ignore
> the legacy paths... alternatively we could use the legacy paths for
> a3xx and the new path for a5xx since we have links for a3xx files...
> but i thought it was too much noise..
> 
> > 
> > Also, I noticed we are missing a few a5xx fw files, but perhaps that
> > should be fixed with a separate patch.
> 
> as you want. I can include them if you prefer and resend.

Whenever I've added MODULE_FIRMWARE information I've listed only the
first-choice firmware paths.

initramfs-tools will warn when including a module if some of the listed
firmware is not available, so listing multiple paths for the same
firmware will likely mean it always warns about one of them.

Ben.

-- 
Ben Hutchings
When in doubt, use brute force. - Ken Thompson

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux