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: <a5380bc2-08f9-1f09-343b-0e1a86693721@gmail.com>
Date:   Thu, 13 Oct 2022 11:20:51 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCH] MAINTAINERS: Move from git://github.com to
 https://github.com
Content-Language: en-US
To:     Conor Dooley <conor@kernel.org>,
        Palmer Dabbelt <palmer@rivosinc.com>,
        Scott Branden <sbranden@broadcom.com>
Cc:     linux-kernel@vger.kernel.org
References: <20221013153258.28228-1-palmer@rivosinc.com>
 <Y0hEt51q+Fs0ICbB@spud>
From:   Florian Fainelli <f.fainelli@gmail.com>
In-Reply-To: <Y0hEt51q+Fs0ICbB@spud>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 10/13/22 10:02, Conor Dooley wrote:
> On Thu, Oct 13, 2022 at 08:32:58AM -0700, Palmer Dabbelt wrote:
>> Github deprecated the git:// links about a year ago, but it looks like
>> there's still a handful of them in the MAINTAINERS file.  Conor pointed
>> this out about the RISC-V KVM tree, but I figured it'd be better to just
>> fix them all -- I've got a bunch of insteadOf so I didn't even notice
>> the deprecation, but new contributors probably don't and might get a bit
>> confused.
>>
>> Reported-by: Conor Dooley <conor@kernel.org>
>> Link: https://github.blog/2021-09-01-improving-git-protocol-security-github/
>> Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>
>> ---
>> @@ -3990,7 +3990,7 @@ M:	Ray Jui <rjui@broadcom.com>
>>   M:	Scott Branden <sbranden@broadcom.com>
>>   R:	Broadcom internal kernel review list <bcm-kernel-feedback-list@broadcom.com>
>>   S:	Maintained
>> -T:	git git://github.com/broadcom/mach-bcm
>> +T:	git https://github.com/broadcom/mach-bcm
> 
> This link is dead, I can't find a repo with that name on their GitHub
> profile.
> @Florian, should this be updated to the stblinux repo instead?

Yes this should be updated to the stblinux repository, thanks! If you 
want to split on a per-maintainer basis, I can take the Broadcom entries 
separately  through the Broadcom ARM SoC pull request, else:

Acked-by: Florian Fainelli <f.fainelli@gmail.com>
-- 
Florian