* [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
@ 2022-06-03 11:27 ` Simon Horman
0 siblings, 0 replies; 12+ messages in thread
From: Simon Horman @ 2022-06-03 11:27 UTC (permalink / raw)
To: Jonathan Corbet; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
Correct a typo in the description of interaction between
the TCM and MMU.
Found by inspection.
Signed-off-by: Louis Peens <louis.peens@corigine.com>
Signed-off-by: Simon Horman <simon.horman@corigine.com>
---
Documentation/arm/tcm.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/arm/tcm.rst b/Documentation/arm/tcm.rst
index b256f9783883..1dc6c39220f9 100644
--- a/Documentation/arm/tcm.rst
+++ b/Documentation/arm/tcm.rst
@@ -34,7 +34,7 @@ CPU so it is usually wise not to overlap any physical RAM with
the TCM.
The TCM memory can then be remapped to another address again using
-the MMU, but notice that the TCM if often used in situations where
+the MMU, but notice that the TCM is often used in situations where
the MMU is turned off. To avoid confusion the current Linux
implementation will map the TCM 1 to 1 from physical to virtual
memory in the location specified by the kernel. Currently Linux
--
2.30.2
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
@ 2022-06-03 11:27 ` Simon Horman
0 siblings, 0 replies; 12+ messages in thread
From: Simon Horman @ 2022-06-03 11:27 UTC (permalink / raw)
To: Jonathan Corbet; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
Correct a typo in the description of interaction between
the TCM and MMU.
Found by inspection.
Signed-off-by: Louis Peens <louis.peens@corigine.com>
Signed-off-by: Simon Horman <simon.horman@corigine.com>
---
Documentation/arm/tcm.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/arm/tcm.rst b/Documentation/arm/tcm.rst
index b256f9783883..1dc6c39220f9 100644
--- a/Documentation/arm/tcm.rst
+++ b/Documentation/arm/tcm.rst
@@ -34,7 +34,7 @@ CPU so it is usually wise not to overlap any physical RAM with
the TCM.
The TCM memory can then be remapped to another address again using
-the MMU, but notice that the TCM if often used in situations where
+the MMU, but notice that the TCM is often used in situations where
the MMU is turned off. To avoid confusion the current Linux
implementation will map the TCM 1 to 1 from physical to virtual
memory in the location specified by the kernel. Currently Linux
--
2.30.2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
2022-06-03 11:27 ` Simon Horman
@ 2022-06-09 16:01 ` Jonathan Corbet
-1 siblings, 0 replies; 12+ messages in thread
From: Jonathan Corbet @ 2022-06-09 16:01 UTC (permalink / raw)
To: Simon Horman; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
Simon Horman <simon.horman@corigine.com> writes:
> Correct a typo in the description of interaction between
> the TCM and MMU.
>
> Found by inspection.
>
> Signed-off-by: Louis Peens <louis.peens@corigine.com>
> Signed-off-by: Simon Horman <simon.horman@corigine.com>
Here too I have to ask: what does this signoff chain mean? If two
developers were needed for a single-character typo fix, then a
Co-developed-by line is appropriate. If you are forwarding a patch from
Louis, then a From: line to get the authorship right is indicated ... ?
> Documentation/arm/tcm.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/arm/tcm.rst b/Documentation/arm/tcm.rst
> index b256f9783883..1dc6c39220f9 100644
> --- a/Documentation/arm/tcm.rst
> +++ b/Documentation/arm/tcm.rst
> @@ -34,7 +34,7 @@ CPU so it is usually wise not to overlap any physical RAM with
> the TCM.
>
> The TCM memory can then be remapped to another address again using
> -the MMU, but notice that the TCM if often used in situations where
> +the MMU, but notice that the TCM is often used in situations where
> the MMU is turned off. To avoid confusion the current Linux
> implementation will map the TCM 1 to 1 from physical to virtual
Thanks,
jon
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
@ 2022-06-09 16:01 ` Jonathan Corbet
0 siblings, 0 replies; 12+ messages in thread
From: Jonathan Corbet @ 2022-06-09 16:01 UTC (permalink / raw)
To: Simon Horman; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
Simon Horman <simon.horman@corigine.com> writes:
> Correct a typo in the description of interaction between
> the TCM and MMU.
>
> Found by inspection.
>
> Signed-off-by: Louis Peens <louis.peens@corigine.com>
> Signed-off-by: Simon Horman <simon.horman@corigine.com>
Here too I have to ask: what does this signoff chain mean? If two
developers were needed for a single-character typo fix, then a
Co-developed-by line is appropriate. If you are forwarding a patch from
Louis, then a From: line to get the authorship right is indicated ... ?
> Documentation/arm/tcm.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/arm/tcm.rst b/Documentation/arm/tcm.rst
> index b256f9783883..1dc6c39220f9 100644
> --- a/Documentation/arm/tcm.rst
> +++ b/Documentation/arm/tcm.rst
> @@ -34,7 +34,7 @@ CPU so it is usually wise not to overlap any physical RAM with
> the TCM.
>
> The TCM memory can then be remapped to another address again using
> -the MMU, but notice that the TCM if often used in situations where
> +the MMU, but notice that the TCM is often used in situations where
> the MMU is turned off. To avoid confusion the current Linux
> implementation will map the TCM 1 to 1 from physical to virtual
Thanks,
jon
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
2022-06-09 16:01 ` Jonathan Corbet
@ 2022-06-09 16:44 ` Simon Horman
-1 siblings, 0 replies; 12+ messages in thread
From: Simon Horman @ 2022-06-09 16:44 UTC (permalink / raw)
To: Jonathan Corbet; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
On Thu, Jun 09, 2022 at 10:01:01AM -0600, Jonathan Corbet wrote:
> Simon Horman <simon.horman@corigine.com> writes:
>
> > Correct a typo in the description of interaction between
> > the TCM and MMU.
> >
> > Found by inspection.
> >
> > Signed-off-by: Louis Peens <louis.peens@corigine.com>
> > Signed-off-by: Simon Horman <simon.horman@corigine.com>
>
> Here too I have to ask: what does this signoff chain mean? If two
> developers were needed for a single-character typo fix, then a
> Co-developed-by line is appropriate. If you are forwarding a patch from
> Louis, then a From: line to get the authorship right is indicated ... ?
Hi Jon,
The patch was created by me.
Due to internal processes a release by a colleague is provided, that is the
purpose of Louis's sign off (overkill in this case, I agree).
If Louis's sign-off is not necessary from your perspective then I'm happy
for you to drop it or for my to repost the patch without it.
> > Documentation/arm/tcm.rst | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/arm/tcm.rst b/Documentation/arm/tcm.rst
> > index b256f9783883..1dc6c39220f9 100644
> > --- a/Documentation/arm/tcm.rst
> > +++ b/Documentation/arm/tcm.rst
> > @@ -34,7 +34,7 @@ CPU so it is usually wise not to overlap any physical RAM with
> > the TCM.
> >
> > The TCM memory can then be remapped to another address again using
> > -the MMU, but notice that the TCM if often used in situations where
> > +the MMU, but notice that the TCM is often used in situations where
> > the MMU is turned off. To avoid confusion the current Linux
> > implementation will map the TCM 1 to 1 from physical to virtual
>
> Thanks,
>
> jon
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
@ 2022-06-09 16:44 ` Simon Horman
0 siblings, 0 replies; 12+ messages in thread
From: Simon Horman @ 2022-06-09 16:44 UTC (permalink / raw)
To: Jonathan Corbet; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
On Thu, Jun 09, 2022 at 10:01:01AM -0600, Jonathan Corbet wrote:
> Simon Horman <simon.horman@corigine.com> writes:
>
> > Correct a typo in the description of interaction between
> > the TCM and MMU.
> >
> > Found by inspection.
> >
> > Signed-off-by: Louis Peens <louis.peens@corigine.com>
> > Signed-off-by: Simon Horman <simon.horman@corigine.com>
>
> Here too I have to ask: what does this signoff chain mean? If two
> developers were needed for a single-character typo fix, then a
> Co-developed-by line is appropriate. If you are forwarding a patch from
> Louis, then a From: line to get the authorship right is indicated ... ?
Hi Jon,
The patch was created by me.
Due to internal processes a release by a colleague is provided, that is the
purpose of Louis's sign off (overkill in this case, I agree).
If Louis's sign-off is not necessary from your perspective then I'm happy
for you to drop it or for my to repost the patch without it.
> > Documentation/arm/tcm.rst | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/arm/tcm.rst b/Documentation/arm/tcm.rst
> > index b256f9783883..1dc6c39220f9 100644
> > --- a/Documentation/arm/tcm.rst
> > +++ b/Documentation/arm/tcm.rst
> > @@ -34,7 +34,7 @@ CPU so it is usually wise not to overlap any physical RAM with
> > the TCM.
> >
> > The TCM memory can then be remapped to another address again using
> > -the MMU, but notice that the TCM if often used in situations where
> > +the MMU, but notice that the TCM is often used in situations where
> > the MMU is turned off. To avoid confusion the current Linux
> > implementation will map the TCM 1 to 1 from physical to virtual
>
> Thanks,
>
> jon
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
2022-06-09 16:44 ` Simon Horman
@ 2022-06-09 17:27 ` Jonathan Corbet
-1 siblings, 0 replies; 12+ messages in thread
From: Jonathan Corbet @ 2022-06-09 17:27 UTC (permalink / raw)
To: Simon Horman; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
Simon Horman <simon.horman@corigine.com> writes:
> On Thu, Jun 09, 2022 at 10:01:01AM -0600, Jonathan Corbet wrote:
>> Simon Horman <simon.horman@corigine.com> writes:
>>
>> > Correct a typo in the description of interaction between
>> > the TCM and MMU.
>> >
>> > Found by inspection.
>> >
>> > Signed-off-by: Louis Peens <louis.peens@corigine.com>
>> > Signed-off-by: Simon Horman <simon.horman@corigine.com>
>>
>> Here too I have to ask: what does this signoff chain mean? If two
>> developers were needed for a single-character typo fix, then a
>> Co-developed-by line is appropriate. If you are forwarding a patch from
>> Louis, then a From: line to get the authorship right is indicated ... ?
>
> The patch was created by me.
>
> Due to internal processes a release by a colleague is provided, that is the
> purpose of Louis's sign off (overkill in this case, I agree).
>
> If Louis's sign-off is not necessary from your perspective then I'm happy
> for you to drop it or for my to repost the patch without it.
So are you perhaps using it like a Reviewed-by? If so, that's the tag
that should be used. But Signed-off-by has a well-defined meaning
described by the DCO, and "release by a colleague" isn't in there, so
I think that tag is not appropriate in this patch.
Thanks,
jon
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
@ 2022-06-09 17:27 ` Jonathan Corbet
0 siblings, 0 replies; 12+ messages in thread
From: Jonathan Corbet @ 2022-06-09 17:27 UTC (permalink / raw)
To: Simon Horman; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
Simon Horman <simon.horman@corigine.com> writes:
> On Thu, Jun 09, 2022 at 10:01:01AM -0600, Jonathan Corbet wrote:
>> Simon Horman <simon.horman@corigine.com> writes:
>>
>> > Correct a typo in the description of interaction between
>> > the TCM and MMU.
>> >
>> > Found by inspection.
>> >
>> > Signed-off-by: Louis Peens <louis.peens@corigine.com>
>> > Signed-off-by: Simon Horman <simon.horman@corigine.com>
>>
>> Here too I have to ask: what does this signoff chain mean? If two
>> developers were needed for a single-character typo fix, then a
>> Co-developed-by line is appropriate. If you are forwarding a patch from
>> Louis, then a From: line to get the authorship right is indicated ... ?
>
> The patch was created by me.
>
> Due to internal processes a release by a colleague is provided, that is the
> purpose of Louis's sign off (overkill in this case, I agree).
>
> If Louis's sign-off is not necessary from your perspective then I'm happy
> for you to drop it or for my to repost the patch without it.
So are you perhaps using it like a Reviewed-by? If so, that's the tag
that should be used. But Signed-off-by has a well-defined meaning
described by the DCO, and "release by a colleague" isn't in there, so
I think that tag is not appropriate in this patch.
Thanks,
jon
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
2022-06-09 17:27 ` Jonathan Corbet
@ 2022-06-09 17:54 ` Simon Horman
-1 siblings, 0 replies; 12+ messages in thread
From: Simon Horman @ 2022-06-09 17:54 UTC (permalink / raw)
To: Jonathan Corbet; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
On Thu, Jun 09, 2022 at 11:27:51AM -0600, Jonathan Corbet wrote:
> Simon Horman <simon.horman@corigine.com> writes:
>
> > On Thu, Jun 09, 2022 at 10:01:01AM -0600, Jonathan Corbet wrote:
> >> Simon Horman <simon.horman@corigine.com> writes:
> >>
> >> > Correct a typo in the description of interaction between
> >> > the TCM and MMU.
> >> >
> >> > Found by inspection.
> >> >
> >> > Signed-off-by: Louis Peens <louis.peens@corigine.com>
> >> > Signed-off-by: Simon Horman <simon.horman@corigine.com>
> >>
> >> Here too I have to ask: what does this signoff chain mean? If two
> >> developers were needed for a single-character typo fix, then a
> >> Co-developed-by line is appropriate. If you are forwarding a patch from
> >> Louis, then a From: line to get the authorship right is indicated ... ?
> >
> > The patch was created by me.
> >
> > Due to internal processes a release by a colleague is provided, that is the
> > purpose of Louis's sign off (overkill in this case, I agree).
> >
> > If Louis's sign-off is not necessary from your perspective then I'm happy
> > for you to drop it or for my to repost the patch without it.
>
> So are you perhaps using it like a Reviewed-by? If so, that's the tag
> that should be used. But Signed-off-by has a well-defined meaning
> described by the DCO, and "release by a colleague" isn't in there, so
> I think that tag is not appropriate in this patch.
Thanks Jon,
I'll post a v2 without the tag in question.
Kind regards,
Simon
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
@ 2022-06-09 17:54 ` Simon Horman
0 siblings, 0 replies; 12+ messages in thread
From: Simon Horman @ 2022-06-09 17:54 UTC (permalink / raw)
To: Jonathan Corbet; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
On Thu, Jun 09, 2022 at 11:27:51AM -0600, Jonathan Corbet wrote:
> Simon Horman <simon.horman@corigine.com> writes:
>
> > On Thu, Jun 09, 2022 at 10:01:01AM -0600, Jonathan Corbet wrote:
> >> Simon Horman <simon.horman@corigine.com> writes:
> >>
> >> > Correct a typo in the description of interaction between
> >> > the TCM and MMU.
> >> >
> >> > Found by inspection.
> >> >
> >> > Signed-off-by: Louis Peens <louis.peens@corigine.com>
> >> > Signed-off-by: Simon Horman <simon.horman@corigine.com>
> >>
> >> Here too I have to ask: what does this signoff chain mean? If two
> >> developers were needed for a single-character typo fix, then a
> >> Co-developed-by line is appropriate. If you are forwarding a patch from
> >> Louis, then a From: line to get the authorship right is indicated ... ?
> >
> > The patch was created by me.
> >
> > Due to internal processes a release by a colleague is provided, that is the
> > purpose of Louis's sign off (overkill in this case, I agree).
> >
> > If Louis's sign-off is not necessary from your perspective then I'm happy
> > for you to drop it or for my to repost the patch without it.
>
> So are you perhaps using it like a Reviewed-by? If so, that's the tag
> that should be used. But Signed-off-by has a well-defined meaning
> described by the DCO, and "release by a colleague" isn't in there, so
> I think that tag is not appropriate in this patch.
Thanks Jon,
I'll post a v2 without the tag in question.
Kind regards,
Simon
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
2022-06-09 17:54 ` Simon Horman
@ 2022-06-09 18:26 ` Jonathan Corbet
-1 siblings, 0 replies; 12+ messages in thread
From: Jonathan Corbet @ 2022-06-09 18:26 UTC (permalink / raw)
To: Simon Horman; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
Simon Horman <simon.horman@corigine.com> writes:
> Thanks Jon,
>
> I'll post a v2 without the tag in question.
Thanks, appreciated. Sorry to be obnoxious about it.
jon
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage
@ 2022-06-09 18:26 ` Jonathan Corbet
0 siblings, 0 replies; 12+ messages in thread
From: Jonathan Corbet @ 2022-06-09 18:26 UTC (permalink / raw)
To: Simon Horman; +Cc: Linus Walleij, linux-doc, linux-arm-kernel, oss-drivers
Simon Horman <simon.horman@corigine.com> writes:
> Thanks Jon,
>
> I'll post a v2 without the tag in question.
Thanks, appreciated. Sorry to be obnoxious about it.
jon
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2022-06-09 18:27 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-03 11:27 [PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage Simon Horman
2022-06-03 11:27 ` Simon Horman
2022-06-09 16:01 ` Jonathan Corbet
2022-06-09 16:01 ` Jonathan Corbet
2022-06-09 16:44 ` Simon Horman
2022-06-09 16:44 ` Simon Horman
2022-06-09 17:27 ` Jonathan Corbet
2022-06-09 17:27 ` Jonathan Corbet
2022-06-09 17:54 ` Simon Horman
2022-06-09 17:54 ` Simon Horman
2022-06-09 18:26 ` Jonathan Corbet
2022-06-09 18:26 ` Jonathan Corbet
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.