All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dmitry Osipenko <digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Daniel Lezcano <daniel.lezcano-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: "Thierry Reding"
	<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"Jonathan Hunter"
	<jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	"Peter De Schrijver"
	<pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	"Rafael J. Wysocki" <rjw-LthD3rsA81gm4RdzfppkhA@public.gmane.org>,
	"Michał Mirosław"
	<mirq-linux-CoA6ZxLDdyEEUmgCuDUIdw@public.gmane.org>,
	"Jasper Korten" <jja2000-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"David Heidelberg" <david-W22tF5X+A20@public.gmane.org>,
	"Peter Geis" <pgwipeout-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v9 13/17] cpuidle: tegra: Squash Tegra30 driver into the common driver
Date: Fri, 21 Feb 2020 21:20:32 +0300	[thread overview]
Message-ID: <9de1aad1-1926-db4d-73e3-d10f77bb8a36@gmail.com> (raw)
In-Reply-To: <20200221174128.GW10516-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>

21.02.2020 20:41, Daniel Lezcano пишет:
> On Fri, Feb 21, 2020 at 07:59:14PM +0300, Dmitry Osipenko wrote:
>> 21.02.2020 19:29, Daniel Lezcano пишет:
>>> On Thu, Feb 13, 2020 at 02:51:30AM +0300, Dmitry Osipenko wrote:
>>>> Tegra20 and Terga30 SoCs have common C1 and CC6 idling states and thus
>>>> share the same code paths, there is no point in having separate drivers
>>>> for a similar hardware. This patch merely moves functionality of the old
>>>> driver into the new, although the CC6 state is kept disabled for now since
>>>> old driver had a rudimentary support for this state (allowing to enter
>>>> into CC6 only when secondary CPUs are put offline), while new driver can
>>>> provide a full-featured support. The new feature will be enabled by
>>>> another patch.
>>>>
>>>> Acked-by: Peter De Schrijver <pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>>> Tested-by: Peter Geis <pgwipeout-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>>> Tested-by: Jasper Korten <jja2000-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>>> Tested-by: David Heidelberg <david-W22tF5X+A20@public.gmane.org>
>>>> Signed-off-by: Dmitry Osipenko <digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>>> ---
>>>>  arch/arm/mach-tegra/Makefile          |   3 -
>>>>  arch/arm/mach-tegra/cpuidle-tegra30.c | 123 --------------------------
>>>
>>> Add the -M option when resending please.
>>
>> Okay, thank you very much for taking a look at the patches!
> 
> Yeah, sorry for the delay. Nice cleanup BTW.

No problems, thank you :)

WARNING: multiple messages have this Message-ID (diff)
From: Dmitry Osipenko <digetx@gmail.com>
To: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: "Thierry Reding" <thierry.reding@gmail.com>,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Peter De Schrijver" <pdeschrijver@nvidia.com>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	"Michał Mirosław" <mirq-linux@rere.qmqm.pl>,
	"Jasper Korten" <jja2000@gmail.com>,
	"David Heidelberg" <david@ixit.cz>,
	"Peter Geis" <pgwipeout@gmail.com>,
	linux-pm@vger.kernel.org, linux-tegra@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v9 13/17] cpuidle: tegra: Squash Tegra30 driver into the common driver
Date: Fri, 21 Feb 2020 21:20:32 +0300	[thread overview]
Message-ID: <9de1aad1-1926-db4d-73e3-d10f77bb8a36@gmail.com> (raw)
In-Reply-To: <20200221174128.GW10516@linaro.org>

21.02.2020 20:41, Daniel Lezcano пишет:
> On Fri, Feb 21, 2020 at 07:59:14PM +0300, Dmitry Osipenko wrote:
>> 21.02.2020 19:29, Daniel Lezcano пишет:
>>> On Thu, Feb 13, 2020 at 02:51:30AM +0300, Dmitry Osipenko wrote:
>>>> Tegra20 and Terga30 SoCs have common C1 and CC6 idling states and thus
>>>> share the same code paths, there is no point in having separate drivers
>>>> for a similar hardware. This patch merely moves functionality of the old
>>>> driver into the new, although the CC6 state is kept disabled for now since
>>>> old driver had a rudimentary support for this state (allowing to enter
>>>> into CC6 only when secondary CPUs are put offline), while new driver can
>>>> provide a full-featured support. The new feature will be enabled by
>>>> another patch.
>>>>
>>>> Acked-by: Peter De Schrijver <pdeschrijver@nvidia.com>
>>>> Tested-by: Peter Geis <pgwipeout@gmail.com>
>>>> Tested-by: Jasper Korten <jja2000@gmail.com>
>>>> Tested-by: David Heidelberg <david@ixit.cz>
>>>> Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
>>>> ---
>>>>  arch/arm/mach-tegra/Makefile          |   3 -
>>>>  arch/arm/mach-tegra/cpuidle-tegra30.c | 123 --------------------------
>>>
>>> Add the -M option when resending please.
>>
>> Okay, thank you very much for taking a look at the patches!
> 
> Yeah, sorry for the delay. Nice cleanup BTW.

