Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <41c72642-7d46-d7d1-d49a-6bef685f934c@linaro.org>
Date:   Wed, 17 May 2023 14:10:53 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.11.0
Subject: Re: [PATCH] arm64: dts: microchip: add missing cache properties
To:     Steen Hegelund <steen.hegelund@microchip.com>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Lars Povlsen <lars.povlsen@microchip.com>,
        Daniel Machon <daniel.machon@microchip.com>,
        UNGLinuxDriver@microchip.com, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
References: <20230421223155.115339-1-krzysztof.kozlowski@linaro.org>
 <151169f7-dcdb-47f1-9616-67c3c388a233@linaro.org>
 <7772a5981d9a3d46056a53997162d1d75d22a701.camel@microchip.com>
Content-Language: en-US
From:   Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
In-Reply-To: <7772a5981d9a3d46056a53997162d1d75d22a701.camel@microchip.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

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@github.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@linaro.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