From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [RESEND PATCH] clk: stm32: fix: stm32 clock drivers are not compiled by default To: Stephen Boyd , Michael Turquette , CC: , , benjamin GAIGNARD , Loic PALLARDY References: <1525329609-7291-1-git-send-email-gabriel.fernandez@st.com> <0066b4db-8da6-007d-780d-a8ffc4884ad0@st.com> <152548833044.138124.7516235724401510636@swboyd.mtv.corp.google.com> <152642091618.237094.8294094887191293130@swboyd.mtv.corp.google.com> <152645747055.210890.12866212092853247736@swboyd.mtv.corp.google.com> From: Alexandre Torgue Message-ID: <503ec7d8-e493-3458-e974-5fab5649aaf8@st.com> Date: Wed, 16 May 2018 10:01:56 +0200 MIME-Version: 1.0 In-Reply-To: <152645747055.210890.12866212092853247736@swboyd.mtv.corp.google.com> Content-Type: text/plain; charset="utf-8"; format=flowed List-ID: On 05/16/2018 09:57 AM, Stephen Boyd wrote: > 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. > Thanks Stephen