linux-kernel - Re: [PATCH 1/2] usb: host: Enable compile testing for some of drivers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20191230194249.GA26581@kozik-lap>
Date:   Mon, 30 Dec 2019 20:42:49 +0100
From:   Krzysztof Kozlowski <krzk@...nel.org>
To:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:     Felipe Balbi <balbi@...nel.org>, linux-usb@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] usb: host: Enable compile testing for some of drivers

On Mon, Dec 30, 2019 at 08:36:21PM +0100, Greg Kroah-Hartman wrote:
> On Mon, Dec 30, 2019 at 08:29:56PM +0100, Greg Kroah-Hartman wrote:
> > On Mon, Dec 30, 2019 at 06:22:14PM +0100, Krzysztof Kozlowski wrote:
> > > Some of the USB host drivers can be compile tested to increase build
> > > coverage.  Add 'if' conditional to 'default y' so they will not get
> > > enabled by default on all other architectures.
> > > 
> > > Signed-off-by: Krzysztof Kozlowski <krzk@...nel.org>
> > > ---
> > >  drivers/usb/host/Kconfig | 54 ++++++++++++++++++++--------------------
> > >  1 file changed, 27 insertions(+), 27 deletions(-)
> > 
> > Nice, thanks for these, they should help out a lot with development.
> 
> And now I get this build warning with this patch:
> 
> WARNING: unmet direct dependencies detected for PHY_DA8XX_USB
>   Depends on [n]: ARCH_DAVINCI_DA8XX
>   Selected by [m]:
>   - USB_OHCI_HCD_DAVINCI [=m] && USB_SUPPORT [=y] && USB [=m] && (ARCH_DAVINCI_DA8XX || COMPILE_TEST [=y]) && USB_OHCI_HCD [=m]
> 
> 
> Care to provide a fix?

That's my fault. I was testing entire patchset which includes also other
trees. I sent patch for PHY here:
https://lore.kernel.org/lkml/20191230172449.17648-2-krzk@kernel.org/T/#u

Probably they should go together or we could ignore the error as it is
only for build testing.

Best regards,
Krzysztof

> 
> thanks,
> 
> greg k-h