All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Szyprowski <m.szyprowski@samsung.com>
To: Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Chanwoo Choi <cw00.choi@samsung.com>,
	Georgi Djakov <georgi.djakov@linaro.org>,
	inki.dae@samsung.com
Cc: krzk@kernel.org, devicetree@vger.kernel.org, robh+dt@kernel.org,
	a.swigon@samsung.com, myungjoo.ham@samsung.com,
	sw0312.kim@samsung.com, b.zolnierkie@samsung.com,
	linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org,
	linux-samsung-soc@vger.kernel.org,
	dri-devel@lists.freedesktop.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v9 0/5] Exynos: Simple QoS for exynos-bus using interconnect
Date: Fri, 13 Nov 2020 12:00:32 +0100	[thread overview]
Message-ID: <2e4f8aa1-7be0-f65c-da7c-277327626812@samsung.com> (raw)
In-Reply-To: <6687cdd3-6e5b-f3c1-f784-33cc7c0d589a@samsung.com>

Hi Sylwester,

On 13.11.2020 11:32, Sylwester Nawrocki wrote:
> On 13.11.2020 10:07, Chanwoo Choi wrote:
>> On 11/13/20 5:48 PM, Georgi Djakov wrote:
>>> On 11/12/20 16:09, Sylwester Nawrocki wrote:
> [...]
>>> Good work Sylwester! Thank you and all the reviewers! What would be the merge
>>> path for this patchset? Looks like there is no build dependency between patches.
>>> Should i take just patches 2,3 or also patch 1? Chanwoo?
>> Hi Georgi,
>>
>> If you take the patch 2,3, I'll apply patch 1,4 to devfreq.git.
>> Hi Sylwester,
>> First of all, thanks for your work to finish it for a long time.
>> I'm very happy about finishing this work. It is very necessary feature
>> for the QoS. Once again, thank for your work.
> I would also like to thank everyone for provided feedback!
>
> As far as building is concerned the patches could be applied in any
> order. I think we could also apply the drm/exynos patch in same
> merge window. There could be runtime (or git bisect) regression
> only in case when INTERCONNECT is enabled and only (or as first)
> the dts and drm/exynos patches are applied.
>
> Hmm, maybe it's better to hold on with the drm patch, INTERCONNECT
> is disabled in arch/arm/configs/{multi_v7_defconfig, exynos_defconfig}
> but it is enabled in arch/arm64/configs/defconfig.

I don't think we need to delay DRM patch. Exynos DRM mixer is not 
available on ARM64 SoCs, so this won't be an issue.

Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland


WARNING: multiple messages have this Message-ID (diff)
From: Marek Szyprowski <m.szyprowski@samsung.com>
To: Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Chanwoo Choi <cw00.choi@samsung.com>,
	Georgi Djakov <georgi.djakov@linaro.org>,
	inki.dae@samsung.com
Cc: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
	b.zolnierkie@samsung.com, dri-devel@lists.freedesktop.org,
	linux-pm@vger.kernel.org, sw0312.kim@samsung.com,
	a.swigon@samsung.com, robh+dt@kernel.org,
	linux-kernel@vger.kernel.org, myungjoo.ham@samsung.com,
	krzk@kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v9 0/5] Exynos: Simple QoS for exynos-bus using interconnect
Date: Fri, 13 Nov 2020 12:00:32 +0100	[thread overview]
Message-ID: <2e4f8aa1-7be0-f65c-da7c-277327626812@samsung.com> (raw)
In-Reply-To: <6687cdd3-6e5b-f3c1-f784-33cc7c0d589a@samsung.com>

Hi Sylwester,

On 13.11.2020 11:32, Sylwester Nawrocki wrote:
> On 13.11.2020 10:07, Chanwoo Choi wrote:
>> On 11/13/20 5:48 PM, Georgi Djakov wrote:
>>> On 11/12/20 16:09, Sylwester Nawrocki wrote:
> [...]
>>> Good work Sylwester! Thank you and all the reviewers! What would be the merge
>>> path for this patchset? Looks like there is no build dependency between patches.
>>> Should i take just patches 2,3 or also patch 1? Chanwoo?
>> Hi Georgi,
>>
>> If you take the patch 2,3, I'll apply patch 1,4 to devfreq.git.
>> Hi Sylwester,
>> First of all, thanks for your work to finish it for a long time.
>> I'm very happy about finishing this work. It is very necessary feature
>> for the QoS. Once again, thank for your work.
> I would also like to thank everyone for provided feedback!
>
> As far as building is concerned the patches could be applied in any
> order. I think we could also apply the drm/exynos patch in same
> merge window. There could be runtime (or git bisect) regression
> only in case when INTERCONNECT is enabled and only (or as first)
> the dts and drm/exynos patches are applied.
>
> Hmm, maybe it's better to hold on with the drm patch, INTERCONNECT
> is disabled in arch/arm/configs/{multi_v7_defconfig, exynos_defconfig}
> but it is enabled in arch/arm64/configs/defconfig.

I don't think we need to delay DRM patch. Exynos DRM mixer is not 
available on ARM64 SoCs, so this won't be an issue.

Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland


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

WARNING: multiple messages have this Message-ID (diff)
From: Marek Szyprowski <m.szyprowski@samsung.com>
To: Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Chanwoo Choi <cw00.choi@samsung.com>,
	Georgi Djakov <georgi.djakov@linaro.org>,
	inki.dae@samsung.com
Cc: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
	b.zolnierkie@samsung.com, dri-devel@lists.freedesktop.org,
	linux-pm@vger.kernel.org, sw0312.kim@samsung.com,
	a.swigon@samsung.com, robh+dt@kernel.org,
	linux-kernel@vger.kernel.org, myungjoo.ham@samsung.com,
	krzk@kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v9 0/5] Exynos: Simple QoS for exynos-bus using interconnect
