From: Jose Ignacio Tornos Martinez <jtornosm@redhat.com>
To: yongqin.liu@linaro.org
Cc: amit.pundir@linaro.org, davem@davemloft.net, edumazet@google.com,
inventor500@vivaldi.net, jstultz@google.com, jtornosm@redhat.com,
kuba@kernel.org, linux-kernel@vger.kernel.org,
linux-usb@vger.kernel.org, netdev@vger.kernel.org,
pabeni@redhat.com, stable@vger.kernel.org,
sumit.semwal@linaro.org
Subject: Re: [PATCH] net: usb: ax88179_178a: fix link status when link is set to down/up
Date: Thu, 30 May 2024 16:13:00 +0200 [thread overview]
Message-ID: <20240530141301.434601-1-jtornosm@redhat.com> (raw)
In-Reply-To: <CAMSo37U3Pree8XbHNBOzNXhFAiPss+8FQms1bLy06xeMeWfTcg@mail.gmail.com>
Hello Yongqin,
> is there any message that I could check to make sure if the
> initialization is finished?
> or like with adding some printk lines for some kernel functions to hack
>
>> Anyway, I will try to reproduce here and analyze it.
>
> Thanks very much! And please feel free to let me know if there is
> anything I could help with on the Android build.
I have finally managed to reproduce an error similar to the one mentioned
during the boot stage. I created a systemd service with a similar
configuration script to do the same and it works (I can reconfigure the
mac address at boot time, the ip address is configured and the interface
works), because the driver is completely initialized. In order to reproduce,
I have introduced a big delay in the probe operation to get closer in time
to the configuration script and the problem is there.
Maybe, the script set_ethaddr.sh could be synchronized with the driver, but
I think, if possible, it is better to check in a better way in the driver;
I will try it. When I have something I can comment you, if you can test it,
to be sure about the solution.
By the way, I have tried with my other fix to avoid the spurious link
messages, but it didn't help.
Best regards
José Ignacio
next prev parent reply other threads:[~2024-05-30 14:13 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-05-10 9:08 [PATCH] net: usb: ax88179_178a: fix link status when link is set to down/up Jose Ignacio Tornos Martinez
2024-05-11 15:23 ` Simon Horman
2024-05-13 23:00 ` patchwork-bot+netdevbpf
2024-05-23 4:15 ` Yongqin Liu
2024-05-23 6:45 ` Jose Ignacio Tornos Martinez
2024-05-24 20:08 ` Yongqin Liu
2024-05-28 9:18 ` Jose Ignacio Tornos Martinez
2024-05-28 15:15 ` Yongqin Liu
2024-05-30 14:13 ` Jose Ignacio Tornos Martinez [this message]
2024-06-13 9:45 ` Jose Ignacio Tornos Martinez
2024-06-13 9:59 ` Jose Ignacio Tornos Martinez
2024-06-13 11:46 ` Yongqin Liu
2024-06-14 17:48 ` Yongqin Liu
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=20240530141301.434601-1-jtornosm@redhat.com \
--to=jtornosm@redhat.com \
--cc=amit.pundir@linaro.org \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=inventor500@vivaldi.net \
--cc=jstultz@google.com \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=stable@vger.kernel.org \
--cc=sumit.semwal@linaro.org \
--cc=yongqin.liu@linaro.org \
/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;
as well as URLs for NNTP newsgroup(s).