[LEDE-DEV] [PATCH] brcm63xx: Add Comtrend AR-5315u support

Florian Fainelli f.fainelli at gmail.com
Wed Mar 1 11:56:22 PST 2017


On 03/01/2017 11:44 AM, Daniel wrote:
> 2017-03-01 19:46 GMT+01:00 Florian Fainelli <f.fainelli at gmail.com>:
>> On 03/01/2017 10:37 AM, Daniel Gonzalez Cabanelas wrote:
>>> Add support for the Comtrend AR-5315u router
>>>
>>> This a BCM6318 based board, 64 MB RAM, 16 MB SPI flash,
>>> with an onboard BCM43217 wifi, 4 ethernet ports and 1
>>> USB host port
>>>
>>> Signed-off-by: Daniel Gonzalez Cabanelas <dgcbueu at gmail.com>
>>
>> There is something wrong with your patches submitted, in that they don't
>> conform to this:
>>
>> subject: something
>>
>> Commit message
>>
>> Signed-off-by: myself
>> ---
>> <git diff>
>>
>> Essentially, we are missing the --- between your Signed-off-by tag and
>> the git diff, how are you producing these patches?
> 
> Hi Florian. For the patches I use this procedure
> 
> git add file1 file2
> git diff --cached > mypatch.patch

Woah that seems complicated, you are essentially not tracking your files
with git here, so rebasing, branching and so on is not going to work
quite well. Highly recommend you commit and rebase changes when you need to.

> 
> Then I copy the patch into my email client (KMail) and send it as a
> regular mail to the mailing list
> 
>> Can you make sure you
>> use git format-patch + git send-email and do a git commit -s  when
>> creating local changesets?
> 
> I guess this generates the patch with the required header?
> git add file1 file2
> git commit -s
> git format-patch -1 HEAD
> 
> only the stuff between --- and signed-off is missing, or is there
> anything else wrong?.

That's the only thing.

>  Just to be sure because I also want to continue
> using KMail for sending patches.

Why use KMail when git-send-email is a known good SMTP client? It won't
mangle your patches unlike what Kmail could do...
-- 
Florian



More information about the Lede-dev mailing list