No problems, thank you :)

  parent reply	other threads:[~2020-02-21 18:20 UTC|newest]

Thread overview: 93+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-12 23:51 [PATCH v9 00/17] Consolidate and improve NVIDIA Tegra CPUIDLE driver(s) Dmitry Osipenko
2020-02-12 23:51 ` Dmitry Osipenko
2020-02-12 23:51 ` [PATCH v9 02/17] ARM: tegra: Add tegra_pm_park_secondary_cpu() Dmitry Osipenko
2020-02-12 23:51 ` [PATCH v9 06/17] ARM: tegra: Expose PM functions required for new cpuidle driver Dmitry Osipenko
     [not found]   ` <20200212235134.12638-7-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 15:18     ` Daniel Lezcano
2020-02-21 15:18       ` Daniel Lezcano
     [not found] ` <20200212235134.12638-1-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-12 23:51   ` [PATCH v9 01/17] ARM: tegra: Compile sleep-tegra20/30.S unconditionally Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
2020-02-12 23:51   ` [PATCH v9 03/17] ARM: tegra: Remove pen-locking from cpuidle-tegra20 Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
2020-02-21 14:58     ` Daniel Lezcano
2020-02-12 23:51   ` [PATCH v9 04/17] ARM: tegra: Change tegra_set_cpu_in_lp2() type to void Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
     [not found]     ` <20200212235134.12638-5-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 15:04       ` Daniel Lezcano
2020-02-21 15:04         ` Daniel Lezcano
2020-02-12 23:51   ` [PATCH v9 05/17] ARM: tegra: Propagate error from tegra_idle_lp2_last() Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
     [not found]     ` <20200212235134.12638-6-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 15:16       ` Daniel Lezcano
2020-02-21 15:16         ` Daniel Lezcano
     [not found]         ` <20200221151612.GJ10516-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2020-02-21 17:21           ` Dmitry Osipenko
2020-02-21 17:21             ` Dmitry Osipenko
     [not found]             ` <1a8c81ab-6f6a-8221-6a4e-c080ba595836-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 17:40               ` Daniel Lezcano
2020-02-21 17:40                 ` Daniel Lezcano
     [not found]                 ` <20200221174033.GV10516-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2020-02-21 18:42                   ` Dmitry Osipenko
2020-02-21 18:42                     ` Dmitry Osipenko
     [not found]                     ` <535bff1a-cb38-a36e-4c8e-1e656618588e-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 19:16                       ` Daniel Lezcano
2020-02-21 19:16                         ` Daniel Lezcano
2020-02-12 23:51   ` [PATCH v9 07/17] ARM: tegra: Rename some of the newly exposed PM functions Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
     [not found]     ` <20200212235134.12638-8-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 15:19       ` Daniel Lezcano
2020-02-21 15:19         ` Daniel Lezcano
2020-02-12 23:51   ` [PATCH v9 10/17] arm: tegra20: cpuidle: Make abort_flag atomic Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
     [not found]     ` <20200212235134.12638-11-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 15:25       ` Daniel Lezcano
2020-02-21 15:25         ` Daniel Lezcano
2020-02-12 23:51   ` [PATCH v9 11/17] arm: tegra20/30: cpuidle: Remove unnecessary memory barrier Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
     [not found]     ` <20200212235134.12638-12-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 15:25       ` Daniel Lezcano
2020-02-21 15:25         ` Daniel Lezcano
2020-02-12 23:51   ` [PATCH v9 13/17] cpuidle: tegra: Squash Tegra30 driver into the common driver Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
2020-02-21 16:29     ` Daniel Lezcano
2020-02-21 16:59       ` Dmitry Osipenko
     [not found]         ` <89a9838c-faf3-b890-cea2-aad53df1eac3-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 17:41           ` Daniel Lezcano
2020-02-21 17:41             ` Daniel Lezcano
     [not found]             ` <20200221174128.GW10516-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2020-02-21 18:20               ` Dmitry Osipenko [this message]
2020-02-21 18:20                 ` Dmitry Osipenko
2020-02-12 23:51   ` [PATCH v9 16/17] ARM: multi_v7_defconfig: Enable Tegra cpuidle driver Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
     [not found]     ` <20200212235134.12638-17-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 16:30       ` Daniel Lezcano
