* Request to update 8852a firmware of rtw89 to v0.13.8.0 @ 2021-04-29 8:59 Pkshih 2021-05-11 18:05 ` Kalle Valo 0 siblings, 1 reply; 7+ messages in thread From: Pkshih @ 2021-04-29 8:59 UTC (permalink / raw) To: linux-firmware; +Cc: linux-wireless Hello, Please help to update 8852a firmware. This is first time I send a pull request. If anything is wrong, please let me know. Thank you Ping-Ke --- The following changes since commit 2f4f0f826eed328675e8a84b1d80e41a25e94ec4: rtw89: 8852a: update fw to v0.13.8.0 (2021-04-29 16:51:35 +0800) are available in the Git repository at: https://github.com/pkshih/linux-firmware.git for you to fetch changes up to 2f4f0f826eed328675e8a84b1d80e41a25e94ec4: rtw89: 8852a: update fw to v0.13.8.0 (2021-04-29 16:51:35 +0800) ---------------------------------------------------------------- ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Request to update 8852a firmware of rtw89 to v0.13.8.0 2021-04-29 8:59 Request to update 8852a firmware of rtw89 to v0.13.8.0 Pkshih @ 2021-05-11 18:05 ` Kalle Valo 2021-05-11 19:10 ` Brian Norris 0 siblings, 1 reply; 7+ messages in thread From: Kalle Valo @ 2021-05-11 18:05 UTC (permalink / raw) To: Pkshih; +Cc: linux-firmware, linux-wireless Pkshih <pkshih@realtek.com> writes: > Please help to update 8852a firmware. > This is first time I send a pull request. If anything is wrong, please let me know. rtw89 driver is not in upstream yet, and it has not been reviewed either. I recommend holding off pulling the firmware until the driver is reviewed. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Request to update 8852a firmware of rtw89 to v0.13.8.0 2021-05-11 18:05 ` Kalle Valo @ 2021-05-11 19:10 ` Brian Norris 2021-05-13 6:09 ` Kalle Valo 0 siblings, 1 reply; 7+ messages in thread From: Brian Norris @ 2021-05-11 19:10 UTC (permalink / raw) To: Kalle Valo; +Cc: Pkshih, linux-firmware, linux-wireless On Tue, May 11, 2021 at 11:05 AM Kalle Valo <kvalo@codeaurora.org> wrote: > > Pkshih <pkshih@realtek.com> writes: > > > Please help to update 8852a firmware. > > This is first time I send a pull request. If anything is wrong, please let me know. > > rtw89 driver is not in upstream yet, and it has not been reviewed > either. I recommend holding off pulling the firmware until the driver is > reviewed. FWIW, preliminary rtw89 firmware was already merged. This is just an update. I also wonder, what's the harm? People like me are actively testing the early versions, and I've done a very barebones amount of review (I hope to do more). It really helps me to have up-to-date firmware published somewhere central -- I guess I can take GitHub instead, but again, why not kernel.org? If anyone else is looking at testing, they know where to get the pieces too, and we can compare results. Brian ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Request to update 8852a firmware of rtw89 to v0.13.8.0 2021-05-11 19:10 ` Brian Norris @ 2021-05-13 6:09 ` Kalle Valo 2021-05-13 9:38 ` Pkshih 0 siblings, 1 reply; 7+ messages in thread From: Kalle Valo @ 2021-05-13 6:09 UTC (permalink / raw) To: Brian Norris; +Cc: Pkshih, linux-firmware, linux-wireless Brian Norris <briannorris@chromium.org> writes: > On Tue, May 11, 2021 at 11:05 AM Kalle Valo <kvalo@codeaurora.org> wrote: >> >> Pkshih <pkshih@realtek.com> writes: >> >> > Please help to update 8852a firmware. >> > This is first time I send a pull request. If anything is wrong, >> > please let me know. >> >> rtw89 driver is not in upstream yet, and it has not been reviewed >> either. I recommend holding off pulling the firmware until the driver is >> reviewed. > > FWIW, preliminary rtw89 firmware was already merged. This is just an update. > > I also wonder, what's the harm? People like me are actively testing > the early versions, and I've done a very barebones amount of review (I > hope to do more). It really helps me to have up-to-date firmware > published somewhere central -- I guess I can take GitHub instead, but > again, why not kernel.org? If anyone else is looking at testing, they > know where to get the pieces too, and we can compare results. It can create confusion to the users if during review we make changes how firmware files are handled. Some drivers have windows style .ini files which are not ok in an upstream driver, there could be changes in the file format etc. But I have no strong opinion here, my main motivation is just to try to keep things simple for maintaining the "interface" between linux-firmware and kernel wireless drivers. My preference is that the firmware files are ready for submission when a new driver is submitted for review, but the firmware files are submitted only after the driver is accepted. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: Request to update 8852a firmware of rtw89 to v0.13.8.0 2021-05-13 6:09 ` Kalle Valo @ 2021-05-13 9:38 ` Pkshih 2021-05-13 21:08 ` Brian Norris 0 siblings, 1 reply; 7+ messages in thread From: Pkshih @ 2021-05-13 9:38 UTC (permalink / raw) To: Kalle Valo, Brian Norris; +Cc: linux-firmware, linux-wireless > -----Original Message----- > From: kvalo=codeaurora.org@mg.codeaurora.org [mailto:kvalo=codeaurora.org@mg.codeaurora.org] On > Behalf Of Kalle Valo > Sent: Thursday, May 13, 2021 2:09 PM > To: Brian Norris > Cc: Pkshih; linux-firmware@kernel.org; linux-wireless@vger.kernel.org > Subject: Re: Request to update 8852a firmware of rtw89 to v0.13.8.0 > > Brian Norris <briannorris@chromium.org> writes: > > > On Tue, May 11, 2021 at 11:05 AM Kalle Valo <kvalo@codeaurora.org> wrote: > >> > >> Pkshih <pkshih@realtek.com> writes: > >> > >> > Please help to update 8852a firmware. > >> > This is first time I send a pull request. If anything is wrong, > >> > please let me know. > >> > >> rtw89 driver is not in upstream yet, and it has not been reviewed > >> either. I recommend holding off pulling the firmware until the driver is > >> reviewed. > > > > FWIW, preliminary rtw89 firmware was already merged. This is just an update. > > > > I also wonder, what's the harm? People like me are actively testing > > the early versions, and I've done a very barebones amount of review (I > > hope to do more). It really helps me to have up-to-date firmware > > published somewhere central -- I guess I can take GitHub instead, but > > again, why not kernel.org? If anyone else is looking at testing, they > > know where to get the pieces too, and we can compare results. > > It can create confusion to the users if during review we make changes > how firmware files are handled. Some drivers have windows style .ini > files which are not ok in an upstream driver, there could be changes in > the file format etc. > > But I have no strong opinion here, my main motivation is just to try to > keep things simple for maintaining the "interface" between > linux-firmware and kernel wireless drivers. My preference is that the > firmware files are ready for submission when a new driver is submitted > for review, but the firmware files are submitted only after the driver > is accepted. > I think I can maintain the firmware in GitHub before driver is accepted, and add some information about firmware version and link in the cover of patchset. Then, people can take proper firmware. -- Ping-Ke ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Request to update 8852a firmware of rtw89 to v0.13.8.0 2021-05-13 9:38 ` Pkshih @ 2021-05-13 21:08 ` Brian Norris 2021-05-14 6:22 ` Kalle Valo 0 siblings, 1 reply; 7+ messages in thread From: Brian Norris @ 2021-05-13 21:08 UTC (permalink / raw) To: Pkshih; +Cc: Kalle Valo, linux-firmware, linux-wireless On Thu, May 13, 2021 at 2:38 AM Pkshih <pkshih@realtek.com> wrote: > > -----Original Message----- > > From: kvalo=codeaurora.org@mg.codeaurora.org [mailto:kvalo=codeaurora.org@mg.codeaurora.org] On > > Behalf Of Kalle Valo > > It can create confusion to the users if during review we make changes > > how firmware files are handled. Some drivers have windows style .ini > > files which are not ok in an upstream driver, there could be changes in > > the file format etc. Sure, good point. I figured if there's no driver merged anywhere upstream, people (e.g., me) are taking their own life in their hands trying to utilize arbitrary versions from the mailing list. But I could see why you still don't want the potential mismatch. > > But I have no strong opinion here, my main motivation is just to try to > > keep things simple for maintaining the "interface" between > > linux-firmware and kernel wireless drivers. My preference is that the > > firmware files are ready for submission when a new driver is submitted > > for review, but the firmware files are submitted only after the driver > > is accepted. Ack, that's definitely important to me. The first versions had no publicly posted firmware, and at least one person (not me at the time) asked for it. My extension of that is that I don't want to _discourage_ vendors posting their firmware to public repos :) > I think I can maintain the firmware in GitHub before driver is accepted, > and add some information about firmware version and link in the cover of > patchset. Then, people can take proper firmware. Sure, that can work. Thanks, Brian ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Request to update 8852a firmware of rtw89 to v0.13.8.0 2021-05-13 21:08 ` Brian Norris @ 2021-05-14 6:22 ` Kalle Valo 0 siblings, 0 replies; 7+ messages in thread From: Kalle Valo @ 2021-05-14 6:22 UTC (permalink / raw) To: Brian Norris; +Cc: Pkshih, linux-firmware, linux-wireless Brian Norris <briannorris@chromium.org> writes: > On Thu, May 13, 2021 at 2:38 AM Pkshih <pkshih@realtek.com> wrote: >> > -----Original Message----- >> > From: kvalo=codeaurora.org@mg.codeaurora.org >> > [mailto:kvalo=codeaurora.org@mg.codeaurora.org] On >> > Behalf Of Kalle Valo > >> > It can create confusion to the users if during review we make changes >> > how firmware files are handled. Some drivers have windows style .ini >> > files which are not ok in an upstream driver, there could be changes in >> > the file format etc. > > Sure, good point. I figured if there's no driver merged anywhere > upstream, people (e.g., me) are taking their own life in their hands > trying to utilize arbitrary versions from the mailing list. But I > could see why you still don't want the potential mismatch. > >> > But I have no strong opinion here, my main motivation is just to try to >> > keep things simple for maintaining the "interface" between >> > linux-firmware and kernel wireless drivers. My preference is that the >> > firmware files are ready for submission when a new driver is submitted >> > for review, but the firmware files are submitted only after the driver >> > is accepted. > > Ack, that's definitely important to me. The first versions had no > publicly posted firmware, and at least one person (not me at the time) > asked for it. My extension of that is that I don't want to > _discourage_ vendors posting their firmware to public repos :) Yeah, I don't want to discourage posting either. It's just that we don't want to do it too early nor too late, which doesn't make it easy for submitters :) -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-05-14 6:22 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-04-29 8:59 Request to update 8852a firmware of rtw89 to v0.13.8.0 Pkshih 2021-05-11 18:05 ` Kalle Valo 2021-05-11 19:10 ` Brian Norris 2021-05-13 6:09 ` Kalle Valo 2021-05-13 9:38 ` Pkshih 2021-05-13 21:08 ` Brian Norris 2021-05-14 6:22 ` Kalle Valo
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).