From: Guenter Roeck <firstname.lastname@example.org> To: Krzysztof Kozlowski <email@example.com> Cc: Wim Van Sebroeck <firstname.lastname@example.org>, Thomas Bogendoerfer <email@example.com>, Philipp Zabel <firstname.lastname@example.org>, email@example.com, firstname.lastname@example.org, email@example.com, John Crispin <firstname.lastname@example.org>, Lee Jones <email@example.com>, kernel test robot <firstname.lastname@example.org> Subject: Re: [PATCH] MIPS: ralink: of: fix build of rt2880_wdt watchdog module Date: Mon, 5 Jul 2021 09:32:51 -0700 [thread overview] Message-ID: <email@example.com> (raw) In-Reply-To: <CA+Eumj4aZZPFFB-3PEeBVo6fXW_Fz97oswA3pZuu2aiysnp=nQ@mail.gmail.com> On 7/5/21 9:21 AM, Krzysztof Kozlowski wrote: > On Mon, 5 Jul 2021 at 18:08, Guenter Roeck <firstname.lastname@example.org> wrote: >> >> On 7/5/21 8:15 AM, Krzysztof Kozlowski wrote: >>> On 14/05/2021 16:04, Krzysztof Kozlowski wrote: >>>> On 14/05/2021 09:48, Guenter Roeck wrote: >>>>> On 5/14/21 6:17 AM, Krzysztof Kozlowski wrote: >>>>>> When rt2880_wdt watchdog driver is built as a module, the >>>>>> rt_sysc_membase needs to be exported (it is being used via inlined >>>>>> rt_sysc_r32): >>>>>> >>>>>> ERROR: modpost: "rt_sysc_membase" [drivers/watchdog/rt2880_wdt.ko] undefined! >>>>>> >>>>>> Reported-by: kernel test robot <email@example.com> >>>>>> Signed-off-by: Krzysztof Kozlowski <firstname.lastname@example.org> >>>>> >>>>> Acked-by: Guenter Roeck <email@example.com> >>>>> >>>>> I don't see a recent change in the code. Has that problem been there >>>>> all along ? >>>> >>>> I think the problem was there always but 0-day builder did not hit it >>>> until recently: >>>> https://lore.kernel.org/lkml/202105082122.yrF7ploNfirstname.lastname@example.org/ >>> >>> Hi Guenter, >>> >>> It seems that patch was not applied. Anything to fix here? >>> >> I see commit fef532ea0cd8 ("MIPS: ralink: export rt_sysc_membase for >> rt2880_wdt.c"). Does that not solve the problem ? > > Yes, that fixes it. It was sent later, though. :( > Happens to me all the time. What is important is that things are getting fixed. Personally I don't care that much for the credit. Note though that your patch subject was a bit misleading, and it may well be possible that the mips maintainers didn't realize that the fix needed to be applied to the mips tree. The subject line in the applied patch describes the change much better as it focuses on the change (what is done), while the subject line in your patch focuses on the impact (what it fixes). Guenter
next prev parent reply other threads:[~2021-07-05 16:32 UTC|newest] Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-05-14 13:17 Krzysztof Kozlowski 2021-05-14 13:48 ` Guenter Roeck 2021-05-14 14:04 ` Krzysztof Kozlowski 2021-07-05 15:15 ` Krzysztof Kozlowski 2021-07-05 16:08 ` Guenter Roeck 2021-07-05 16:21 ` Krzysztof Kozlowski 2021-07-05 16:32 ` Guenter Roeck [this message] 2021-05-14 18:31 ` John Crispin
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 \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --subject='Re: [PATCH] MIPS: ralink: of: fix build of rt2880_wdt watchdog module' \ /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
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).