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: <d2f7d5dd-cada-a3a2-2962-9c93d847f491@amd.com>
Date:   Tue, 25 Jul 2023 08:19:31 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.12.0
Subject: Re: [Enable Designware XGMAC VLAN Stripping Feature 1/2] dt-bindings:
 net: snps,dwmac: Add description for rx-vlan-offload
Content-Language: en-US
To:     Geert Uytterhoeven <geert@linux-m68k.org>,
        Joe Perches <joe@perches.com>
Cc:     Jakub Kicinski <kuba@kernel.org>,
        Krzysztof Kozlowski <krzk@kernel.org>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        workflows@vger.kernel.org,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
References: <20230721062617.9810-1-boon.khai.ng@intel.com>
 <20230721062617.9810-2-boon.khai.ng@intel.com>
 <e552cea3-abbb-93e3-4167-aebe979aac6b@kernel.org>
 <DM8PR11MB5751EAB220E28AECF6153522C13FA@DM8PR11MB5751.namprd11.prod.outlook.com>
 <8e2f9c5f-6249-4325-58b2-a14549eb105d@kernel.org>
 <20230721185557.199fb5b8@kernel.org>
 <c690776ce6fd247c2b2aeb805744d5779b6293ab.camel@perches.com>
 <20230724180428.783866cc@kernel.org>
 <213f5d2b13225f9ed4bdadda3c492ffc79940b13.camel@perches.com>
 <CAMuHMdX0DpNSDRSJ7D85NGV2ri2kFHjTtXJE5aOaTD3jNoyc-w@mail.gmail.com>
From:   Mario Limonciello <mario.limonciello@amd.com>
In-Reply-To: <CAMuHMdX0DpNSDRSJ7D85NGV2ri2kFHjTtXJE5aOaTD3jNoyc-w@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TjdPUHhtU3d5Zi9pSzlPcDlleDNaR0ZwNng4ZjJJZ29NbWtiK0hPVFFGRHcr?=
 =?utf-8?B?ZnB0MFJIdXhBd3lGZVcyV3lCczFqaEpNdnZ6L2RudCtHS2tqRTUyRXBIS2F6?=
 =?utf-8?B?Tnd2WjJDa08rck5EL21GcjArQUpXM3NIQUI2cjBzMm9NZHFoK1hNRXF2ck4w?=
 =?utf-8?B?R2tWbEJ5ZkVDTVR0RzBsSFd5YWJFK3ZnRGpwOXBPMTFyYzNzd3ZjamF5UGs1?=
 =?utf-8?B?azdjMWVmV2V6OXNXR3lpWmlMQ05MRk5rM1Q1ZzQ2U1JzTUxLWHN1QWxMVmsy?=
 =?utf-8?B?WGRJajhtRUdKWGV6NnoySWduU0QrWjJRak4yMWV5ZEJrbGJrdjd3cm1tTlZN?=
 =?utf-8?B?cUJvcExEYU52MFBZa2lqYlp1bTN3dUxDbE8rR2E3dzFTOTNLbVI2MmRDVldI?=
 =?utf-8?B?allzUlo3N2xyRjg4R1VoalgrSEZOVnhsRGlDV0RlSHgwVkIwOFRlYWVMRzdo?=
 =?utf-8?B?eVVvRG5veXhqcjV5YnpIeG4xMGVNTmRteU54aE1oV3U4R21UeEhMa2wzaDky?=
 =?utf-8?B?TVMybkttSnhGTjI1b09KSzdOemRFMEt3MFdyWGNGVmpMbEtsamM4TUdCc0Fx?=
 =?utf-8?B?aVN2QUhoRE5rcEdneWY3M1RBYUpPbVQrME9NOEcrZXZsSkprTmM5UDRzcDJj?=
 =?utf-8?B?eGFKamhKUWxxSjkwVVdMeVM5WDZ0bVlEai9TM1o1azRwZDRUc1ZiTmhnZ0RS?=
 =?utf-8?B?MHNsaWxLcEh0Wlc3bXJmYWZYeTVjWDJlS3AwSUtxRlg3dHRzaVZGVkNqSHR2?=
 =?utf-8?B?ZjA1dGV3cmVoSzByckpzcGIrVXFNMnE0MnFvYU1sLzFDbW00dUJETCtrWG4v?=
 =?utf-8?B?bVNMcTVpckJVZmh5NDRYWUtlMkxHQ2FlRTZGek5rMTR5WXBxNUJKejhQSzFR?=
 =?utf-8?B?b1BLcmNPL2I1WjQxR1c5K2x3c2JnVWJicnhqSWVVMGEvWFFzRWMxRUdndVBz?=
 =?utf-8?B?dkc5bHNNWDE1UXZpNnA0TXJ3a3lnd0lSalJ4YWEvNkovaTQ4REZVOG53Y2JS?=
 =?utf-8?B?dUh3YU55enhyVHNDLys5OVlMOTJsRjduK2llNzhqbHIyS3hiREJ1WFNZb1pn?=
 =?utf-8?B?TG9yY1V6V0NCMG1qRElKOTgwb25lYk8xTUtDMkZUREdHakYwVXlYOUFrYjlW?=
 =?utf-8?B?UmxTWGxWNWhOckNpelRSR2JsTm1hc2pxb1RoWldvMDRIRnZuazZRWHd4MWdM?=
 =?utf-8?B?cVRnOWRYSnRxMEN1dVQyZ3lRRnJHdGNpVTRjdTBDSFA4T293QlRxWm5hemZG?=
 =?utf-8?B?U2Y1eVRNZjRtbzFMODBHOVIxam9yM3loUE9vclFoYjZ3RTZKWVBnM1ZtamQ2?=
 =?utf-8?B?M2M2QW1zaVcwL0w5MklqYXE5eVEreVRrUGVXMFREYk9XdkNkUDBLOFY2ZzFu?=
 =?utf-8?B?R0dLR1g3MnBpUHVvejZhNFJ6VnFWaml2VXc1alJCWGlaclROdW9OMG4wekk3?=
 =?utf-8?B?QjV2WTNBKzZBaGkrLy9MTk8yT1ZCYjF4Z2p2T3FuT01ibVh1U09CN1dqNXFB?=
 =?utf-8?B?SFBuME5sbE1MVnlVYWFBbGhEQ2thRm1xYzU2TTBycnpaQlFORE1PRWx6dFJX?=
 =?utf-8?B?VDJNV1hST0p6dEc5WXB5aEQ5NnJ2czRTdXBVOVlmcGFQb1JORUVuWS9OREFE?=
 =?utf-8?B?L28zKy9HQTNqSDlUcWVkSnk1NDBkWUFabHFzUUcyWkNOdXhBL3hxekUyZmdP?=
 =?utf-8?B?Wlhxc3dsOEcyTjVmNHNVYkx4VVF1V3JraW1OQUwyVHE5bUhUKzJzVFJ0YXJv?=
 =?utf-8?B?TFkwM0xyeVFZVFhHNXN5cGtLK1JKTWlpcVorZ0RTb0JIcDdaR1VvT0RlZDFK?=
 =?utf-8?B?ZUhUSUdTZHZ5RXE1ZkRnVisyaHJDSTM0YWk5T3FUMVhHZlRpQWROekNFcFpK?=
 =?utf-8?B?c2pqZjBrZk9uMU1sS1FYSlZNWloxTFIreXdyeEtPcTh1a2dCYWJ0WkNDMEt0?=
 =?utf-8?B?ZU05WFRCU050K2NOQ1p1eFh1RDVobVhxWVh2MExUMXJ5eGNMb2planlvQ1Bp?=
 =?utf-8?B?azh2c2pFdHFGRFF3MkEwTXlocm5NN3psamRFNy9DLzcwTy9SL3FFdWU1WXlI?=
 =?utf-8?B?aUxOeC9DdVU2UUhzQW1KNndzRlZWcnd2Ym1DN0hWOENVZ2FuTE9wNGJzMC96?=
 =?utf-8?B?NGMyUkcvTWkxbE40MjVrNjRhc2dORTA2eXVqWkNnSFVNRWlLUjRWQ1VFTGNT?=
 =?utf-8?Q?IWpQVJ8bqqWhnPF1DXDs4DIrTwEyDgVXdf9J8q+JxTEH?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b568cfea-9756-4d73-8d88-08db8d11cd4b
