linux-kernel - Re: [RESEND PATCH] clk: stm32: fix: stm32 clock drivers are not compiled by default

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: <152645747055.210890.12866212092853247736@swboyd.mtv.corp.google.com>
Date:   Wed, 16 May 2018 00:57:50 -0700
From:   Stephen Boyd <sboyd@...nel.org>
To:     Alexandre Torgue <alexandre.torgue@...com>,
        Michael Turquette <mturquette@...libre.com>,
        gabriel.fernandez@...com
Cc:     linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org,
        benjamin GAIGNARD <benjamin.gaignard@...com>,
        Loic PALLARDY <loic.pallardy@...com>
Subject: Re: [RESEND PATCH] clk: stm32: fix: stm32 clock drivers are not compiled by
 default

Quoting Alexandre Torgue (2018-05-16 00:34:21)
> >>
> >> Sorry I don't understand. Clock driver is mandatory to boot each
> >> platform and depends on machine type. Do you see a use case where we
> >> could need to disable the clock driver? (it would impose to change
> >> devicetree to use fixed clocks and to have a booloader which configures
> >> all PLL and clocks).
> > 
> > No. My understanding of the patch that introduced the options was to
> > allow the user to disable these drivers if they wanted to. I'm just
> > reiterating the spirit of that patch. That must be wrong?
> > 
> Actually initial patch is a mistake. You can't boot STM32 platform 
> without clock driver. This patch fixes that, and offers the possibility 
> to not build the clock driver of the machine which are not used 
> (currently all machine are selected but to reduce memory footprint you 
> could disable not used machine in defconfig).

Ok I stuck the patch into clk-fixes. Will send off to Linus tomorrow or the
next day.