* [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
@ 2024-02-19 14:41 Josua Mayer
2024-02-20 12:19 ` Pratyush Yadav
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Josua Mayer @ 2024-02-19 14:41 UTC (permalink / raw)
To: Tudor Ambarus, Pratyush Yadav, Michael Walle, Miquel Raynal,
Richard Weinberger, Vignesh Raghavendra, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-mtd
Cc: Yazan Shhady, Rob Herring, devicetree, linux-kernel, Josua Mayer
Some spi flash memories have an interrupt signal which can be used for
signalling on-chip events such as busy status or ecc errors to the host.
Add binding for "interrupts" property so that boards wiring this signal
may describe the connection.
Signed-off-by: Josua Mayer <josua@solid-run.com>
Acked-by: Rob Herring <robh@kernel.org>
---
Changes in v7:
- split out of original patchset because it should go via mtd tree
- collected acked-by Rob Herring from v6
- Link to v6: https://lore.kernel.org/r/20240212-add-am64-som-v6-0-b59edb2bc8c3@solid-run.com
---
Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml b/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml
index 58f0cea160ef..6e3afb42926e 100644
--- a/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml
+++ b/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml
@@ -52,6 +52,9 @@ properties:
minItems: 1
maxItems: 2
+ interrupts:
+ maxItems: 1
+
m25p,fast-read:
type: boolean
description:
---
base-commit: 6613476e225e090cc9aad49be7fa504e290dd33d
change-id: 20240219-mtd-flash-interrupt-binding-647f78d123cb
Best regards,
--
Josua Mayer <josua@solid-run.com>
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
2024-02-19 14:41 [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property Josua Mayer
@ 2024-02-20 12:19 ` Pratyush Yadav
2024-02-20 12:31 ` Josua Mayer
2024-02-21 8:27 ` Michael Walle
2024-02-26 10:02 ` Tudor Ambarus
2 siblings, 1 reply; 10+ messages in thread
From: Pratyush Yadav @ 2024-02-20 12:19 UTC (permalink / raw)
To: Josua Mayer
Cc: Tudor Ambarus, Pratyush Yadav, Michael Walle, Miquel Raynal,
Richard Weinberger, Vignesh Raghavendra, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-mtd, Yazan Shhady,
Rob Herring, devicetree, linux-kernel
On Mon, Feb 19 2024, Josua Mayer wrote:
> Some spi flash memories have an interrupt signal which can be used for
> signalling on-chip events such as busy status or ecc errors to the host.
>
> Add binding for "interrupts" property so that boards wiring this signal
> may describe the connection.
>
> Signed-off-by: Josua Mayer <josua@solid-run.com>
> Acked-by: Rob Herring <robh@kernel.org>
Acked-by: Pratyush Yadav <pratyush@kernel.org>
BTW, I don't see any support in SPI NOR for handling these interrupts.
Do you plan to add them in a later patchset? If not, what do you get by
describing them?
> ---
> Changes in v7:
> - split out of original patchset because it should go via mtd tree
> - collected acked-by Rob Herring from v6
> - Link to v6: https://lore.kernel.org/r/20240212-add-am64-som-v6-0-b59edb2bc8c3@solid-run.com
> ---
> Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml b/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml
> index 58f0cea160ef..6e3afb42926e 100644
> --- a/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml
> +++ b/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml
> @@ -52,6 +52,9 @@ properties:
> minItems: 1
> maxItems: 2
>
> + interrupts:
> + maxItems: 1
> +
> m25p,fast-read:
> type: boolean
> description:
>
--
Regards,
Pratyush Yadav
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
2024-02-20 12:19 ` Pratyush Yadav
@ 2024-02-20 12:31 ` Josua Mayer
2024-02-20 13:30 ` Pratyush Yadav
0 siblings, 1 reply; 10+ messages in thread
From: Josua Mayer @ 2024-02-20 12:31 UTC (permalink / raw)
To: Pratyush Yadav
Cc: Tudor Ambarus, Michael Walle, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-mtd, Yazan Shhady, Rob Herring, devicetree,
linux-kernel
Am 20.02.24 um 13:19 schrieb Pratyush Yadav:
> On Mon, Feb 19 2024, Josua Mayer wrote:
>
>> Some spi flash memories have an interrupt signal which can be used for
>> signalling on-chip events such as busy status or ecc errors to the host.
>>
>> Add binding for "interrupts" property so that boards wiring this signal
>> may describe the connection.
>>
>> Signed-off-by: Josua Mayer <josua@solid-run.com>
>> Acked-by: Rob Herring <robh@kernel.org>
> Acked-by: Pratyush Yadav <pratyush@kernel.org>
>
> BTW, I don't see any support in SPI NOR for handling these interrupts.
> Do you plan to add them in a later patchset?
No current plans, I have little knowledge how spi-nor works in the kernel.
> If not, what do you get by
> describing them?
Foremost I get to submit a correct device-tree (describes hardware)
to the kernel, without maintainers getting all over me for introducing
new dtbs_check errors.
And I really do prefer submitting a complete device-tree so that all
knowledge I have gained reading private schematics is readily
available the next time someone works on it.
>
>> ---
>> Changes in v7:
>> - split out of original patchset because it should go via mtd tree
>> - collected acked-by Rob Herring from v6
>> - Link to v6: https://lore.kernel.org/r/20240212-add-am64-som-v6-0-b59edb2bc8c3@solid-run.com
>> ---
>> Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml b/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml
>> index 58f0cea160ef..6e3afb42926e 100644
>> --- a/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml
>> +++ b/Documentation/devicetree/bindings/mtd/jedec,spi-nor.yaml
>> @@ -52,6 +52,9 @@ properties:
>> minItems: 1
>> maxItems: 2
>>
>> + interrupts:
>> + maxItems: 1
>> +
>> m25p,fast-read:
>> type: boolean
>> description:
>>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
2024-02-20 12:31 ` Josua Mayer
@ 2024-02-20 13:30 ` Pratyush Yadav
0 siblings, 0 replies; 10+ messages in thread
From: Pratyush Yadav @ 2024-02-20 13:30 UTC (permalink / raw)
To: Josua Mayer
Cc: Pratyush Yadav, Tudor Ambarus, Michael Walle, Miquel Raynal,
Richard Weinberger, Vignesh Raghavendra, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-mtd, Yazan Shhady,
Rob Herring, devicetree, linux-kernel
On Tue, Feb 20 2024, Josua Mayer wrote:
> Am 20.02.24 um 13:19 schrieb Pratyush Yadav:
>> On Mon, Feb 19 2024, Josua Mayer wrote:
>>
>>> Some spi flash memories have an interrupt signal which can be used for
>>> signalling on-chip events such as busy status or ecc errors to the host.
>>>
>>> Add binding for "interrupts" property so that boards wiring this signal
>>> may describe the connection.
>>>
>>> Signed-off-by: Josua Mayer <josua@solid-run.com>
>>> Acked-by: Rob Herring <robh@kernel.org>
>> Acked-by: Pratyush Yadav <pratyush@kernel.org>
>>
>> BTW, I don't see any support in SPI NOR for handling these interrupts.
>> Do you plan to add them in a later patchset?
> No current plans, I have little knowledge how spi-nor works in the kernel.
>> If not, what do you get by
>> describing them?
> Foremost I get to submit a correct device-tree (describes hardware)
> to the kernel, without maintainers getting all over me for introducing
> new dtbs_check errors.
>
> And I really do prefer submitting a complete device-tree so that all
> knowledge I have gained reading private schematics is readily
> available the next time someone works on it.
Fair enough. I guessed as much but thanks for the confirmation!
[...]
--
Regards,
Pratyush Yadav
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
2024-02-19 14:41 [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property Josua Mayer
2024-02-20 12:19 ` Pratyush Yadav
@ 2024-02-21 8:27 ` Michael Walle
2024-02-21 9:13 ` Josua Mayer
2024-02-26 10:02 ` Tudor Ambarus
2 siblings, 1 reply; 10+ messages in thread
From: Michael Walle @ 2024-02-21 8:27 UTC (permalink / raw)
To: Josua Mayer, Tudor Ambarus, Pratyush Yadav, Miquel Raynal,
Richard Weinberger, Vignesh Raghavendra, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-mtd
Cc: Yazan Shhady, Rob Herring, devicetree, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 287 bytes --]
Hi,
On Mon Feb 19, 2024 at 3:41 PM CET, Josua Mayer wrote:
> Some spi flash memories have an interrupt signal which can be used for
> signalling on-chip events such as busy status or ecc errors to the host.
Do you have an example? Maybe one with a public datasheet?
-michael
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 252 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
2024-02-21 8:27 ` Michael Walle
@ 2024-02-21 9:13 ` Josua Mayer
2024-02-21 9:23 ` Michael Walle
0 siblings, 1 reply; 10+ messages in thread
From: Josua Mayer @ 2024-02-21 9:13 UTC (permalink / raw)
To: Michael Walle, Tudor Ambarus, Pratyush Yadav, Miquel Raynal,
Richard Weinberger, Vignesh Raghavendra, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-mtd
Cc: Yazan Shhady, Rob Herring, devicetree, linux-kernel
Hi,
Am 21.02.24 um 09:27 schrieb Michael Walle:
> Hi,
>
> On Mon Feb 19, 2024 at 3:41 PM CET, Josua Mayer wrote:
>> Some spi flash memories have an interrupt signal which can be used for
>> signalling on-chip events such as busy status or ecc errors to the host.
> Do you have an example? Maybe one with a public datasheet?
My example is Infineon S28HS512T, however datasheet download requires
user account.
S26HS512T has interrupt line, too, and datasheet is downloadable without
registration:
https://www.infineon.com/cms/en/product/memories/nor-flash/semper-nor-flash-family/semper-nor-flash/#!documents
>
> -michael
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
2024-02-21 9:13 ` Josua Mayer
@ 2024-02-21 9:23 ` Michael Walle
2024-02-21 10:48 ` Pratyush Yadav
0 siblings, 1 reply; 10+ messages in thread
From: Michael Walle @ 2024-02-21 9:23 UTC (permalink / raw)
To: Josua Mayer, Tudor Ambarus, Pratyush Yadav, Miquel Raynal,
Richard Weinberger, Vignesh Raghavendra, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-mtd, Takahiro Kuwano,
Takahiro Kuwano
Cc: Yazan Shhady, Rob Herring, devicetree, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 888 bytes --]
[+ Takahiro]
Hi,
On Wed Feb 21, 2024 at 10:13 AM CET, Josua Mayer wrote:
> Hi,
>
> Am 21.02.24 um 09:27 schrieb Michael Walle:
> > Hi,
> >
> > On Mon Feb 19, 2024 at 3:41 PM CET, Josua Mayer wrote:
> >> Some spi flash memories have an interrupt signal which can be used for
> >> signalling on-chip events such as busy status or ecc errors to the host.
> > Do you have an example? Maybe one with a public datasheet?
>
> My example is Infineon S28HS512T, however datasheet download requires
> user account.
>
> S26HS512T has interrupt line, too, and datasheet is downloadable without
> registration:
> https://www.infineon.com/cms/en/product/memories/nor-flash/semper-nor-flash-family/semper-nor-flash/#!documents
Thanks, as far as I can see, both are hyperbus flashes. I'm asking
because I'm not aware of any SPI NOR flash with an interrupt line.
-michael
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 252 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
2024-02-21 9:23 ` Michael Walle
@ 2024-02-21 10:48 ` Pratyush Yadav
2024-02-21 11:52 ` Michael Walle
0 siblings, 1 reply; 10+ messages in thread
From: Pratyush Yadav @ 2024-02-21 10:48 UTC (permalink / raw)
To: Michael Walle
Cc: Josua Mayer, Tudor Ambarus, Pratyush Yadav, Miquel Raynal,
Richard Weinberger, Vignesh Raghavendra, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-mtd, Takahiro Kuwano,
Takahiro Kuwano, Yazan Shhady, Rob Herring, devicetree,
linux-kernel
On Wed, Feb 21 2024, Michael Walle wrote:
> [+ Takahiro]
>
> Hi,
>
> On Wed Feb 21, 2024 at 10:13 AM CET, Josua Mayer wrote:
>> Hi,
>>
>> Am 21.02.24 um 09:27 schrieb Michael Walle:
>> > Hi,
>> >
>> > On Mon Feb 19, 2024 at 3:41 PM CET, Josua Mayer wrote:
>> >> Some spi flash memories have an interrupt signal which can be used for
>> >> signalling on-chip events such as busy status or ecc errors to the host.
>> > Do you have an example? Maybe one with a public datasheet?
>>
>> My example is Infineon S28HS512T, however datasheet download requires
>> user account.
>>
>> S26HS512T has interrupt line, too, and datasheet is downloadable without
>> registration:
>> https://www.infineon.com/cms/en/product/memories/nor-flash/semper-nor-flash-family/semper-nor-flash/#!documents
>
> Thanks, as far as I can see, both are hyperbus flashes. I'm asking
> because I'm not aware of any SPI NOR flash with an interrupt line.
I found this datasheet [0] for S28H flash family from Infineon on
Google. These are SPI NOR flashes. In pinout you can see there is an
INT# signal. The signal description says: "System Interrupt (INT#). When
LOW, the device is indicating that an internal event has occurred."
Further in section 4.1.1.5 "INT# Output" it says:
HL-T/HS-T supports INT# output pin to indicate to the host system
that an event has occurred within the flash device. The user can
configure the INT# output pin to transition to the active (LOW)
state when:
- 2-bit ECC error is detected
- 1-bit ECC error is detected
- Transitioning from the Busy to the Ready state
[0] https://www.infineon.com/dgdl/Infineon-S28HS256T_S28HS512T_S28HS01GT_S28HL256T_S28HL512T_S28HL01GT_256-Mb_(32-MB)_512-Mb_(64-MB)_1-Gb_(128-MB)_HS-T_(1.8-V)_HL-T_(3.0-V)_Semper_Flash_with_Octal_Interface-DataSheet-v03_00-EN.pdf?fileId=8ac78c8c7d0d8da4017d0ee6bca96f97&da=t
--
Regards,
Pratyush Yadav
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
2024-02-21 10:48 ` Pratyush Yadav
@ 2024-02-21 11:52 ` Michael Walle
0 siblings, 0 replies; 10+ messages in thread
From: Michael Walle @ 2024-02-21 11:52 UTC (permalink / raw)
To: Pratyush Yadav
Cc: Josua Mayer, Tudor Ambarus, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-mtd, Takahiro Kuwano, Takahiro Kuwano,
Yazan Shhady, Rob Herring, devicetree, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 2131 bytes --]
On Wed Feb 21, 2024 at 11:48 AM CET, Pratyush Yadav wrote:
> On Wed, Feb 21 2024, Michael Walle wrote:
>
> > [+ Takahiro]
> >
> > Hi,
> >
> > On Wed Feb 21, 2024 at 10:13 AM CET, Josua Mayer wrote:
> >> Hi,
> >>
> >> Am 21.02.24 um 09:27 schrieb Michael Walle:
> >> > Hi,
> >> >
> >> > On Mon Feb 19, 2024 at 3:41 PM CET, Josua Mayer wrote:
> >> >> Some spi flash memories have an interrupt signal which can be used for
> >> >> signalling on-chip events such as busy status or ecc errors to the host.
> >> > Do you have an example? Maybe one with a public datasheet?
> >>
> >> My example is Infineon S28HS512T, however datasheet download requires
> >> user account.
> >>
> >> S26HS512T has interrupt line, too, and datasheet is downloadable without
> >> registration:
> >> https://www.infineon.com/cms/en/product/memories/nor-flash/semper-nor-flash-family/semper-nor-flash/#!documents
> >
> > Thanks, as far as I can see, both are hyperbus flashes. I'm asking
> > because I'm not aware of any SPI NOR flash with an interrupt line.
>
> I found this datasheet [0] for S28H flash family from Infineon on
> Google. These are SPI NOR flashes. In pinout you can see there is an
> INT# signal. The signal description says: "System Interrupt (INT#). When
> LOW, the device is indicating that an internal event has occurred."
> Further in section 4.1.1.5 "INT# Output" it says:
>
> HL-T/HS-T supports INT# output pin to indicate to the host system
> that an event has occurred within the flash device. The user can
> configure the INT# output pin to transition to the active (LOW)
> state when:
>
> - 2-bit ECC error is detected
> - 1-bit ECC error is detected
> - Transitioning from the Busy to the Ready state
>
> [0] https://www.infineon.com/dgdl/Infineon-S28HS256T_S28HS512T_S28HS01GT_S28HL256T_S28HL512T_S28HL01GT_256-Mb_(32-MB)_512-Mb_(64-MB)_1-Gb_(128-MB)_HS-T_(1.8-V)_HL-T_(3.0-V)_Semper_Flash_with_Octal_Interface-DataSheet-v03_00-EN.pdf?fileId=8ac78c8c7d0d8da4017d0ee6bca96f97&da=t
Okay then,
Acked-by: Michael Walle <mwalle@kernel.org>
-michael
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 252 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property
2024-02-19 14:41 [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property Josua Mayer
2024-02-20 12:19 ` Pratyush Yadav
2024-02-21 8:27 ` Michael Walle
@ 2024-02-26 10:02 ` Tudor Ambarus
2 siblings, 0 replies; 10+ messages in thread
From: Tudor Ambarus @ 2024-02-26 10:02 UTC (permalink / raw)
To: Pratyush Yadav, Michael Walle, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-mtd, Josua Mayer
Cc: Tudor Ambarus, Yazan Shhady, Rob Herring, devicetree, linux-kernel
On Mon, 19 Feb 2024 15:41:19 +0100, Josua Mayer wrote:
> Some spi flash memories have an interrupt signal which can be used for
> signalling on-chip events such as busy status or ecc errors to the host.
>
> Add binding for "interrupts" property so that boards wiring this signal
> may describe the connection.
>
>
> [...]
Applied to git://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git,
spi-nor/next branch. Thanks!
[1/1] dt-bindings: mtd: spi-nor: add optional interrupts property
(no commit info)
Cheers,
--
Tudor Ambarus <tudor.ambarus@linaro.org>
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2024-02-26 10:02 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-19 14:41 [PATCH v7] dt-bindings: mtd: spi-nor: add optional interrupts property Josua Mayer
2024-02-20 12:19 ` Pratyush Yadav
2024-02-20 12:31 ` Josua Mayer
2024-02-20 13:30 ` Pratyush Yadav
2024-02-21 8:27 ` Michael Walle
2024-02-21 9:13 ` Josua Mayer
2024-02-21 9:23 ` Michael Walle
2024-02-21 10:48 ` Pratyush Yadav
2024-02-21 11:52 ` Michael Walle
2024-02-26 10:02 ` Tudor Ambarus
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).