From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E3DA0C4332F for ; Thu, 2 Nov 2023 16:50:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1347674AbjKBQus (ORCPT ); Thu, 2 Nov 2023 12:50:48 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50678 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229458AbjKBQup (ORCPT ); Thu, 2 Nov 2023 12:50:45 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D19E9197 for ; Thu, 2 Nov 2023 09:50:42 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 423B9C433C7; Thu, 2 Nov 2023 16:50:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1698943842; bh=O65CvmFj/uSkKEy0WcrjZ/zC6Z5PEe3jCXYxUB4rgn8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=WqHOwunEfj3ryjn3rI1GdDY/gYn70gF6ZC03KbwHTXi6N5CK70qiIjUiS2jJmagO+ mGkKrd1UBBPzbM/7W/qtjXC+d862hgPQvOg+hOpy9L0+M9KTbq4dI6tbxoKNyR/YLv e1YhPqRxjWrTRlvcjfWP/TJlfBjI8bXg3RzdD0l1vIQUwr4IoiP8RvFJHp4V9TNmH2 3n13MvxcGb6EXxVqVoehw893dXopCRfTPcbR3Hy8FFufPuHa5lVX0n9hQ9xhc1UuTW Z3FQ2kC25lFLmHdRgZ4bWYBfHTzaB7MUUMVYxZf72qKNB/VjFFwQqIj/MgaddSst3A CmvPvkwQ1NaAQ== Date: Thu, 2 Nov 2023 16:50:38 +0000 From: Mark Brown To: Naresh Solanki Cc: zev@bewilderbeest.net, Liam Girdwood , Patrick Rudolph , linux-kernel@vger.kernel.org Subject: Re: [PATCH] drivers/regulator: Notify sysfs about status changes Message-ID: References: <20231005133059.917577-1-naresh.solanki@9elements.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="Y4NRhtzLRrVeYR/J" Content-Disposition: inline In-Reply-To: X-Cookie: A is for Apple. Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --Y4NRhtzLRrVeYR/J Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Nov 02, 2023 at 09:03:40PM +0530, Naresh Solanki wrote: > On Thu, 2 Nov 2023 at 20:31, Mark Brown wrote: > > That's the opposite sense to what I was thinking of - we're reporting > > voltage changes and enables to userspace rather than just errors. My > > concern here is that this could generate an awful lot of notificaitons > > for normal operation on systems that don't use the uevents, I was > > expecting this to be used for errors. Could you remind me what the use > > case is here, I think I might've got myself confused sorry? > Sorry for confusion caused because I should first described my application > requirements. > Currently my application is interested in know regulator status i.e., > ENABLE, DISABLE or ERROR. > Also events are needed specifically to get them logged like > UNDER_VOLTAGE, OVER_CURRENT, REGULATION_OUT, > OVER_TEMP. Ah, right. Everything except for the enable and disable there looks like it should be OK since they should normally just not happen but the enables and disables might get a bit frequent with runtime PM - not *super* frequent like voltage scaling but enough that people could have an issue with it. Netlink feels like it might be a better fit? Not really looked at the kernel side of implementing that and how sensible that ends up looking. --Y4NRhtzLRrVeYR/J Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmVD010ACgkQJNaLcl1U h9CQvAf+OUh1Bjk8QXBKZsuTgrsqaYL1dI10kSTfA9pcvbW0cY5VlRcvj9xPOKDf zZ4QoBzWadi/awU7rfibV2j2hrOwlmtRVE+FgDQk4rGajTfO06YTbmBvjybW7NUS H64d+uJ5WhwqYkH7xjmb8noqXX7Dv1/v67umU/jxPm9MjkVWdMRCl99WD62/qkW3 KT0B/rTKH6HeJ6fwIgkXV/Y/64V4TxYYdpfCyP4vLusHilhEWcxAUZDFemXKwCWI yS8qtFQ7FCql2URysRGP9cXnGK2qw6oPY26dnqU8RkHqEmzFejoGEm9Sx9FaBik+ k/qT9BdbXCHbS9xdOa8fDjDOfGP9dA== =8nPx -----END PGP SIGNATURE----- --Y4NRhtzLRrVeYR/J--