linux-kernel - Re: [PATCH] arm64: dts: microchip: add missing cache properties

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <41c72642-7d46-d7d1-d49a-6bef685f934c@linaro.org>
Date:   Wed, 17 May 2023 14:10:53 +0200
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Steen Hegelund <steen.hegelund@...rochip.com>
Cc:     Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Lars Povlsen <lars.povlsen@...rochip.com>,
        Daniel Machon <daniel.machon@...rochip.com>,
        UNGLinuxDriver@...rochip.com, devicetree@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: microchip: add missing cache properties

On 17/05/2023 13:38, Steen Hegelund wrote:
> Hi Krzysztof,
> 
> I would love to do that, but I am not familiar with the procedure, so maybe you
> could help me out?

Hm, there is no dedicated maintainer for Microchip ARM64 platforms? I
mean one which actually handles the patches?

It looks like it, because my recent changes were going through me. This
also means that maybe several other changes got ignored. For example:

https://lore.kernel.org/all/20230221105039.316819-2-robert.marko@sartura.hr/

https://lore.kernel.org/all/20220420194600.3416282-1-michael@walle.cc/

and others here:
https://lore.kernel.org/all/?q=dfn%3Aarch%2Farm64%2Fboot%2Fdts%2Fmicrochip%2F



> 
> This is my understanding of what I need to do:
> 
> Clone the upstream repo listed in MAINTAINERS:
> 
> git clone git@...hub.com:microchip-ung/linux-upstream.git
> cd linux-upstream
> git checkout sparx5-next
> 
> Fetch the latest mainline tag from upstream:
> 
> git fetch git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git tag\
>  v6.4-rc2 --no-tags
> 
> Rebase the current branch on top of that tag:
> 
> git rebase v6.4-rc2
> 
> Use b4 to fetch and apply the mail thread patch series:
> 
> b4 shazam -tsl 20230421223155.115339-1-krzysztof.kozlowski@...aro.org

You should collect some more patches... For one patch it is probably too
much effort. I can take it instead.

> 
> Tag the current work for inclusion in the next kernel version with a decription:
> 
> git tag -s sparx5-dt-6.5

git tag -a -s sparx5-dt-6.5

Because you need to provide some explanation. Take a look at examples
from other sub-arch maintainers what to write in the tag:

https://lore.kernel.org/soc/20230410170233.5931-1-andersson@kernel.org/T/#u

https://lore.kernel.org/soc/20230405080438.156805-1-krzysztof.kozlowski@linaro.org/T/

> 
> Push work that to the public repo:
> 
> git push origin sparx5-dt-6.5
> 
> Create a pull request (to stdout) to be included in an email to the maintainers:
> 
> git request-pull v6.4-rc2 origin sparx5-dt-6.5
> 
> Send this PR to the maintainers and CC co-maintainers.
> 
> Is this the correct procedure?
> Who should I send the PR email to (is there a list somewhere)?

Yes, it's correct with few nits I mentioned.

You send it to arm@, soc@, Olof and Arnd. Addresses are in examples above.

I will be preparing today the pull with various cleanups for arm-soc, so
I will take the patch if you do not mind.

For future (and all previous patches), please think what do you
(you=Microchip) want to do with it. If you do not handle the patches,
then someone should or the platform should be marked as "Odd fixes".

Best regards,
Krzysztof