From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f54.google.com (mail-ej1-f54.google.com [209.85.218.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 89E43415F24 for ; Mon, 24 Aug 2026 12:22:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787574156; cv=none; b=kSvkRWJypUN6w7yLcQSYeaNFwfQT8MuAl5h8fCZXJzW3j75Vtz+OAHTsbtqz+X9ssOWnSJ5ssT7gV3g1Ncb7SuHoA1FLJl4nssM4ZHTwsWVE7PI4DfzyDLxYDCPKx19nhumILhc1mmBJjGwRF6HVpk93aX1akPaIbhReYVt6jEg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787574156; c=relaxed/simple; bh=t62Zykvs+8NdpMy0PpJs95Iv6bI1meByLMw4pOoPMnI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=IjG8ABlg0Do+eWpUJoQPmb9LGiU3eNs6XQBYHikMZ2sgi7ka64dvuWk18tKGMKarzI2rIri8hvLKB3mK7TpVDQCXmDZoQweCXWtVl4NkE1OcdHm5bARYY/yKhGlQp6NTsIRCzdFW+JlOMKWm6HBPukyH1Jt2SXijP35Zwh72eew= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b=pQFt21Ik; arc=none smtp.client-ip=209.85.218.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b="pQFt21Ik" Received: by mail-ej1-f54.google.com with SMTP id a640c23a62f3a-c15d111ca99so403392566b.0 for ; Mon, 24 Aug 2026 05:22:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre.com; s=google; t=1787574151; x=1788178951; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=MHbWNJk5cOn3r4GzbI0kltGoUU2l7AC2RyCAmVHKUJU=; b=pQFt21IkwjmiiRhJbVs9vq5sUPKqNA/BiBzX2+4R8CA5wNXT5sFAXkGqI30NCwHa0Q IU7yZd09xRt4vG9OjoNqeimz8I1ZTi3c7ZmE+m0ssjKiAgWzUX1HjtxhfEREDRMXM3B4 dCehuya0jS5YZgk+kgPbMHbp+Fm86obxRW1ww8YdtAYxNdSURRrUdHlMMRkFGNvEktAB Tptvb6uR6imm0sNWpu52XYIZaozPAgTijXUZrq92GqmFBpmZEB1a6qLg7Y+JA+j8/u5s qzoBtwmKbGWsFFaH3jl0tNWJK/aCUDORjew3pLnZPEWDWqTL3Jf1Gu6+umBE2MVV8eHr /18Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787574151; x=1788178951; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=MHbWNJk5cOn3r4GzbI0kltGoUU2l7AC2RyCAmVHKUJU=; b=U7DozeAkEObmrYgevpRA2p2Bw8vTnpz/YpRNhUlBPg5mquIc2gq8nmu8i4C96zew43 fzbsEtlM+QIv0DAEDN/yXF8sm45KwmnyEr9r51X12+Tqsfv+k2WWUB+QcPXR2b+RpI5+ k3cV4pXWELfjIfpnNo9ljJ7Vlf7B/c0BwLofmYoul9R6B9OwSyIWo7+JWfYSEzvOFSJA QtOD3mGyWU2nzr2pwvJCTK2z5Joe7Gl4/bSCTYvwhGcAnNW+Xy1/dMmipjVp/IWEuxRF wP47VrdsXZ3sPVDV0ify34TBRMdspk+OtlYeEfwviuJ4JrWnv+RYLbXo0R0M6O6y1nhu caQA== X-Forwarded-Encrypted: i=1; AHgh+RpAs456joke5a8jbeUNMUzx2SSBiRPUIXiYwqWNDi00fK+5IkN90yqgU64fzmPJFVygSTqNxZlonroh6oA=@vger.kernel.org X-Gm-Message-State: AFuF++lEBpK+jyvUT9u9IyvbGlFgaKTWHJxwolu3uuiuzCWSgGur2mrK oSwCxf/rpBPF0f97sM6RFEoD+WkUAhOfSGZcxZ7LjJmZa1DDP5MKC/COLmDDGqvwqYE= X-Gm-Gg: AR+sD12Mjd/VxmXJ3+jjCUue3x7kO8L10txWGQILdLHLfzTRyNzpN7Mo06tvQPRt1kx 33N77W4K31QfYzuopEgsmWV+g8nXIbN3lhe98xNWThqZft1fTSUb2/aM8ZwGqnGyPEX9gF2p9go xrT31k1mZqV9Mr1TsDiPCWdX8Opc+WIimvjvq2EhwN0oQr6/ibbD1BexXQt2DVfNvMYCcy9+/EI k+E3wCKEeFrd84WBbjuJdrmA8rtQT39zemes2QrdhFNKaFfPbRJnmTW2VtiYH1A8ul8Q+nKbrKM iErfOdNs1iw+uK8EN0DKefzWKrhbv0K0QaI+K0OBabE7xhwdSzGxjFLDrXvKaAJpCfRf3pM5SeT Rb0yF5dmsxMez+GtMFsh7q173WUiYN4J668JEXW13hVI4MIHd6e36Y6qBxMRzwlNeoDgv7zsGVf DXoj8OqzRFRd4/lJFG/2/GwEgtVakqMQD/PNcky9P2dUUvEc63T/3eCWrUeaE3DOapS6boWy+D4 axiC4Y2Mxt7qP3KPuXkFfvdIf8BLp4vM6R84ySWZWZovqYDNVb5 X-Received: by 2002:a17:907:845:b0:c20:fed2:898b with SMTP id a640c23a62f3a-c2492782dbemr1724856766b.23.1787574151309; Mon, 24 Aug 2026 05:22:31 -0700 (PDT) Received: from localhost (p200300f65f19a904efa8a94b704adaa2.dip0.t-ipconnect.de. [2003:f6:5f19:a904:efa8:a94b:704a:daa2]) by smtp.gmail.com with UTF8SMTPSA id a640c23a62f3a-c24962a3c13sm1308598466b.26.2026.08.24.05.22.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 24 Aug 2026 05:22:30 -0700 (PDT) Date: Mon, 24 Aug 2026 14:22:29 +0200 From: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= To: Hans de Goede Cc: Bjorn Andersson , Cristian Marussi , Sudeep Holla , Daniel Lezcano , arm-scmi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, imx@lists.linux.dev, linux-kernel@vger.kernel.org, Bjorn Andersson Subject: Re: [PATCH v3 1/3] module: add SCMI device table alias support Message-ID: References: <20260820194850.459739-1-johannes.goede@oss.qualcomm.com> <20260820194850.459739-2-johannes.goede@oss.qualcomm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="2tfcczky2nyuuvxm" Content-Disposition: inline In-Reply-To: <20260820194850.459739-2-johannes.goede@oss.qualcomm.com> --2tfcczky2nyuuvxm Content-Type: text/plain; protected-headers=v1; charset=us-ascii Content-Disposition: inline Subject: Re: [PATCH v3 1/3] module: add SCMI device table alias support MIME-Version: 1.0 Hello, I just replied to v2 and only then found out there is a v3 already. There is something wrong with my mail filtering, sorry. I'm forwarding my review comments to here (the part that still applies). On Thu, Aug 20, 2026 at 09:48:48PM +0200, Hans de Goede wrote: > diff --git a/include/linux/device-id/scmi.h b/include/linux/device-id/scmi.h > new file mode 100644 > index 000000000000..4150481524d4 > --- /dev/null > +++ b/include/linux/device-id/scmi.h > @@ -0,0 +1,19 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +#ifndef LINUX_DEVICE_ID_SCMI_H > +#define LINUX_DEVICE_ID_SCMI_H > + > +#ifdef __KERNEL__ > +#include > +typedef unsigned long kernel_ulong_t; > +#endif > + > +#define SCMI_NAME_SIZE 32 > +#define SCMI_MODULE_PREFIX "scmi:" > + > +struct scmi_device_id { > + __u8 protocol_id; > + char name[SCMI_NAME_SIZE]; > + kernel_ulong_t driver_data; Please use union { kernel_ulong_t driver_data; const void *driver_data_ptr; }; here instead of the plain driver_data member (or don't add driver_data at all if not needed). This yields better type-safety, see e.g. https://lore.kernel.org/all/cover.1780048925.git.u.kleine-koenig@baylibre.com for an extended explanation. > +}; > + > +#endif /* ifndef LINUX_DEVICE_ID_SCMI_H */ > diff --git a/include/linux/mod_devicetable.h b/include/linux/mod_devicetable.h > index a397213bedac..d241421c36aa 100644 > --- a/include/linux/mod_devicetable.h > +++ b/include/linux/mod_devicetable.h > @@ -49,6 +49,7 @@ > #include "device-id/pnp.h" > #include "device-id/rio.h" > #include "device-id/rpmsg.h" > +#include "device-id/scmi.h" Please don't add new includes to mod_devicetable.h. That header is ugly and each include makes it worse. Just add to the modpost source. > #include "device-id/sdio.h" > #include "device-id/sdw.h" > #include "device-id/serio.h" Thanks Uwe --2tfcczky2nyuuvxm Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmqMN4MACgkQj4D7WH0S /k4yMwf+N+KJ5sPPSlOgLDa7K8R0+3z3wNZSXpKG1mfo73lwlNuUQ0o9mdIVnCSh HjOljqygnPT1KDnD0tyk4aBt8VyDQ7dC93v2tUbPE2EFoHWT9BR9lAcDJY3TqVqz tjL9DZtz1eAcw4OY5tm7FW6fBRLJTlGAgyzkcgBHxrB/6dIPZ2BaHa+eoJFUN6xU dBhVHRKQFKBn79sVAVD5tTw46v3SHrs1nbvdIsEtAeVZ+RE+fI+LWfjIknQiSaZ3 4Nlmi2JnGiX9tgfeDp4HEArbnCzrtXLfmr4z8sEMjKzmk6yTKI5AxNXgwRVyD7Eu UpZzkHPe9Uo1jawnTLk7PSsbMrZ2/g== =yxWM -----END PGP SIGNATURE----- --2tfcczky2nyuuvxm--