mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Andrew Lunn <andrew@lunn.ch>
To: Abdurrahman Hussain <abdurrahman@nexthop.ai>
Cc: Mark Brown <broonie@kernel.org>,
	Michal Simek <michal.simek@amd.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	linux-spi@vger.kernel.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 3/3] spi: xilinx: use device property accessors.
Date: Tue, 20 Jan 2026 21:41:27 +0100	[thread overview]
Message-ID: <d5909def-1ce8-409e-a5cd-2405da89e5e2@lunn.ch> (raw)
In-Reply-To: <80A8F67E-7A01-4F9F-9D84-29722678A2CE@nexthop.ai>

> We could, but we don’t own the Xilinx IP blocks. Are we not justified in using PRP0001
> hack until the driver owner adds the HIDs? Wasn’t PRP0001 created as an escape hatch for
> these kind of scenarios?

I suspect your wording is wrong here. You probably have a license for
the Xilinx IP blocks, you are using in your synthesising for use in
your FPGA.

That i think you are trying to say is that you don't own the software
driver for the Xilinx IP blocks? But that should not matter. The Linux
community Maintains these drivers, and can make modifications to them.
You as part of this community can propose a patch which adds the
needed IDs to the driver.

The "escape hatch" is generally used when there is a mature DT
binding, but nothing for ACPI. Linux has a mature and complex set of
DT bindings around network device sub-components, where Linux drivers
all the sub-components. ACPI has nothing in this area, because network
devices used in the ACPI world tend to use firmware, not Linux to
drive the hardware. In such a case, using the escape hatch makes
sense. However, I2C, SPI, etc all have well established ACPI bindings
and are part of the basic ACPI standard. It makes no sense to use the
"escape hatch" hatch for such devices. Please follow the ACPI
standard.

       Andrew

  reply	other threads:[~2026-01-20 20:41 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-19  7:06 [PATCH v3 0/3] spi: xilinx: switch to device properties and make IRQs optional Abdurrahman Hussain via B4 Relay
2026-01-19  7:06 ` [PATCH v3 1/3] spi: dt-bindings: xilinx: make interrupts optional Abdurrahman Hussain via B4 Relay
2026-01-19  7:06 ` [PATCH v3 2/3] spi: xilinx: make irq optional Abdurrahman Hussain via B4 Relay
2026-01-19  7:06 ` [PATCH v3 3/3] spi: xilinx: use device property accessors Abdurrahman Hussain via B4 Relay
2026-01-19 16:32   ` Mark Brown
2026-01-19 16:47     ` Abdurrahman Hussain
2026-01-19 16:50       ` Mark Brown
2026-01-19 17:15         ` Abdurrahman Hussain
2026-01-19 17:32           ` Mark Brown
2026-01-19 17:47             ` Abdurrahman Hussain
2026-01-19 18:38               ` Mark Brown
2026-01-19 18:52                 ` Michal Simek
2026-01-19 19:01                   ` Mark Brown
2026-01-19 19:17                     ` Michal Simek
2026-01-19 19:56                       ` Mark Brown
2026-01-19 20:30                         ` Abdurrahman Hussain
2026-01-19 23:20                           ` Mark Brown
2026-01-20  0:20                             ` Abdurrahman Hussain
2026-01-20 18:45                               ` Mark Brown
2026-01-20 19:11                                 ` Abdurrahman Hussain
2026-01-20 20:41                                   ` Andrew Lunn [this message]
2026-01-20 20:49                                   ` Andrew Lunn
2026-01-20 21:04                                   ` Mark Brown
2026-01-20 21:28                                     ` Andy Shevchenko
2026-01-20 21:38                                       ` Andy Shevchenko
2026-01-21  8:11                                         ` Michal Simek
2026-01-21  8:40                                           ` Andy Shevchenko
2026-01-20  9:23                         ` Andy Shevchenko
2026-01-20 19:34                           ` Andy Shevchenko
2026-01-19 19:18                     ` Abdurrahman Hussain
2026-01-21 15:10   ` Michal Simek
2026-01-20 19:23 ` (subset) [PATCH v3 0/3] spi: xilinx: switch to device properties and make IRQs optional Mark Brown
2026-01-21  8:15   ` Michal Simek
2026-01-21 11:39     ` Mark Brown
2026-01-21 12:26       ` Michal Simek
2026-01-21 12:35         ` Mark Brown

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=d5909def-1ce8-409e-a5cd-2405da89e5e2@lunn.ch \
    --to=andrew@lunn.ch \
    --cc=abdurrahman@nexthop.ai \
    --cc=broonie@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-spi@vger.kernel.org \
    --cc=michal.simek@amd.com \
    --cc=robh@kernel.org \
    /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

all inboxes | Powered by JetHome®