diff mbox series

[1/3] dt-bindings: clock: add pcm reset for ipq806x lcc

Message ID 20220615163408.30154-1-ansuelsmth@gmail.com (mailing list archive)
State Superseded
Headers show
Series [1/3] dt-bindings: clock: add pcm reset for ipq806x lcc | expand

Commit Message

Christian Marangi June 15, 2022, 4:34 p.m. UTC
Add pcm reset define for ipq806x lcc.

Signed-off-by: Christian 'Ansuel' Marangi <ansuelsmth@gmail.com>
---
 include/dt-bindings/clock/qcom,lcc-ipq806x.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Christian Marangi June 15, 2022, 4:53 p.m. UTC | #1
On Wed, Jun 15, 2022 at 10:43:10AM -0700, Krzysztof Kozlowski wrote:
> On 15/06/2022 09:34, Christian 'Ansuel' Marangi wrote:
> > Add pcm reset define for ipq806x lcc.
> > 
> > Signed-off-by: Christian 'Ansuel' Marangi <ansuelsmth@gmail.com>
> To prevent any confusion about identities (we have strict rules about
> these), I need to ask - who uses this email address?
> 
> https://lore.kernel.org/all/?q=ansuelsmth%40gmail.com
> 
> Best regards,
> Krzysztof

Same person. Started using extended name, wanted to do this change from
a long time but all the patch were already pushed so I couldn't change
it since they were already proposed and on the various mailing list.
Christian Marangi June 15, 2022, 5:04 p.m. UTC | #2
On Wed, Jun 15, 2022 at 10:50:17AM -0700, Krzysztof Kozlowski wrote:
> On 15/06/2022 09:53, Ansuel Smith wrote:
> > On Wed, Jun 15, 2022 at 10:43:10AM -0700, Krzysztof Kozlowski wrote:
> >> On 15/06/2022 09:34, Christian 'Ansuel' Marangi wrote:
> >>> Add pcm reset define for ipq806x lcc.
> >>>
> >>> Signed-off-by: Christian 'Ansuel' Marangi <ansuelsmth@gmail.com>
> >> To prevent any confusion about identities (we have strict rules about
> >> these), I need to ask - who uses this email address?
> >>
> >> https://lore.kernel.org/all/?q=ansuelsmth%40gmail.com
> >>
> >> Best regards,
> >> Krzysztof
> > 
> > Same person. Started using extended name, wanted to do this change from
> > a long time but all the patch were already pushed so I couldn't change
> > it since they were already proposed and on the various mailing list.
> 
> Previously "Ansuel Smith" was used entirely, without any parts of this
> name. Here 'Ansuel' appears in quotes, which usually is used for nicknames.
> 
> Is "Ansuel Smith" your real name or a nickname? What do you mean by
> "extended name"?
>

Ansuel is second name. Smith is not real... (sorry)
So with extendend name I mean full name + second name that is
Christian Ansuel Marangi.

Honestly it's a very stupid mistake by me not using the full name from
the start.

> 
> Best regards,
> Krzysztof
Krzysztof Kozlowski June 15, 2022, 5:43 p.m. UTC | #3
On 15/06/2022 09:34, Christian 'Ansuel' Marangi wrote:
> Add pcm reset define for ipq806x lcc.
> 
> Signed-off-by: Christian 'Ansuel' Marangi <ansuelsmth@gmail.com>
To prevent any confusion about identities (we have strict rules about
these), I need to ask - who uses this email address?

https://lore.kernel.org/all/?q=ansuelsmth%40gmail.com

Best regards,
Krzysztof
Krzysztof Kozlowski June 15, 2022, 5:50 p.m. UTC | #4
On 15/06/2022 09:53, Ansuel Smith wrote:
> On Wed, Jun 15, 2022 at 10:43:10AM -0700, Krzysztof Kozlowski wrote:
>> On 15/06/2022 09:34, Christian 'Ansuel' Marangi wrote:
>>> Add pcm reset define for ipq806x lcc.
>>>
>>> Signed-off-by: Christian 'Ansuel' Marangi <ansuelsmth@gmail.com>
>> To prevent any confusion about identities (we have strict rules about
>> these), I need to ask - who uses this email address?
>>
>> https://lore.kernel.org/all/?q=ansuelsmth%40gmail.com
>>
>> Best regards,
>> Krzysztof
> 
> Same person. Started using extended name, wanted to do this change from
> a long time but all the patch were already pushed so I couldn't change
> it since they were already proposed and on the various mailing list.

Previously "Ansuel Smith" was used entirely, without any parts of this
name. Here 'Ansuel' appears in quotes, which usually is used for nicknames.

Is "Ansuel Smith" your real name or a nickname? What do you mean by
"extended name"?


Best regards,
Krzysztof
Stephen Boyd June 15, 2022, 8:02 p.m. UTC | #5
Quoting Ansuel Smith (2022-06-15 10:04:40)
> 
> Ansuel is second name. Smith is not real... (sorry)
> So with extendend name I mean full name + second name that is
> Christian Ansuel Marangi.
> 
> Honestly it's a very stupid mistake by me not using the full name from
> the start.

