Received-SPF: pass (google.com: domain of linux-kernel+bounces-59471-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <465b2b4d-3c6e-4a64-b3be-f146cb237bda@vaisala.com>
Date: Fri, 9 Feb 2024 16:31:56 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v3 1/2] usb: core: Amend initial authorized_default value
Content-Language: en-US
To: Greg KH <gregkh@linuxfoundation.org>
Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
 vesa.jaaskelainen@vaisala.com, geert@linux-m68k.org
References: <20240209094651.5379-1-niko.mauno@vaisala.com>
 <2024020903-october-placidly-20cc@gregkh>
 <65582610-72ba-4644-9032-91c09f50a6bf@vaisala.com>
 <2024020911-utilize-etching-595a@gregkh>
From: Niko Mauno <niko.mauno@vaisala.com>
In-Reply-To: <2024020911-utilize-etching-595a@gregkh>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?YVhnUGFRV3V5M0JDVE0zQTZ2U1NGVlMxU0N4YzJvQ21SbkloOEVmVEk4T0NQ?=
 =?utf-8?B?SEgwMi95eG9YYmdRRS9XNkkzaVpqTnN0ay9PYjVwNTEzK1krbzBYNXQ0Tm5q?=
 =?utf-8?B?UDVtaEtxbStjK0VMSlRFWDdnZ0ttc1dHVG1mQVp4b2hrUitaNmN1RjNxQllM?=
 =?utf-8?B?RnRiaFNHTDZWSGpkQ21iTkVRd2Y2N25zdkJmV3hLVXVUTVFwdTdRN1BWaGpo?=
 =?utf-8?B?MktpYTJwblRnTVBBcitROGJxOGQ3MzBZck9HUExrUXk2VnA1aldUQ0V0OTNW?=
 =?utf-8?B?d0JzVmZGN0VXTUFtaTZzRVVvcEF4YlhSOUVDNDBETXlHZUx2ZDA0MFJScG0z?=
 =?utf-8?B?d2RDU1hQNS9HcllockY0dG05RTJlZ2lrUWNxUEhaU2szaWEyd2xYUG96dFJF?=
 =?utf-8?B?MzR0czIzcm5UU1V6QmptRkl2NTB5OWV6RTcvRFI3R0ZIMmF3NjJzVVkzMW1Q?=
 =?utf-8?B?a0dUZHlOYTBsT1lscGwzZnpmbEtPdm51M291ZHIxSDgyZUVoMEozcXI0UVgv?=
 =?utf-8?B?MUpJUkh0L3JUU2dQejh1MXNVNm9wd3ppaTY1ckJzNXV6TGhjc21yQksxaElW?=
 =?utf-8?B?YUlodW5vSGEyaWxYZ1FkcUpjbnZFck41c1RJQ1BXbGk0U0xkU3RkemF2VEVQ?=
 =?utf-8?B?UDhRMnl2UERyMjhYWXJXVU1RcUZ1dXRmdGJ3WTRYM3JsaVVzbDhuMTVYNDlx?=
 =?utf-8?B?Q09PRGRWN3JPMFNFME4vUExmcE5TR0ZXVWdEMzd3RXZrT25PbDFPK21NeFV3?=
 =?utf-8?B?NTRMcjMyUlRSZklya3RPRjhLWVh3SXZDSStRMWxrbEpSK3F6L1NHZ3oyZS9l?=
 =?utf-8?B?UHprYUdOYUV2dmplVWlMM0pXRU14RnVDU2hoRlI2MUZDbmNXaXNocW00aDZW?=
 =?utf-8?B?cjBnZ1ppWHUrQ2dPdWlrVy9OUjBoOWsyRC9mOUdmcjZjSG0zNDZJQVRJcjh3?=
 =?utf-8?B?N04xd2ZEdlRzalVuMlFTZStDS1V5MVpHQTlrV3VLeXFrWEhVVDhka3gyMlFZ?=
 =?utf-8?B?Y0xYa0ladlBWekV4TWMxc2l6bjU4Y2VkNW0wSVRJalNxSTdqdVI2NmlaaVdX?=
 =?utf-8?B?dHZyNDhSSVkxWG5GWnVZZ1ZjR0p4UjZ4aU83bkpBR01kSXBqelVpWFVLRWxz?=
 =?utf-8?B?clBwdkNXMU4zY3kxRlh0QmhpR204aXZCQ2o5VkVONXFVN2szcjFYemMvcDRY?=
 =?utf-8?B?OTZLYzRRcFZpV1EyeHU2czRhTnBqQ2pSWG5QTVVlb3dYWGhJM1E0am00OEpu?=
 =?utf-8?B?dERKRFdaY0pkZGtsTXRJRWd6OXh2UGFHdEl2VjlVdU9FOEMvdDcrWjVqNGJp?=
 =?utf-8?B?ZzgwbElsUFN2MnkyUm1FV2RGWTZSbTBXeFdVQmU0a2NGdTlIaTFCM2llVUtC?=
 =?utf-8?B?Nk9aazlMazFSQjk5YWEvSG9FRkd0TlZKQzByMkJzSHJESzhlOVdIUUFYNHRa?=
 =?utf-8?B?UlJpbnl6UzlaYzdaUzNUbjBUZ0VYR0hMaGxDNlhiWTVWVEZMaGJuRVBIODVR?=
 =?utf-8?B?Uy9qWlRoK2E2ai9lN01ta3NZN09Yd2VYYjdMNUdidnA5MGR2NG40SzlXZ0l6?=
 =?utf-8?B?VlVRd05OOFppc0tqTWN1T2JhUThlUm1CbW1aOXFEYVMzVUd2ZStkc2FuTS9v?=
 =?utf-8?B?VTdUUFN1YU02SjJwamw4VWd0TkJWemxsS0p3d3NHSXdsUXdCQU9aVjZTTnVj?=
 =?utf-8?B?YWpIVjgvNXNuOWFxWW44NXU0UmpQZ0NuUGtNN1NhS3ZHNUw0YkwrY1dnOU04?=
 =?utf-8?B?NnYvTHl5TGZCOHZSclVXWXE0cWNWN01USEZSRU1xY1ZnQWp5MDBIa1o5a3lQ?=
 =?utf-8?B?SkY2bm9ObEZFWlp6OThzNCtoNzAwR2I5MGVCYVJYUTNFRjhYbEUyNldKVmV1?=
 =?utf-8?B?dmgrbEdtNVhRMHlVVkFCbnZraUNIeG1VK0xRR2JJdUVvUXY5YnRLZHlZOGFI?=
 =?utf-8?B?OTlKVEZwTGZDOE1vVWdrcVlhczdhWkl2Yk82bGhXdVRCRjdjODNMTzIzdDNz?=
 =?utf-8?B?QnNyMFVDbDBFdGVDSFhIMEtPZWlaaDdESXRmQlpGVUhZNDlkRW9uMnVqOHBL?=
 =?utf-8?B?bHBKUHFGVTREeUxPRFJvT3JjU05SYWhjQW9KbldLS09rMk5PR2Vpa3JhVU9r?=
 =?utf-8?B?dHhjeE9PZTJnd3UzZUNCeWF2ZzlCZWZHdmFJYUJxRU5YQ2k5ZFdyUG9tN2sz?=
 =?utf-8?B?WUE9PQ==?=
