All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Chanho Park" <chanho61.park@samsung.com>
To: "'Sam Protsenko'" <semen.protsenko@linaro.org>,
	"'Krzysztof Kozlowski'" <krzysztof.kozlowski@canonical.com>,
	"'Rob Herring'" <robh+dt@kernel.org>
Cc: "'Jaewon Kim'" <jaewon02.kim@samsung.com>,
	"'David Virag'" <virag.david003@gmail.com>,
	"'Youngmin Nam'" <youngmin.nam@samsung.com>,
	"'Wolfram Sang'" <wsa@kernel.org>,
	"'Arnd Bergmann'" <arnd@arndb.de>, <linux-i2c@vger.kernel.org>,
	<devicetree@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>,
	<linux-samsung-soc@vger.kernel.org>
Subject: RE: [PATCH 5/6] i2c: exynos5: Add bus clock support
Date: Thu, 2 Dec 2021 19:30:11 +0900	[thread overview]
Message-ID: <002f01d7e767$96417940$c2c46bc0$@samsung.com> (raw)
In-Reply-To: <20211201190455.31646-6-semen.protsenko@linaro.org>

> In new Exynos SoCs (like Exynos850) where HSI2C is implemented as a part
> of USIv2 block, there are two clocks provided to HSI2C controller:
>   - PCLK: bus clock (APB), provides access to register interface
>   - IPCLK: operating IP-core clock; SCL is derived from this one
> 
> Both clocks have to be asserted for HSI2C to be functional in that case.
> 
> Add code to obtain and enable/disable PCLK in addition to already handled
> operating clock. Make it optional though, as older Exynos SoC variants
> only have one HSI2C clock.
> 
> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

Reviewed-by: Chanho Park <chanho61.park@samsung.com>

Best Regards,
Chanho Park



WARNING: multiple messages have this Message-ID (diff)
From: "Chanho Park" <chanho61.park@samsung.com>
To: "'Sam Protsenko'" <semen.protsenko@linaro.org>,
	"'Krzysztof Kozlowski'" <krzysztof.kozlowski@canonical.com>,
	"'Rob Herring'" <robh+dt@kernel.org>
Cc: "'Jaewon Kim'" <jaewon02.kim@samsung.com>,
	"'David Virag'" <virag.david003@gmail.com>,
	"'Youngmin Nam'" <youngmin.nam@samsung.com>,
	"'Wolfram Sang'" <wsa@kernel.org>,
	"'Arnd	Bergmann'" <arnd@arndb.de>, <linux-i2c@vger.kernel.org>,
	<devicetree@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>,
	<linux-samsung-soc@vger.kernel.org>
Subject: RE: [PATCH 5/6] i2c: exynos5: Add bus clock support
Date: Thu, 2 Dec 2021 19:30:11 +0900	[thread overview]
Message-ID: <002f01d7e767$96417940$c2c46bc0$@samsung.com> (raw)
In-Reply-To: <20211201190455.31646-6-semen.protsenko@linaro.org>

> In new Exynos SoCs (like Exynos850) where HSI2C is implemented as a part
> of USIv2 block, there are two clocks provided to HSI2C controller:
>   - PCLK: bus clock (APB), provides access to register interface
>   - IPCLK: operating IP-core clock; SCL is derived from this one
> 
> Both clocks have to be asserted for HSI2C to be functional in that case.
> 
> Add code to obtain and enable/disable PCLK in addition to already handled
> operating clock. Make it optional though, as older Exynos SoC variants
> only have one HSI2C clock.
> 
> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

Reviewed-by: Chanho Park <chanho61.park@samsung.com>

Best Regards,
Chanho Park



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2021-12-02 10:30 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-01 19:04 [PATCH 0/6] i2c: exynos5: Add support for modern Exynos SoCs Sam Protsenko
2021-12-01 19:04 ` Sam Protsenko
2021-12-01 19:04 ` [PATCH 1/6] dt-bindings: i2c: exynos5: Convert to dtschema Sam Protsenko
2021-12-01 19:04   ` Sam Protsenko
2021-12-02  8:22   ` Krzysztof Kozlowski
2021-12-02  8:22     ` Krzysztof Kozlowski
2021-12-01 19:04 ` [PATCH 2/6] dt-bindings: i2c: exynos5: Add exynosautov9-hsi2c compatible Sam Protsenko
2021-12-01 19:04   ` Sam Protsenko
2021-12-02  8:23   ` Krzysztof Kozlowski
2021-12-02  8:23     ` Krzysztof Kozlowski
2021-12-01 19:04 ` [PATCH 3/6] dt-bindings: i2c: exynos5: Add bus clock Sam Protsenko
2021-12-01 19:04   ` Sam Protsenko
2021-12-02  8:25   ` Krzysztof Kozlowski
2021-12-02  8:25     ` Krzysztof Kozlowski
2021-12-01 19:04 ` [PATCH 4/6] i2c: exynos5: Add support for ExynosAutoV9 SoC Sam Protsenko
2021-12-01 19:04   ` Sam Protsenko
2021-12-02  8:29   ` Krzysztof Kozlowski
2021-12-02  8:29     ` Krzysztof Kozlowski
2021-12-01 19:04 ` [PATCH 5/6] i2c: exynos5: Add bus clock support Sam Protsenko
2021-12-01 19:04   ` Sam Protsenko
2021-12-02  8:28   ` Krzysztof Kozlowski
2021-12-02  8:28     ` Krzysztof Kozlowski
2021-12-02 10:30   ` Chanho Park [this message]
2021-12-02 10:30     ` Chanho Park
2021-12-01 19:04 ` [PATCH 6/6] i2c: exynos5: Mention Exynos850 and ExynosAutoV9 in Kconfig Sam Protsenko
2021-12-01 19:04   ` Sam Protsenko
2021-12-02  8:29   ` Krzysztof Kozlowski
2021-12-02  8:29     ` Krzysztof Kozlowski
2021-12-02  9:33   ` Chanho Park
2021-12-02  9:33     ` Chanho Park

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='002f01d7e767$96417940$c2c46bc0$@samsung.com' \
    --to=chanho61.park@samsung.com \
    --cc=arnd@arndb.de \
    --cc=devicetree@vger.kernel.org \
    --cc=jaewon02.kim@samsung.com \
    --cc=krzysztof.kozlowski@canonical.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=semen.protsenko@linaro.org \
    --cc=virag.david003@gmail.com \
    --cc=wsa@kernel.org \
    --cc=youngmin.nam@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.