From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <1521210128.23017.17.camel@linux.intel.com> Subject: Re: [PATCH v1 1/1] clk: Remove redundant selection of RATIONAL From: Andy Shevchenko To: Michael Turquette , Stephen Boyd , linux-clk@vger.kernel.org Date: Fri, 16 Mar 2018 16:22:08 +0200 In-Reply-To: <1517240807.7000.1303.camel@linux.intel.com> References: <20180119174401.23051-1-andriy.shevchenko@linux.intel.com> <1517240807.7000.1303.camel@linux.intel.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 List-ID: On Mon, 2018-01-29 at 17:46 +0200, Andy Shevchenko wrote: > On Fri, 2018-01-19 at 19:44 +0200, Andy Shevchenko wrote: > > Since the commit > > > > 0777591e715a ("clk: fractional-divider: switch to rational best > > approximation") > > > > enables rational best approximation to all common based clocks, > > there > > is > > no need to repeat the selection in each of them. > > > > No functional changes intended. > > Any comment on this? Do we have any active maintainer for clock related patches? Who is that right now? Can we somehow keep patches commented? > > > > > Signed-off-by: Andy Shevchenko > > --- > > drivers/clk/Kconfig | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig > > index 98ce9fc6e6c0..2e65cf731b00 100644 > > --- a/drivers/clk/Kconfig > > +++ b/drivers/clk/Kconfig > > @@ -76,7 +76,6 @@ config COMMON_CLK_SI5351 > > tristate "Clock driver for SiLabs 5351A/B/C" > > depends on I2C > > select REGMAP_I2C > > - select RATIONAL > > ---help--- > > This driver supports Silicon Labs 5351A/B/C programmable > > clock > > generators. > > @@ -105,7 +104,6 @@ config COMMON_CLK_CDCE706 > > tristate "Clock driver for TI CDCE706 clock synthesizer" > > depends on I2C > > select REGMAP_I2C > > - select RATIONAL > > ---help--- > > This driver supports TI CDCE706 programmable 3-PLL clock > > synthesizer. > > > > -- Andy Shevchenko Intel Finland Oy