Date: Fri, 13 Nov 2020 12:00:32 +0100	[thread overview]
Message-ID: <2e4f8aa1-7be0-f65c-da7c-277327626812@samsung.com> (raw)
In-Reply-To: <6687cdd3-6e5b-f3c1-f784-33cc7c0d589a@samsung.com>

Hi Sylwester,

On 13.11.2020 11:32, Sylwester Nawrocki wrote:
> On 13.11.2020 10:07, Chanwoo Choi wrote:
>> On 11/13/20 5:48 PM, Georgi Djakov wrote:
>>> On 11/12/20 16:09, Sylwester Nawrocki wrote:
> [...]
>>> Good work Sylwester! Thank you and all the reviewers! What would be the merge
>>> path for this patchset? Looks like there is no build dependency between patches.
>>> Should i take just patches 2,3 or also patch 1? Chanwoo?
>> Hi Georgi,
>>
>> If you take the patch 2,3, I'll apply patch 1,4 to devfreq.git.
>> Hi Sylwester,
>> First of all, thanks for your work to finish it for a long time.
>> I'm very happy about finishing this work. It is very necessary feature
>> for the QoS. Once again, thank for your work.
> I would also like to thank everyone for provided feedback!
>
> As far as building is concerned the patches could be applied in any
> order. I think we could also apply the drm/exynos patch in same
> merge window. There could be runtime (or git bisect) regression
> only in case when INTERCONNECT is enabled and only (or as first)
> the dts and drm/exynos patches are applied.
>
> Hmm, maybe it's better to hold on with the drm patch, INTERCONNECT
> is disabled in arch/arm/configs/{multi_v7_defconfig, exynos_defconfig}
> but it is enabled in arch/arm64/configs/defconfig.

I don't think we need to delay DRM patch. Exynos DRM mixer is not 
available on ARM64 SoCs, so this won't be an issue.

Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2020-11-13 11:02 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20201112141041eucas1p1a29130955afd4ec1d5d94cf17183920c@eucas1p1.samsung.com>
2020-11-12 14:09 ` [PATCH v9 0/5] Exynos: Simple QoS for exynos-bus using interconnect Sylwester Nawrocki
2020-11-12 14:09   ` Sylwester Nawrocki
2020-11-12 14:09   ` Sylwester Nawrocki
     [not found]   ` <CGME20201112141047eucas1p1f4294148c61b778bbb321863b8ef0092@eucas1p1.samsung.com>
2020-11-12 14:09     ` [PATCH v9 1/5] dt-bindings: devfreq: Add documentation for the interconnect properties Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
     [not found]   ` <CGME20201112141050eucas1p250f62d09adf38b97c521ef67b4b540f0@eucas1p2.samsung.com>
2020-11-12 14:09     ` [PATCH v9 2/5] interconnect: Add generic interconnect driver for Exynos SoCs Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
     [not found]   ` <CGME20201112141051eucas1p1c94de8fd6032e0a9966cbff9e41c0f2f@eucas1p1.samsung.com>
2020-11-12 14:09     ` [PATCH v9 3/5] MAINTAINERS: Add entry for Samsung interconnect drivers Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
2020-11-12 16:21       ` Krzysztof Kozlowski
2020-11-12 16:21         ` Krzysztof Kozlowski
2020-11-12 16:21         ` Krzysztof Kozlowski
     [not found]   ` <CGME20201112141053eucas1p1acf592a7ece14104a408b77afebb9eed@eucas1p1.samsung.com>
2020-11-12 14:09     ` [PATCH v9 4/5] PM / devfreq: exynos-bus: Add registration of interconnect child device Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
     [not found]   ` <CGME20201112141055eucas1p10bb68467fbb8c481b1275433f5f5f467@eucas1p1.samsung.com>
2020-11-12 14:09     ` [PATCH v9 5/5] drm: exynos: mixer: Add interconnect support Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
2020-11-12 14:09       ` Sylwester Nawrocki
2020-11-13  8:48   ` [PATCH v9 0/5] Exynos: Simple QoS for exynos-bus using interconnect Georgi Djakov
2020-11-13  8:48     ` Georgi Djakov
2020-11-13  9:07     ` Chanwoo Choi
2020-11-13  9:07       ` Chanwoo Choi
2020-11-13  9:07       ` Chanwoo Choi
2020-11-13  9:02       ` Georgi Djakov
2020-11-13  9:02         ` Georgi Djakov
2020-11-13 10:32       ` Sylwester Nawrocki
2020-11-13 10:32         ` Sylwester Nawrocki
2020-11-13 10:32         ` Sylwester Nawrocki
2020-11-13 11:00         ` Marek Szyprowski [this message]
2020-11-13 11:00           ` Marek Szyprowski
2020-11-13 11:00           ` Marek Szyprowski
2020-11-23  6:59   ` Chanwoo Choi
2020-11-23  6:59     ` Chanwoo Choi
2020-11-23  6:59     ` Chanwoo Choi

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=2e4f8aa1-7be0-f65c-da7c-277327626812@samsung.com \
    --to=m.szyprowski@samsung.com \
    --cc=a.swigon@samsung.com \
    --cc=b.zolnierkie@samsung.com \
    --cc=cw00.choi@samsung.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=georgi.djakov@linaro.org \
    --cc=inki.dae@samsung.com \
    --cc=krzk@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=myungjoo.ham@samsung.com \
    --cc=robh+dt@kernel.org \
    --cc=s.nawrocki@samsung.com \
    --cc=sw0312.kim@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.