Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH 2/2] usb: cdc-wdm: Constify static struct wwan_port_ops
To:     Rikard Falkeborn <rikard.falkeborn@gmail.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Loic Poulain <loic.poulain@linaro.org>,
        Wei Yongjun <weiyongjun1@huawei.com>
Cc:     Oliver Neukum <oneukum@suse.com>,
        Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp>,
        Junlin Yang <yangjunlin@yulong.com>, linux-usb@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20210929132143.36822-1-rikard.falkeborn@gmail.com>
 <20210929132143.36822-3-rikard.falkeborn@gmail.com>
From:   Oliver Neukum <oneukum@suse.com>
Message-ID: <053aca41-e67d-a64d-14ed-ed5bfcc50b67@suse.com>
Date:   Thu, 30 Sep 2021 11:08:10 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.12.0
In-Reply-To: <20210929132143.36822-3-rikard.falkeborn@gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Content-Language: en-US
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?R1VUWGNSbWR1QTNvK1kxUmsrV0h5Vkc3Y2p2NW56dmFNMzFxNWFBZStrdmFi?=
 =?utf-8?B?ckFvWlRkMUhJSWJOWUxIcFhzOEpTWHIwVnhzMExRL1plQkxqeTR0QWJUdXEr?=
 =?utf-8?B?TTA5dHkza3dSMFcvbFlENFVreVh2dE5Ib3JFK0cxckxMaU11L25RYWF2dDFC?=
 =?utf-8?B?REdRVU8xSitxdGZpRmRCbWRsWW40SElwc3dVTEM1bWZaTTU4YVR0TmdobFhn?=
 =?utf-8?B?UEZNWGlLYnM3NXdkSmVJVWpxZFovdXZpTW91ck5nRHRTb2V5ZEYvaFhxWnNn?=
 =?utf-8?B?T21veUJiY0s4a0s1Wm91UW9Rb3ZlU0dEMVBXTzlSdlZ1NThKRVZLUWU0NFBq?=
 =?utf-8?B?R0U3WlF0V3FNNzBNcEMwaXhCa0FFNVZMT1haVkhsS20wajRWbVNaWVlZOXBa?=
 =?utf-8?B?QzRidVJnS2lCZnZrS1l0U2pSaHU4UHYvNGZUaGkzbFlKSUhBYUxldW10QVBB?=
 =?utf-8?B?N24rTm9KaFlzQ2pET3MxSmsxSzlWb0JwR25iQjZFSFBwUG13SjFFYXZsVjdD?=
 =?utf-8?B?NUd5SkRqWWl1N1UrRWQvMUhXN1lmZmRKY1IrTHJFUkhUUStJNlRlM1RaMDli?=
 =?utf-8?B?RWhhMXlzWldKRWJlOGFLVU5HT1hoWWhGY3FCMnFsNit4UTJzWURmQ0dIT0M1?=
 =?utf-8?B?SHh3eDlwVmc5TFJjRjFHV3lha2gzQkVXVlR6RXRnL2NpKzV4UmJBc2xEMGh2?=
 =?utf-8?B?enI0QjF4YjlhWDlHOWZMQUR1TCt6ZDM0d3drdldsZVRubHB2NGVUVUZKSHJT?=
 =?utf-8?B?TUdjWmNJMkg1YzhocUtKdmVyaHVGTUtWaXJVZ3ZjVjgzUFc4ek9lL3pJL2ph?=
 =?utf-8?B?U2ZFWldvZHlmNU9KOXdxQ0VDTFdVQUtLTjRGOHg1VVpWQjY4bi9aSi9HRjJm?=
 =?utf-8?B?TmxWL1N0ZVpZMEM4OHEwelk0anh1ZTNqcURiNU5OZ0Iwdm4ydEg2UDBNNDVZ?=
 =?utf-8?B?d2p1RFBZZ0ZMNTd1UWRpUGV0TEp5NTlpV1FOK1FudG1BZmZUM1ZyUGZaQy9T?=
 =?utf-8?B?RjVVMityS0E2UUZ6QVo0NXhVbVZBOHpuTkJvblk4ck9wbXlNL1ZOM3BOSGVR?=
 =?utf-8?B?a3ZWTnYyeFp4Nzg4Yk13YnVCbDlZeFRhc291NlpBT0MrZVJuZThWMkU4MnJW?=
 =?utf-8?B?a2pON2YxNkFaL21PU2RkRnFjS3dyV3YycG1PSmNaZ3A5cDNUamJEMHptQkZE?=
 =?utf-8?B?ZDVCSXNlNWVQSUdmdW9SNHpmQWh1WCtjNnRZaDY1TG5tVlJySGUzUjBLbjJu?=
 =?utf-8?B?T1ZkRnIyZTNRMG9TTnZMZ2tiTTE1ZG5PZTFBOHVlaXJtQ25KcEVEbzVxQ3d2?=
 =?utf-8?B?cnlPMEMxZ04wMHZ6Skc2RjZPNGZKY3F1bis2OHoxWGNaUkxPNzBqekNTYTVt?=
 =?utf-8?B?MzNxY3dqNWNWUUFHRWdkMys1SlFYNW0rd3EvZ0YwOVJHVW9QQjhXelFTT3NO?=
 =?utf-8?B?a1hPQTdmS2JFV0kwa2crUHBoQzVEN1FoNFkyU1Z0RWhmaERuVzRrVGtNWHQ1?=
 =?utf-8?B?aXFVYmpVa1VtUDEyVmFxaDBpTmpzTzRXcS9lMjRFc0RXcUFzbExYVXcwWGNM?=
 =?utf-8?B?d2hLQ3B1bll4dFQvb2xtK282OUtKUHpKdW0rTDNqUnR0RG9Ld1JjcUpnNjlo?=
 =?utf-8?B?VUVSTkFpamZNbjRBTzlQTzFrQlFCRGVpb1FuMjlXUHdvRklveCtFR3NYMG01?=
 =?utf-8?B?UkJkMHpRanVZS091YUNveCtBa3JGM0xzWXJwUVVVWmpaZi9Dek93UE1zcUM5?=
 =?utf-8?B?ZjQrbGNXbk0xNWEvWWVSbFZMWWJoK3hPOHIrSVNtUkRzME82RzRXV28yRFNs?=
 =?utf-8?B?SzhNY2Z6RGJJekkwQkN3bGhWWmxWQUNVUzRXeCthYTROVzBDSVZxRTY0S1A4?=
 =?utf-8?Q?l7Kd5ksMTYKRG?=
X-OriginatorOrg: suse.com
X-MS-Exchange-CrossTenant-Network-Message-Id: af692e38-0fd4-45aa-66c2-08d983f1d44e
X-MS-Exchange-CrossTenant-AuthSource: DB7PR04MB5050.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Sep 2021 09:08:12.3419
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: waOPu6GzpdP9WkUI1QGAI6ofs08GkeUcga9bK8SNU24ubyh051XNXt/Pjdw8laN//CK2eMMPnCrWGLkGD1pcUQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB7PR04MB4137
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 29.09.21 15:21, Rikard Falkeborn wrote:
> The only usage of wdm_wwan_port_ops is to pass its address to
> wwan_create_port() which takes a pointer to const wwan_port_ops as
> argument. Make it const to allow the compiler to put it in read-only
> memory.
>
> Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>
Acked-by: Oliver Neukum <oneukum@suse.com>