Re: [PATCH v2] clk: let clk_disable() return immediately if clk is NULL or error

2016-04-15 Thread Stephen Boyd
On 04/14, Masahiro Yamada wrote: > > OK, now I notice another problem in my code; > if foo_clk_init() fails for reason [2], > clk_disable() WARN's due to zero enable_count. > > if (WARN_ON(core->enable_count == 0)) > return; > > > > Perhaps, I got screwed up by splitting clock init

Re: [PATCH v2 2/9] mmc: tmio, sh_mobile_sdhi: Add support for variable input clock frequency

2016-04-15 Thread Wolfram Sang
> > That seems like a bug in the clock driver. If it doesn't have > > independent dividers for each clock client then it shouldn't allow any > > client to change the frequency. So, we need MSTP clocks which do not use CLK_SET_RATE_PARENT on r8a7740. Probably the best way is to use the

Re: [PATCH v2 2/9] mmc: tmio, sh_mobile_sdhi: Add support for variable input clock frequency

2016-04-15 Thread Wolfram Sang
> > 1. The SDHI/MMC clocks now run much slower than before. Perhaps this is > > intentional, and a consequence of finding the best way to drive the SD > > card at the target frequency? > > I don't think is generally a problem. Probably even saves a little > power. If you insert an

Re: [PATCH] Documentation: power: reset: move gpio-{poweroff|restart} to proper place

2016-04-15 Thread Rob Herring
On Fri, Apr 15, 2016 at 2:48 AM, Linus Walleij wrote: > On Tue, Apr 12, 2016 at 6:00 PM, Wolfram Sang wrote: > >> From: Wolfram Sang >> >> I did only find them after a fuzzy search, so let them be where one >> would

Re: [alsa-devel] [PATCH 0/4] ASoC: add graph base connection on simple-card

2016-04-15 Thread Jean-Francois Moine
On Fri, 15 Apr 2016 09:26:03 + Kuninori Morimoto wrote: > > > According to Documentation/devicetree/bindings/graph.txt section > > > "Links between endpoints", the endpoint/remote-endpoint links are > > > bi-directional and describe the physical links: the

Re: [alsa-devel] [PATCH 0/4] ASoC: add graph base connection on simple-card

2016-04-15 Thread Kuninori Morimoto
Hi again > > According to Documentation/devicetree/bindings/graph.txt section > > "Links between endpoints", the endpoint/remote-endpoint links are > > bi-directional and describe the physical links: the audio port of the > > codec is connected to the audio controller and reverse. > > There is

Re: [git pull] pinctrl: sh-pfc: Updates for v4.7

2016-04-15 Thread Linus Walleij
On Wed, Apr 13, 2016 at 10:45 AM, Geert Uytterhoeven wrote: > Hi Linus, > > The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca: > > Linux 4.6-rc1 (2016-03-26 16:03:24 -0700) > > are available in the git repository at: > >

Re: [PATCH] Documentation: power: reset: move gpio-{poweroff|restart} to proper place

2016-04-15 Thread Linus Walleij
On Tue, Apr 12, 2016 at 6:00 PM, Wolfram Sang wrote: > From: Wolfram Sang > > I did only find them after a fuzzy search, so let them be where one > would expect them. > > Signed-off-by: Wolfram Sang I saw

Re: [alsa-devel] [PATCH 0/4] ASoC: add graph base connection on simple-card

2016-04-15 Thread Kuninori Morimoto
Hi Jean Thank you for your feedback > According to Documentation/devicetree/bindings/graph.txt section > "Links between endpoints", the endpoint/remote-endpoint links are > bi-directional and describe the physical links: the audio port of the > codec is connected to the audio controller and

Re: [PATCH v5 0/7] PM / Domains: Add DT bindings for the R-Car System Controller

2016-04-15 Thread Geert Uytterhoeven
Hi Simon, On Fri, Apr 15, 2016 at 3:04 AM, Simon Horman wrote: > On Wed, Apr 13, 2016 at 04:44:16PM +0200, Geert Uytterhoeven wrote: >> This patch series introduces DT bindings for the R-Car System >> Controller, as found on Renesas R-Car H1, R-Car Gen2, and R-Car Gen3 >>

Re: [GIT PULL] Renesas ARM Based SoC Drivers Updates for v4.7

2016-04-15 Thread Geert Uytterhoeven
Hi Simon, On Fri, Apr 15, 2016 at 2:33 AM, Simon Horman wrote: > On Thu, Apr 14, 2016 at 10:19:04AM +0200, Geert Uytterhoeven wrote: >> On Wed, Apr 13, 2016 at 10:23 PM, Olof Johansson wrote: >> > On Wed, Apr 13, 2016 at 09:57:39PM +0200, Geert Uytterhoeven