Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8;
Date:   Fri, 20 Oct 2023 10:20:57 -0400
From:   Frank Li <Frank.li@nxp.com>
To:     Miquel Raynal <miquel.raynal@bootlin.com>
Cc:     "Zbigniew, Lukwinski" <zbigniew.lukwinski@linux.intel.com>,
        alexandre.belloni@bootlin.com, conor.culhane@silvaco.com,
        imx@lists.linux.dev, joe@perches.com,
        linux-i3c@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 1/6] i3c: master: add enable(disable) hot join in sys
 entry
Message-ID: <ZTKMyW939rB6sSKq@lizhi-Precision-Tower-5810>
References: <20231018205929.3435110-1-Frank.Li@nxp.com>
 <20231018205929.3435110-2-Frank.Li@nxp.com>
 <57efe6de-cf65-32ed-fd1e-a388e1676c70@linux.intel.com>
 <20231020154528.2f5f68f5@xps-13>
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20231020154528.2f5f68f5@xps-13>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?iso-8859-1?Q?yVkOVRHy+/2Sum8oGpcgkYuortQ7JaCXegiwtbdVKb/A1ZK3WET/yFvQDr?=
 =?iso-8859-1?Q?EiHLb1SiHKIdc71YfeEJtCpyXfc6980Wc6aQ9rZZzCKKkN057tM4NDYIQM?=
 =?iso-8859-1?Q?68RRc2X+9MWizTtglV/+Wc61F23/9r0v30tgtGEeeWIMihM5gq4lgqoSPI?=
 =?iso-8859-1?Q?oetmiCVrn+ewJQZrmAj2WOGiIlWcnRGcWHO5Co3NTaQf4sZLOeNyMoEFWz?=
 =?iso-8859-1?Q?bwD4clhFASjMnPDE6yMTv4ubDQRziH5QCinAS/br8AkNej21TK97ixeEzw?=
 =?iso-8859-1?Q?RNyPtoN56y1fKvMilSACkXUbSzhrxVN5BV8W9L+ouO5aDYjrPaTQ/oTD5I?=
 =?iso-8859-1?Q?LA7kI3S+kWNgSRiXPRoBoG6/bMYwUl2t8Bex11pwMuu9qSeTaqEIHMjo0N?=
 =?iso-8859-1?Q?WUTF/MA9wH5UPM/DXdf4nSrzGPctIfWOcJTa1b/qEbD1Cw82w6w4V9xU84?=
 =?iso-8859-1?Q?7Klq0LukrYk1bqxV0n9n40dNE8uTd62lRhG1HDEv5TFA2Rr/dltzhrrZk+?=
 =?iso-8859-1?Q?AX3qF616AX0KkUvqt1aVHSlAIySdx4y1OlJv5jkge9t7GqGH3tB50NoxUm?=
 =?iso-8859-1?Q?oJBPaO1KOwwXD+Y/EzZcCqb9Dq1M/3MU9/l1eDY1sB01dZR+u+WBMVN7hU?=
 =?iso-8859-1?Q?7KD/6UCYZkVmjoDbMS1SpoVkUR6NQY389iqwHhC2Mx3HaKMB9gD0WpjBFJ?=
 =?iso-8859-1?Q?2AMsA8LAfAXMFpL+BAK7EdRTUWhCk0P2CPos9KgtBi9VV24byXclnx9oMZ?=
 =?iso-8859-1?Q?KahRe+T8h5CJnSClARbr1MileBf5uUGiw1i1mHfG+2AF1sKlJnW8ldy950?=
 =?iso-8859-1?Q?I4WPR3FE96Oisy1hzBgUcDHXutFaDlMBedihy/lTBJ04/m/GMxuNI8WsU+?=
 =?iso-8859-1?Q?QKWRrLR+1M14HRFxAdUGTSyA4NIjoKYydAF3kiHCTKbLO6VFZ33QFwEX7+?=
 =?iso-8859-1?Q?r8yfCKT4geR9eWTUrEx7yQXCBTFgETBOmDM2ri4fvhxvKH6ZbS0DxK2m41?=
 =?iso-8859-1?Q?qrRYhGHqM1ppAMTplHg4c2eRkjmcymGP0KUON3kLHW8qw0mKwji3ct5KcM?=
 =?iso-8859-1?Q?HUJ1Twow1SYGMoiyg6Ldf0+KeNte6PhG4vWolBseXELloa4Kx0Ax4zCcMj?=
 =?iso-8859-1?Q?t2xD6eBic1RHnhwRlYEPFT+rI00twzMuTYEmVLoH8shY/SIRLHkYpx/qcZ?=
 =?iso-8859-1?Q?Pn4W8mTbJgX7zDdP62Sc3adyfxUjRIefb375SJzaBnmzlcJz0KPFyuDB8P?=
 =?iso-8859-1?Q?+a2zkASlIxppN6yG3aceDxNRipgs/hVAd1Fs3qd6v4t4pZX/RMXp6vHUzK?=
 =?iso-8859-1?Q?XgFba87BtaOZSP9JlD91b2dKNJ0UlB6JrenGfIPReh2+nRBsIUO8HiekcO?=
 =?iso-8859-1?Q?v9VW6WsMAutvGoPKq+fHenjEU2kj6k4s+JA+A2hrARX4bOQMpL67ri0oxS?=
 =?iso-8859-1?Q?UyBFqUPlCmr6RQRyX4gbNjlOpzlQvgnW7buWI+CcQtMFWFe6nPDc8RGU7U?=
 =?iso-8859-1?Q?Z9nfYAjS6I8y6IGSGO4LECZy+T74oB7J34VQY1XIcRmevwbw2AIFQ9r+Hn?=
 =?iso-8859-1?Q?l3tvWW+Ml+w4I/1NbP69o/OjQw/o8jmvhsBywdLwr2PYnW/nTc8oXh45Ts?=
 =?iso-8859-1?Q?bMyv1GDnPYf4i+ee6Pb4Pyos0Ndiz+Hesx?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0559ed9d-6a96-42c0-c349-08dbd177cadc
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB4838.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Oct 2023 14:21:04.0188
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: /SCxUJzzXPYp/5XfN8ZbUDgFlTuTZ4Sl9iJOUFnKrAJ2TEtvWDWPmtp0LALt7yu5mgpTwDRWBAn7hdgTW8OGsg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB7619
X-Spam-Status: No, score=-0.6 required=5.0 tests=DKIM_INVALID,DKIM_SIGNED,
	HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,
	SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on fry.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (fry.vger.email [0.0.0.0]); Fri, 20 Oct 2023 07:21:21 -0700 (PDT)

