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;
IronPort-SDR: O59kB8H475LLwzRSjHEMXpDrHBTa7OONWVOg6XyDxF0ugmE0Q0K5ak+xqEhlAyY73qlKdT6+dc
 CLLfkubxWhlA==
IronPort-SDR: uIcVsZKpC1T/XZABeBlT37G65ne6km5mDT/3irZE34DT8lWnPGk+KZtJ6UPqd9vR1kLZns5THn
 JSuZkfIIiucg==
Message-ID: <4c8abeb4ddff2355d9ba397f50b7844a7734c2e9.camel@linux.intel.com>
Subject: Re: [PATCH v2 4/5] thermal: core: genetlink support for
 events/cmd/sampling
From:   Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
To:     Zhang Rui <rui.zhang@intel.com>,
        Daniel Lezcano <daniel.lezcano@linaro.org>
Cc:     rkumbako@codeaurora.org, amit.kucheria@linaro.org,
        linux-kernel@vger.kernel.org
Date:   Wed, 01 Jul 2020 08:49:35 -0700
In-Reply-To: <171936e84c416d7647756d9b453ef2d4475ebdc8.camel@intel.com>
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>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.34.3 (3.34.3-1.fc31) 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


[...]

> 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.
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

Thanks,
Srinivas
> > 
> > 
> >