X-MS-Exchange-CrossTenant-AuthSource: MN0PR12MB6101.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Jul 2023 13:19:40.2699
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: S0T3wfRDg3C45V60ZdksJ8Ngtgo5lc1WK8cu+bXYXNqqMZL9SaKFk0ZeIZEvttTG/KjkIu3b61GsG1XHo72mUA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR12MB8436
X-Spam-Status: No, score=-1.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,NICE_REPLY_A,
        RCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=no autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 7/25/23 02:33, Geert Uytterhoeven wrote:
> Hi Joe,
> 
> On Tue, Jul 25, 2023 at 6:22 AM Joe Perches <joe@perches.com> wrote:
>> I do suggest you instead write wrapper scripts to get
>> the output you want rather than updating the defaults
>> for the script and update the process documentation
>> to let other people know what do to as well.
>>
>> Something akin to Mario Limonciello's suggestion back in 2022:
>>
>> https://lore.kernel.org/lkml/20220617183215.25917-1-mario.limonciello@amd.com/
> 
> FTR, this is more or less what I am using to generate a script
> to send out patches:
> 
>      OUT=...
>      echo git send-email \\ > $OUT
>      # Add -cc
>      # Wrap comment inside $(: ...)
>      # Replace (...) in comment by [...]
>      # Replace ] at EOL by ) again
>      # Add continuation to EOL
>      scripts/get_maintainer.pl $* | \
>      tr -d \" | \
>      sed -e 's/^/--cc "/' \
>          -e 's/ (/" $(: /' \
>          -e 's/ (/ [/' -e 's/)/]/' \
>          -e 's/]$/)/' \
>          -e 's/$/ \\/' | \
>      tee -a $OUT
>      echo "*[0-9][0-9][0-9][0-9]-*.*" >> $OUT
> 
> After generation, I edit the script to
>    - Replace some --cc by --to,
>    - Add/remove some people,
> and run "source $OUT" to send the patches...
> 
> Gr{oetje,eeting}s,

My script is great for single subsystem patches as it gets all the right 
people but I've found problems whenever it crosses multiple subsystems.

Many subsystem owners want to see the whole series of patches to 
understand how they interact.  So the group of patches needs to be 
treated together which would need the wrapper to look at all patches 
instead.