From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="N8w96T/q" Received: from mail-vs1-xe30.google.com (mail-vs1-xe30.google.com [IPv6:2607:f8b0:4864:20::e30]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CDCCCD6 for ; Mon, 4 Dec 2023 00:20:48 -0800 (PST) Received: by mail-vs1-xe30.google.com with SMTP id ada2fe7eead31-464467c2568so1123322137.3 for ; Mon, 04 Dec 2023 00:20:48 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1701678048; x=1702282848; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=28cyFAoKbjfR/kdzsJO4rKA0W8Gwei+uNVU1U+mLnOY=; b=N8w96T/qdSKRauf4IQYWf51yYd7AUPub2Xz3wbmAkfYCAHdPYC3QnCSxZHrYt7B78L B8DnsnFqZQbN+pMBV4SBogerVfO55Oi/ZNQE5tu2SZwfdnZF0TKN5Fqt5uWoRVxx/gC4 YCp9FfEQBuHqBW8/MeMuJdCOyD/G0GlqyDbH8vEkLLUda+I+ijlCcRaAgUskUzvmri9b 4F6z4HE3e5+sCVPV6XyAcv8tmfwGwNWAEmtMUi8Ts5QIl2PYx0MTzv8hgmTP/ARaPCIR miekL2a9cebjExQN6GoYqn7Hobxzf/97mdNxr38L5YpLByG4RG5zVHn2+V0X4O5+j10k SRzw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1701678048; x=1702282848; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=28cyFAoKbjfR/kdzsJO4rKA0W8Gwei+uNVU1U+mLnOY=; b=POjVypCbK0c4ytaQLubstYpTb/h++2jI8N5JXMPMgjc9c09rp6vZX2fHaoYEuvyKXi yp2NtvUMqvHXNYP00S2xCPeTp0athQOQwR0z6otirf3g5fc9Q/qlJsllL/XQoNNjo/4/ nGFZj7LzGP/MiU3yZmVAAa8ow14lBLDrggaitr+eKLa7/kKz/FaKxwz1Bl3LScEM1OML 7wY6PZv/R6zglMpj42iXrxAYtngNk+sUFKx71KNWPKgnDmjfzrXEkBXIr0aoomybGNa5 Xk1xJQCwsuQDq686xzKYcDjoAU+uwELIXsKs9C7wSEWYCCsyb89p55kI1QiobBJdAVvB rxqw== X-Gm-Message-State: AOJu0YySzF4hGlDLBpBflI64z0OfJmD8cMmvYN7oW9c/HXXLd5CAIAxV 9XhqHR2DrW39ce0/NnjsJH7n9sIjoeokduaj3Wa4Iw== X-Google-Smtp-Source: AGHT+IFN6y9qVR68yPdmtEVkrffiZvx/PZkLrVxoEmSzgloE8lDnAEu7zQvU5ulZRB2pdnQGBoz5tKZCHvsO1wQm0SY= X-Received: by 2002:a05:6102:34e2:b0:462:c2e9:6dda with SMTP id bi2-20020a05610234e200b00462c2e96ddamr1269374vsb.31.1701678047986; Mon, 04 Dec 2023 00:20:47 -0800 (PST) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <6384831c05b8ceeaf4a16cf9229770252989b762.camel@gmail.com> <971eb35068639ec404669ea5320c8183ea71a7d0.camel@gmail.com> <61a8f54835c10db7a9c650ee2e3706b47382c634.camel@gmail.com> <7dc3f137-6073-4262-afb5-439d024bbbd2@roeck-us.net> <986fb7dc2a34602fa9c2d57a7a3e06a71cfdc0a0.camel@gmail.com> <66454ca2-d5cb-4701-a237-03b3991a791f@roeck-us.net> In-Reply-To: From: Bartosz Golaszewski Date: Mon, 4 Dec 2023 09:20:37 +0100 Message-ID: Subject: Re: [PATCH v2 2/2] hwmon: ltc4282: add support for the LTC4282 chip To: Linus Walleij Cc: =?UTF-8?B?TnVubyBTw6E=?= , Guenter Roeck , Andy Shevchenko , nuno.sa@analog.com, linux-hwmon@vger.kernel.org, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, Jean Delvare , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jonathan Corbet Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, Dec 4, 2023 at 12:09=E2=80=AFAM Linus Walleij wrote: > > On Sat, Dec 2, 2023 at 10:42=E2=80=AFAM Nuno S=C3=A1 wrote: > > Hopefully, I'll get into the gpio stuff later on. From a brief look, > the auxiliary > > bus might feet and easier than mfd. > > You can also just spawn a random platform_device from the HWMON driver > and let that probe a driver down in drivers/gpio/*. > Please don't. A "random platform_device" even reads like a bad idea. Conceptually the GPIO part is not a child but one of the modules. It should be an MFD device IMO. Bart > static struct platform_device my_gpio_device =3D { > .name =3D "my-gpio", > .id =3D -1, > }; > > my_gpio_device.dev.platform_data =3D ... ; > my_gpio_device.dev.parent =3D dev; > return platform_device_register(&my_gpio_device); > > You can then pass any accessors as platform data. This in a way is what > the MFD or aux buses do just more organized. > > Yours > Linus Walleij