X-OriginatorOrg: vaisala.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 67fcd46b-d539-43bd-7e4b-08dc297bdf11
X-MS-Exchange-CrossTenant-AuthSource: AS4PR06MB8447.eurprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Feb 2024 14:31:58.1069
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 6d7393e0-41f5-4c2e-9b12-4c2be5da5c57
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: xiSftkqz0VTSJaL46cA6nX+7c9niyYRx8dK8WDDJ3KvPxk7vjmynzGkJBS87PUCm/iBu4eA+KKv1d9R98gKsEg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR06MB7386

On 9.2.2024 15.34, Greg KH wrote:
> On Fri, Feb 09, 2024 at 02:39:01PM +0200, Niko Mauno wrote:
>> On 9.2.2024 12.47, Greg KH wrote:
>>> On Fri, Feb 09, 2024 at 11:46:50AM +0200, niko.mauno@vaisala.com wrote:
>>>> From: Niko Mauno <niko.mauno@vaisala.com>
>>>>
>>>> Since the wireless USB implementation has been removed and since the
>>>> behavior with authorized_default values -1 and 1 is now effectively
>>>> same, change the initial value to latter in order to stop using the
>>>> leftover value. The former value can still be passed as a module
>>>> parameter to retain backwards compatibility.
>>>>
>>>> Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
>>>> ---
>>>>    drivers/usb/core/hcd.c | 6 ++----
>>>>    1 file changed, 2 insertions(+), 4 deletions(-)
>>>
>>> Isn't this series already in my tree?
>>>
>>> confused,
>>>
>>> greg k-h
>>
>> Hi Greg, apologies for any confusion I may have caused,
>>
>> Geert requested some further clarification in the new Kconfig help section,
>> please see:
>> https://marc.info/?l=linux-usb&m=170747246528305&w=2
> 
> Great, but I can't take patches that are already in my tree, right?
> 
> Please submit a new one on top of what I currently have.
> 
>> (There shouldn't be any functional change introduced between v2 and v3)
> 
> That always needs to be listed, as my bot says, otherwise we have no
> idea what is going on here.  Remember, some of us get 1000+ emails a day
> to do something with (review, apply, delete, ignore...)
> 
> thanks,
> 
> greg k-h

Thanks, submitted as new patch against usb-next.
-Niko