Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 5 Nov 2021 09:46:27 +0900
From:   Youngmin Nam <youngmin.nam@samsung.com>
To:     Mark Rutland <mark.rutland@arm.com>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>,
        will@kernel.org, daniel.lezcano@linaro.org, tglx@linutronix.de,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-samsung-soc@vger.kernel.org, pullip.cho@samsung.com,
        hoony.yu@samsung.com, hajun.sung@samsung.com,
        myung-su.cha@samsung.com, kgene@kernel.org, kwoo.kang@samsung.com
Subject: Re: [PATCH v2 1/2] clocksource/drivers/exynos_mct_v2: introduce
 Exynos MCT version 2 driver for next Exynos SoC
Message-ID: <20211105004627.GA24234@perf>
MIME-Version: 1.0
In-Reply-To: <20211104094453.GA58798@C02TD0UTHF1T.local>
User-Agent: Mutt/1.9.4 (2018-02-28)
Content-Type: multipart/mixed;
        boundary="----TI73IYgzdP03rYJ9SiN0qM3o3mlayfAqXHuFFMbUzbY.6Ea-=_c65ff_"
CMS-TYPE: 102P
DLP-Filter: Pass
References: <20211102001122.27516-2-youngmin.nam@samsung.com>
        <20211102102802.GA16545@C02TD0UTHF1T.local> <20211103000945.GA48132@perf>
        <83f6574b-bbab-f0c3-7198-f773c3dcfc63@canonical.com>
        <20211103092444.GA7013@perf>
        <f7871718-f244-8a9e-9302-c27bdb2d3cc1@canonical.com>
        <20211103095727.GA19093@perf> <20211103100407.GA35817@C02TD0UTHF1T.local>
        <20211104002102.GA29618@perf> <20211104094453.GA58798@C02TD0UTHF1T.local>
        <CGME20211105001912epcas2p30469b097f7c348f0a59251d03cd0075e@epcas2p3.samsung.com>
Precedence: bulk

------TI73IYgzdP03rYJ9SiN0qM3o3mlayfAqXHuFFMbUzbY.6Ea-=_c65ff_
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline

On Thu, Nov 04, 2021 at 09:44:53AM +0000, Mark Rutland wrote:
> Hi,
> 
> On Thu, Nov 04, 2021 at 09:21:02AM +0900, Youngmin Nam wrote:
> > On Wed, Nov 03, 2021 at 10:04:18AM +0000, Mark Rutland wrote:
> > > On Wed, Nov 03, 2021 at 06:57:28PM +0900, Youngmin Nam wrote:
> > > > On Wed, Nov 03, 2021 at 10:04:36AM +0100, Krzysztof Kozlowski wrote:
> > > > > On 03/11/2021 10:24, Youngmin Nam wrote:
> > > > > > On Wed, Nov 03, 2021 at 09:18:07AM +0100, Krzysztof Kozlowski wrote:
> > > > > >> On 03/11/2021 01:09, Youngmin Nam wrote:
> > > > > >>> On Tue, Nov 02, 2021 at 10:28:10AM +0000, Mark Rutland wrote:
> 
> > > > > >>>> Previously Will asked you to try CLOCK_EVT_FEAT_PERCPU here, and to set
> > > > > >>>> the C3STOP flag on the arch timer via the DT when necessary, rather than
> > > > > >>>> trying to override the arch timer like this:
> 
> > > > > >>> Yes, I believe Will's suggestion definitely will work.
> 
> > > > So, how about we control timer rating value with DT ?
> > > > Of course the default rating value should be lower than arm arch timer's.
> > > > Do you agree with this?
> > > 
> > > No; placing a rating value in the DT is a hack. That should *not* live
> > > in the DT because it's linux-internal detail and not a description of
> > > the HW.
> > 
> > So, how do we use MCTv2 only for clock event device if there are some
> > limitations caused by SoC design implemention ?
> 
> What limitations? Are you thinking of a known issue, or just in case
> there is a bug in future HW?
> 
> If there is a problem, we'll need to describe that in the DT somehow,
> and we need to know speciifcally what that limitation is.
> 
> Above you said that Will's suggestion will definitely work, which
> implies no such limitations.
> 

Using arch timer for event device is highly related with Core power down feature so that it is also related with
power saving scheme in SoC.
Core power down and power saving depend on SoC design implemention.
We can't confirm that using only arch timer can cover all scenario at production level.
So we should be able to use MCTv2 as well.

Why do you enforce using *only* arch timer ?
Why aren't we allowed to use own timer of our SoC ?

What I meant that replied to Will was
I think this will work generally, but we can't confirm this will cover all cases.

> Thanks,
> Mark.
> 

------TI73IYgzdP03rYJ9SiN0qM3o3mlayfAqXHuFFMbUzbY.6Ea-=_c65ff_
Content-Type: text/plain; charset="utf-8"


------TI73IYgzdP03rYJ9SiN0qM3o3mlayfAqXHuFFMbUzbY.6Ea-=_c65ff_--