On Wed, Apr 30, 2025 at 06:32:27AM -0600, Jonathan Corbet wrote: > Gregory Price <gourry@xxxxxxxxxx> writes: > > > Add a simple device primer sufficient to understand the theory > > of operation documentation. > > > > Add carve-out for CDAT with a TODO. > > > > Signed-off-by: Gregory Price <gourry@xxxxxxxxxx> > > --- > > .../driver-api/cxl/devices/device-types.rst | 169 ++++++++++++++++++ > > Documentation/driver-api/cxl/devices/uefi.rst | 9 + > > Documentation/driver-api/cxl/index.rst | 2 + > > 3 files changed, 180 insertions(+) > > create mode 100644 Documentation/driver-api/cxl/devices/device-types.rst > > create mode 100644 Documentation/driver-api/cxl/devices/uefi.rst > > I'm glad to see all this documentation! One little nit... > > > diff --git a/Documentation/driver-api/cxl/devices/device-types.rst b/Documentation/driver-api/cxl/devices/device-types.rst > > new file mode 100644 > > index 000000000000..e8dd051c2c71 > > --- /dev/null > > +++ b/Documentation/driver-api/cxl/devices/device-types.rst > > @@ -0,0 +1,169 @@ > > +.. SPDX-License-Identifier: GPL-2.0 > > + > > +Devices and Protocols > > +##################### > > If you could stick with the section markup conventions we have > established in Documentation/doc-guide/sphinx.rst, I'd appreciate it. > Will do, mostly wanted to get it out for content first. Will give it a once over for formatting now. ~Gregory