2020-02-21 16:30         ` Daniel Lezcano
2020-02-12 23:51   ` [PATCH v9 17/17] ARM: tegra: Enable Tegra cpuidle driver in tegra_defconfig Dmitry Osipenko
2020-02-12 23:51     ` Dmitry Osipenko
2020-02-21 16:31     ` Daniel Lezcano
2020-02-13  0:38   ` [PATCH v9 00/17] Consolidate and improve NVIDIA Tegra CPUIDLE driver(s) Dmitry Osipenko
2020-02-13  0:38     ` Dmitry Osipenko
2020-02-18 14:56   ` Dmitry Osipenko
2020-02-18 14:56     ` Dmitry Osipenko
2020-02-12 23:51 ` [PATCH v9 08/17] ARM: tegra: Make outer_disable() open-coded Dmitry Osipenko
2020-02-12 23:51 ` [PATCH v9 09/17] arm: tegra20: cpuidle: Handle case where secondary CPU hangs on entering LP2 Dmitry Osipenko
     [not found]   ` <20200212235134.12638-10-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 14:55     ` Daniel Lezcano
2020-02-21 14:55       ` Daniel Lezcano
2020-02-21 15:43     ` Daniel Lezcano
2020-02-21 15:43       ` Daniel Lezcano
     [not found]       ` <20200221154318.GO10516-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2020-02-21 16:56         ` Dmitry Osipenko
2020-02-21 16:56           ` Dmitry Osipenko
     [not found]           ` <239a2b66-8da8-2e6c-d19d-9ed207ad0a64-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 17:36             ` Daniel Lezcano
2020-02-21 17:36               ` Daniel Lezcano
     [not found]               ` <20200221173649.GU10516-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2020-02-21 18:19                 ` Dmitry Osipenko
2020-02-21 18:19                   ` Dmitry Osipenko
     [not found]                   ` <b51f3f6b-8287-5ce8-fcaa-77cbab507618-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 20:02                     ` Daniel Lezcano
2020-02-21 20:02                       ` Daniel Lezcano
     [not found]                       ` <f27481cf-ca5e-df47-932b-fcb4713f0d78-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2020-02-21 20:21                         ` Dmitry Osipenko
2020-02-21 20:21                           ` Dmitry Osipenko
     [not found]                           ` <50a8fb7c-f497-2234-c0b0-560aec1c5691-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 20:42                             ` Dmitry Osipenko
2020-02-21 20:42                               ` Dmitry Osipenko
2020-02-21 20:48                             ` Daniel Lezcano
2020-02-21 20:48                               ` Daniel Lezcano
     [not found]                               ` <21e3cc35-cc6b-5452-da93-bdaac43716c5-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2020-02-21 20:54                                 ` Dmitry Osipenko
2020-02-21 20:54                                   ` Dmitry Osipenko
     [not found]                                   ` <c13aa8f9-092b-55ca-742e-17db0184649b-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 21:11                                     ` Daniel Lezcano
2020-02-21 21:11                                       ` Daniel Lezcano
     [not found]                                       ` <f27e7974-f102-f9dc-6b48-9814b88465bf-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2020-02-24 15:12                                         ` Dmitry Osipenko
2020-02-24 15:12                                           ` Dmitry Osipenko
2020-02-12 23:51 ` [PATCH v9 12/17] cpuidle: Refactor and move out NVIDIA Tegra20 driver into drivers/cpuidle Dmitry Osipenko
     [not found]   ` <20200212235134.12638-13-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 15:44     ` Daniel Lezcano
2020-02-21 15:44       ` Daniel Lezcano
2020-02-12 23:51 ` [PATCH v9 14/17] cpuidle: tegra: Squash Tegra114 driver into the common driver Dmitry Osipenko
2020-02-12 23:51 ` [PATCH v9 15/17] cpuidle: tegra: Disable CC6 state if LP2 unavailable Dmitry Osipenko
     [not found]   ` <20200212235134.12638-16-digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-02-21 16:39     ` Daniel Lezcano
2020-02-21 16:39       ` Daniel Lezcano
2020-02-24 16:44 ` [PATCH v9 00/17] Consolidate and improve NVIDIA Tegra CPUIDLE driver(s) Dmitry Osipenko
  -- strict thread matches above, loose matches on Subject: below --
2019-12-18 21:04 Dmitry Osipenko
2019-12-18 21:04 ` [PATCH v9 13/17] cpuidle: tegra: Squash Tegra30 driver into the common driver Dmitry Osipenko

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=9de1aad1-1926-db4d-73e3-d10f77bb8a36@gmail.com \
    --to=digetx-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=daniel.lezcano-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=david-W22tF5X+A20@public.gmane.org \
    --cc=jja2000-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mirq-linux-CoA6ZxLDdyEEUmgCuDUIdw@public.gmane.org \
    --cc=pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=pgwipeout-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=rjw-LthD3rsA81gm4RdzfppkhA@public.gmane.org \
    --cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    /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.