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;
Subject: Re: [PATCH v2 4/5] thermal: core: genetlink support for
 events/cmd/sampling
To:     Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>,
        Zhang Rui <rui.zhang@intel.com>
Cc:     rkumbako@codeaurora.org, amit.kucheria@linaro.org,
        linux-kernel@vger.kernel.org
References: <20200625144509.17918-1-daniel.lezcano@linaro.org>
 <20200625144509.17918-4-daniel.lezcano@linaro.org>
 <73150ef4ca536368f087672b917dd9773417020e.camel@intel.com>
 <23458be6-1e5f-7397-4935-c5268b9f52a6@linaro.org>
 <171936e84c416d7647756d9b453ef2d4475ebdc8.camel@intel.com>
 <4c8abeb4ddff2355d9ba397f50b7844a7734c2e9.camel@linux.intel.com>
From:   Daniel Lezcano <daniel.lezcano@linaro.org>
Message-ID: <8526622b-cb31-4d40-95b6-e209b3101563@linaro.org>
Date:   Wed, 1 Jul 2020 18:31:29 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.8.0
MIME-Version: 1.0
In-Reply-To: <4c8abeb4ddff2355d9ba397f50b7844a7734c2e9.camel@linux.intel.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 01/07/2020 17:49, Srinivas Pandruvada wrote:
> 
> [...]
> 
>> Then I'd prefer we only introduce the events that are used or will be
>> used soon, like the tz disable/enable, to avoid some potential dead
>> code.
>> We can easily add more events when they are needed.
>>
>> Srinivas, do you have plan to use the trip add/delete events?
> Yes and also trip modify.

Ok I will keep those then.

> Also I need to have one more event for heartbeat like event which needs
> confirmation from user space to hardware the user process controlling
> thermal is active not dead. So whenever hardware wants to check health 
> it will send an event, which user space should acknowledge

Could it be the opposite? The userspace sends periodically a message to
tell it is alive instead of having the kernel asking the userspace?

Is it ok if we add this in a separate series ?


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog