* [PATCH] net: ipa: Declare IPA firmware with MODULE_FIRMWARE()
@ 2021-09-26 6:55 Shawn Guo
2021-09-26 13:46 ` Shawn Guo
2021-09-27 19:35 ` Bjorn Andersson
0 siblings, 2 replies; 5+ messages in thread
From: Shawn Guo @ 2021-09-26 6:55 UTC (permalink / raw)
To: David S . Miller
Cc: Jakub Kicinski, Alex Elder, Bjorn Andersson, Steev Klimaszewski,
netdev, linux-arm-msm, linux-kernel, Shawn Guo
Declare IPA firmware with MODULE_FIRMWARE(), so that initramfs tools can
build the firmware into initramfs image or warn on missing of the
firmware.
W: Possible missing firmware /lib/firmware/ipa_fws.mdt for module ipa
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
---
drivers/net/ipa/ipa_main.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/net/ipa/ipa_main.c b/drivers/net/ipa/ipa_main.c
index cdfa98a76e1f..264bebc78d1e 100644
--- a/drivers/net/ipa/ipa_main.c
+++ b/drivers/net/ipa/ipa_main.c
@@ -545,6 +545,8 @@ static int ipa_firmware_load(struct device *dev)
return ret;
}
+MODULE_FIRMWARE(IPA_FW_PATH_DEFAULT);
+
static const struct of_device_id ipa_match[] = {
{
.compatible = "qcom,msm8998-ipa",
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] net: ipa: Declare IPA firmware with MODULE_FIRMWARE()
2021-09-26 6:55 [PATCH] net: ipa: Declare IPA firmware with MODULE_FIRMWARE() Shawn Guo
@ 2021-09-26 13:46 ` Shawn Guo
2021-09-27 16:07 ` Alex Elder
2021-09-27 19:35 ` Bjorn Andersson
1 sibling, 1 reply; 5+ messages in thread
From: Shawn Guo @ 2021-09-26 13:46 UTC (permalink / raw)
To: David S . Miller
Cc: Jakub Kicinski, Alex Elder, Bjorn Andersson, Steev Klimaszewski,
netdev, linux-arm-msm, linux-kernel
On Sun, Sep 26, 2021 at 02:55:29PM +0800, Shawn Guo wrote:
> Declare IPA firmware with MODULE_FIRMWARE(), so that initramfs tools can
> build the firmware into initramfs image or warn on missing of the
> firmware.
>
> W: Possible missing firmware /lib/firmware/ipa_fws.mdt for module ipa
>
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
> ---
> drivers/net/ipa/ipa_main.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/net/ipa/ipa_main.c b/drivers/net/ipa/ipa_main.c
> index cdfa98a76e1f..264bebc78d1e 100644
> --- a/drivers/net/ipa/ipa_main.c
> +++ b/drivers/net/ipa/ipa_main.c
> @@ -545,6 +545,8 @@ static int ipa_firmware_load(struct device *dev)
> return ret;
> }
>
> +MODULE_FIRMWARE(IPA_FW_PATH_DEFAULT);
> +
Just reminded by Steev, .mdt file conventionally means we are using
split firmware. Building only .mdt file into initramfs is not
sufficient. So please disregard the patch.
Shawn
> static const struct of_device_id ipa_match[] = {
> {
> .compatible = "qcom,msm8998-ipa",
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] net: ipa: Declare IPA firmware with MODULE_FIRMWARE()
2021-09-26 13:46 ` Shawn Guo
@ 2021-09-27 16:07 ` Alex Elder
0 siblings, 0 replies; 5+ messages in thread
From: Alex Elder @ 2021-09-27 16:07 UTC (permalink / raw)
To: Shawn Guo, David S . Miller
Cc: Jakub Kicinski, Alex Elder, Bjorn Andersson, Steev Klimaszewski,
netdev, linux-arm-msm, linux-kernel
On 9/26/21 8:46 AM, Shawn Guo wrote:
> Just reminded by Steev, .mdt file conventionally means we are using
> split firmware. Building only .mdt file into initramfs is not
> sufficient. So please disregard the patch.
>
> Shawn
OK, will disregard. May I assume you are going to implement
a patch that does the right thing?
I had a note to myself to investigate using MODULE_FIRMWARE()
but hadn't gotten to it yet; I'm glad to have you do that
instead...
Thank you.
-Alex
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] net: ipa: Declare IPA firmware with MODULE_FIRMWARE()
2021-09-26 6:55 [PATCH] net: ipa: Declare IPA firmware with MODULE_FIRMWARE() Shawn Guo
2021-09-26 13:46 ` Shawn Guo
@ 2021-09-27 19:35 ` Bjorn Andersson
2021-09-27 20:30 ` Alex Elder
1 sibling, 1 reply; 5+ messages in thread
From: Bjorn Andersson @ 2021-09-27 19:35 UTC (permalink / raw)
To: Shawn Guo
Cc: David S . Miller, Jakub Kicinski, Alex Elder, Steev Klimaszewski,
netdev, linux-arm-msm, linux-kernel
On Sat 25 Sep 23:55 PDT 2021, Shawn Guo wrote:
> Declare IPA firmware with MODULE_FIRMWARE(), so that initramfs tools can
> build the firmware into initramfs image or warn on missing of the
> firmware.
>
> W: Possible missing firmware /lib/firmware/ipa_fws.mdt for module ipa
>
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
> ---
> drivers/net/ipa/ipa_main.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/net/ipa/ipa_main.c b/drivers/net/ipa/ipa_main.c
> index cdfa98a76e1f..264bebc78d1e 100644
> --- a/drivers/net/ipa/ipa_main.c
> +++ b/drivers/net/ipa/ipa_main.c
> @@ -545,6 +545,8 @@ static int ipa_firmware_load(struct device *dev)
> return ret;
> }
>
> +MODULE_FIRMWARE(IPA_FW_PATH_DEFAULT);
I think it's fair to say that no device that is capable of running
mkinitcpio should actually use IPA_FW_PATH_DEFAULT, but rather some
device-specific firmware file.
Regards,
Bjorn
> +
> static const struct of_device_id ipa_match[] = {
> {
> .compatible = "qcom,msm8998-ipa",
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] net: ipa: Declare IPA firmware with MODULE_FIRMWARE()
2021-09-27 19:35 ` Bjorn Andersson
@ 2021-09-27 20:30 ` Alex Elder
0 siblings, 0 replies; 5+ messages in thread
From: Alex Elder @ 2021-09-27 20:30 UTC (permalink / raw)
To: Bjorn Andersson, Shawn Guo
Cc: David S . Miller, Jakub Kicinski, Alex Elder, Steev Klimaszewski,
netdev, linux-arm-msm, linux-kernel
On 9/27/21 2:35 PM, Bjorn Andersson wrote:
> On Sat 25 Sep 23:55 PDT 2021, Shawn Guo wrote:
>
>> Declare IPA firmware with MODULE_FIRMWARE(), so that initramfs tools can
>> build the firmware into initramfs image or warn on missing of the
>> firmware.
>>
>> W: Possible missing firmware /lib/firmware/ipa_fws.mdt for module ipa
>>
>> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
>> ---
>> drivers/net/ipa/ipa_main.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/net/ipa/ipa_main.c b/drivers/net/ipa/ipa_main.c
>> index cdfa98a76e1f..264bebc78d1e 100644
>> --- a/drivers/net/ipa/ipa_main.c
>> +++ b/drivers/net/ipa/ipa_main.c
>> @@ -545,6 +545,8 @@ static int ipa_firmware_load(struct device *dev)
>> return ret;
>> }
>>
>> +MODULE_FIRMWARE(IPA_FW_PATH_DEFAULT);
>
> I think it's fair to say that no device that is capable of running
> mkinitcpio should actually use IPA_FW_PATH_DEFAULT, but rather some
> device-specific firmware file.
Actually, yes, I should have thought of that too.
The default used here just specifies the path name
used *if the firmware file name is not specified in
the DTB*.
So I don't think this is correct regardless of whether
the MDT file is split or not. I think that's what
Bjorn is saying here.
-Alex
>
> Regards,
> Bjorn
>
>> +
>> static const struct of_device_id ipa_match[] = {
>> {
>> .compatible = "qcom,msm8998-ipa",
>> --
>> 2.17.1
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-09-27 20:30 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-26 6:55 [PATCH] net: ipa: Declare IPA firmware with MODULE_FIRMWARE() Shawn Guo
2021-09-26 13:46 ` Shawn Guo
2021-09-27 16:07 ` Alex Elder
2021-09-27 19:35 ` Bjorn Andersson
2021-09-27 20:30 ` Alex Elder
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.