Re: Proper use for linking foo.o_shipped after 69ea912fda74 ("kbuild: remove unneeded link_multi_deps")?

2020-05-06 Thread Florian Fainelli
On 5/6/2020 9:24 AM, Masahiro Yamada wrote: >> To me this is a regression, as it used to work and now it does not, thus >> we should be fixing it, any idea about how we go about it without doing >> a plain revert? > > > In fact, a patch exists. > >

Re: Proper use for linking foo.o_shipped after 69ea912fda74 ("kbuild: remove unneeded link_multi_deps")?

2020-05-06 Thread Masahiro Yamada
On Thu, May 7, 2020 at 1:03 AM Florian Fainelli wrote: > > > > On 5/6/2020 7:37 AM, Masahiro Yamada wrote: > > On Wed, May 6, 2020 at 1:45 PM Florian Fainelli > > wrote: > >> > >> Hi Masahiro, Michal, > >> > >> While updating our systems from 4.9 to 5.4, we noticed that one of the > >> kernel

Re: Proper use for linking foo.o_shipped after 69ea912fda74 ("kbuild: remove unneeded link_multi_deps")?

2020-05-06 Thread Florian Fainelli
On 5/6/2020 7:37 AM, Masahiro Yamada wrote: > On Wed, May 6, 2020 at 1:45 PM Florian Fainelli wrote: >> >> Hi Masahiro, Michal, >> >> While updating our systems from 4.9 to 5.4, we noticed that one of the >> kernel modules that we build, which is done by linking an object that we >>

Re: Proper use for linking foo.o_shipped after 69ea912fda74 ("kbuild: remove unneeded link_multi_deps")?

2020-05-06 Thread Masahiro Yamada
On Wed, May 6, 2020 at 1:45 PM Florian Fainelli wrote: > > Hi Masahiro, Michal, > > While updating our systems from 4.9 to 5.4, we noticed that one of the > kernel modules that we build, which is done by linking an object that we > pre-compile out of Kbuild stopped working. > > I bisected it down

Proper use for linking foo.o_shipped after 69ea912fda74 ("kbuild: remove unneeded link_multi_deps")?

2020-05-05 Thread Florian Fainelli
Hi Masahiro, Michal, While updating our systems from 4.9 to 5.4, we noticed that one of the kernel modules that we build, which is done by linking an object that we pre-compile out of Kbuild stopped working. I bisected it down to: commit 69ea912fda74a673d330d23595385e5b73e3a2b9