Please send a patch to the .mailmap file to fix up the name.
Krzysztof Kozlowski June 15, 2022, 8:53 p.m. UTC | #6
On 15/06/2022 10:04, Ansuel Smith wrote:
> On Wed, Jun 15, 2022 at 10:50:17AM -0700, Krzysztof Kozlowski wrote:
>> On 15/06/2022 09:53, Ansuel Smith wrote:
>>> On Wed, Jun 15, 2022 at 10:43:10AM -0700, Krzysztof Kozlowski wrote:
>>>> On 15/06/2022 09:34, Christian 'Ansuel' Marangi wrote:
>>>>> Add pcm reset define for ipq806x lcc.
>>>>>
>>>>> Signed-off-by: Christian 'Ansuel' Marangi <ansuelsmth@gmail.com>
>>>> To prevent any confusion about identities (we have strict rules about
>>>> these), I need to ask - who uses this email address?
>>>>
>>>> https://lore.kernel.org/all/?q=ansuelsmth%40gmail.com
>>>>
>>>> Best regards,
>>>> Krzysztof
>>>
>>> Same person. Started using extended name, wanted to do this change from
>>> a long time but all the patch were already pushed so I couldn't change
>>> it since they were already proposed and on the various mailing list.
>>
>> Previously "Ansuel Smith" was used entirely, without any parts of this
>> name. Here 'Ansuel' appears in quotes, which usually is used for nicknames.
>>
>> Is "Ansuel Smith" your real name or a nickname? What do you mean by
>> "extended name"?
>>
> 
> Ansuel is second name. Smith is not real... (sorry)
> So with extendend name I mean full name + second name that is
> Christian Ansuel Marangi.
> 
> Honestly it's a very stupid mistake by me not using the full name from
> the start.

Not-real names are no accepted [1] and since we cannot validate this, we
trust. Quite a lot in our process depends on trust. Once trust is gone,
it's not easy to get it back... Worth watching - Trust and the Linux
development model; Greg KH [2].

Anyway I have no clue which identity to trust...


[1]
https://www.kernel.org/doc/html/latest/process/submitting-patches.html?highlight=certificate#sign-your-work-the-developer-s-certificate-of-origin

[2] https://www.youtube.com/watch?v=nhJqaZT94z0&t=7044s&ab_channel=hupstream

Best regards,
Krzysztof
Christian Marangi June 15, 2022, 9:32 p.m. UTC | #7
On Wed, Jun 15, 2022 at 01:53:44PM -0700, Krzysztof Kozlowski wrote:
> On 15/06/2022 10:04, Ansuel Smith wrote:
> > On Wed, Jun 15, 2022 at 10:50:17AM -0700, Krzysztof Kozlowski wrote:
> >> On 15/06/2022 09:53, Ansuel Smith wrote:
> >>> On Wed, Jun 15, 2022 at 10:43:10AM -0700, Krzysztof Kozlowski wrote:
> >>>> On 15/06/2022 09:34, Christian 'Ansuel' Marangi wrote:
> >>>>> Add pcm reset define for ipq806x lcc.
> >>>>>
> >>>>> Signed-off-by: Christian 'Ansuel' Marangi <ansuelsmth@gmail.com>
> >>>> To prevent any confusion about identities (we have strict rules about
> >>>> these), I need to ask - who uses this email address?
> >>>>
> >>>> https://lore.kernel.org/all/?q=ansuelsmth%40gmail.com
> >>>>
> >>>> Best regards,
> >>>> Krzysztof
> >>>
> >>> Same person. Started using extended name, wanted to do this change from
> >>> a long time but all the patch were already pushed so I couldn't change
> >>> it since they were already proposed and on the various mailing list.
> >>
> >> Previously "Ansuel Smith" was used entirely, without any parts of this
> >> name. Here 'Ansuel' appears in quotes, which usually is used for nicknames.
> >>
> >> Is "Ansuel Smith" your real name or a nickname? What do you mean by
> >> "extended name"?
> >>
> > 
> > Ansuel is second name. Smith is not real... (sorry)
> > So with extendend name I mean full name + second name that is
> > Christian Ansuel Marangi.
> > 
> > Honestly it's a very stupid mistake by me not using the full name from
> > the start.
> 
> Not-real names are no accepted [1] and since we cannot validate this, we
> trust. Quite a lot in our process depends on trust. Once trust is gone,
> it's not easy to get it back... Worth watching - Trust and the Linux
> development model; Greg KH [2].
> 
> Anyway I have no clue which identity to trust...
>

Again it's the same person, just a change in the name but I see your
point. As suggested in the other patch I will just change to Christian
Marangi sadly if necessary, hoping I regain trust in the future.

Just as an info, as I pushed also other patch and some are with various
revision, how to handle the Sob? Change it and put an alert in the
version changelog or should I keep the old one?

Again sorry for the mess but it had to be done, didn't think it caused
that much problems :(

> 
> [1]
> https://www.kernel.org/doc/html/latest/process/submitting-patches.html?highlight=certificate#sign-your-work-the-developer-s-certificate-of-origin
> 
> [2] https://www.youtube.com/watch?v=nhJqaZT94z0&t=7044s&ab_channel=hupstream
> 
> Best regards,
> Krzysztof

(if the concern is with understanding if I'm not stealing the identity
of someone else, there must be a way to prove that, on github I sign
every commit with the PGP key and I updated the name also there.)
diff mbox series

Patch

diff --git a/include/dt-bindings/clock/qcom,lcc-ipq806x.h b/include/dt-bindings/clock/qcom,lcc-ipq806x.h
index 25b92bbf0ab4..e0fb4acf4ba8 100644
--- a/include/dt-bindings/clock/qcom,lcc-ipq806x.h
+++ b/include/dt-bindings/clock/qcom,lcc-ipq806x.h
@@ -19,4 +19,6 @@ 
 #define SPDIF_CLK			10
 #define AHBIX_CLK			11
 
+#define LCC_PCM_RESET			0
+
 #endif