On Fri, Oct 20, 2023 at 03:45:28PM +0200, Miquel Raynal wrote:
> Hi Lukwinski,
> 
> zbigniew.lukwinski@linux.intel.com wrote on Fri, 20 Oct 2023 10:55:27
> +0200:
> 
> > On 10/18/2023 10:59 PM, Frank Li wrote:
> > > Add hotjoin entry in sys file system allow user enable/disable hotjoin
> > > feature.
> > >
> > > Add (*enable(disable)_hotjoin)() to i3c_master_controller_ops.
> > > Add api i3c_master_enable(disable)_hotjoin();  
> > 
> > What is the use case for having HJ enable knob in sysfs available for user space other than for debug stuff? In other words, does user space really need to enable/disable HJ in runtime for other reason but debug? If it is only for debug maybe it? could be move to debugFS?
> 
> I don't think hotjoin should be considered as a debug feature. The
> problem here is the power consumption which is higher if you enable
> this feature (you need to keep everything clocked and ready to handle
> an IBI) whereas if your design is "fixed" (more like an I2C bus) you
> may save power by disabling this feature.
> 
> A module parameter does not fit here because it's a per-bus
> configuration.

I agree. sys entry is more flexiable. and let controller choose better
power saving policy for difference user case.

Frank

> 
> Thanks,
> Miqu?l