From: Jon Hunter <jonathanh@nvidia.com>
To: Mark Brown <broonie@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>
Cc: Vishwaroop A <va@nvidia.com>,
krzk+dt@kernel.org, robh@kernel.org, conor+dt@kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-tegra@vger.kernel.org, linux-spi@vger.kernel.org
Subject: Re: [PATCH 2/3] dt-bindings: spi: Add DT schema for Tegra SPIDEV controller
Date: Wed, 26 Mar 2025 12:16:53 +0000 [thread overview]
Message-ID: <63b87feb-32ee-423c-8d82-61445414c6f7@nvidia.com> (raw)
In-Reply-To: <25857b7f-5c10-46ec-b0b7-9ff89ca5ab1e@sirena.org.uk>
On 25/03/2025 17:05, Mark Brown wrote:
...
>> The way I imagine it, exporting would involve writing a chip-select to a
>> specific SPI controller's "export" sysfs attribute to have a SPI device
>> created for that particular chip-select and bind it to spidev.
>
> My general feeling with those is that if you're building for them you're
> probably either already modifiying your kernel or easily able to cope
> with doing so.
That's definitely what we do today, modify the kernel directly to
achieve what we need. I am trying to avoid carrying too many out of tree
patches for stuff like this and have something in the kernel that works
by default. This is even more important for 3rd party Linux distros that
will not accept non-upstream code.
Our devkits, very much like Raspberry PI, allow users to connect various
hardware for development and so having an easy way to connect a SPI
device is useful. For any production systems, users will definitely want
a proper device and device-tree bindings. So I am just trying to explore
what would be acceptable. If it is acceptable to have a sysfs interface
for creating a SPI device at runtime, then we can look into that.
Cheers!
Jon
--
nvpublic
next prev parent reply other threads:[~2025-03-26 12:17 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-11-26 13:45 [PATCH 0/3] Add spidev nodes for SPI controllers Vishwaroop A
2024-11-26 13:45 ` [PATCH 1/3] arm64: tegra: " Vishwaroop A
2024-11-26 13:45 ` [PATCH 2/3] dt-bindings: spi: Add DT schema for Tegra SPIDEV controller Vishwaroop A
2024-11-26 13:56 ` Mark Brown
2024-11-27 15:54 ` Jon Hunter
2024-11-27 16:09 ` Mark Brown
2024-11-27 17:24 ` Jon Hunter
2024-11-27 17:31 ` Mark Brown
2025-03-25 10:36 ` Jon Hunter
2025-03-25 12:10 ` Mark Brown
2025-03-25 12:45 ` Thierry Reding
2025-03-25 13:05 ` Thierry Reding
2025-03-25 15:55 ` Mark Brown
2025-03-25 16:38 ` Thierry Reding
2025-03-25 17:05 ` Mark Brown
2025-03-26 12:16 ` Jon Hunter [this message]
2025-03-27 15:33 ` Mark Brown
2025-03-31 12:34 ` Jon Hunter
2025-03-31 12:44 ` Mark Brown
2025-03-31 13:11 ` Jon Hunter
2025-03-31 14:09 ` Mark Brown
2025-03-25 13:57 ` Mark Brown
2024-11-26 13:45 ` [PATCH 3/3] spi: spidev: add "tegra-spidev" name string Vishwaroop A
2024-11-26 14:02 ` Krzysztof Kozlowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=63b87feb-32ee-423c-8d82-61445414c6f7@nvidia.com \
--to=jonathanh@nvidia.com \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=robh@kernel.org \
--cc=thierry.reding@gmail.com \
--cc=va@nvidia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox