Re: [PATCH] media: uvcvideo: Make uvc_event_control attribute name array const

Ricardo Ribalda <[email protected]>
Newsgroups org.kernel.vger.linux-media
Message-ID <CANiDSCsZPwGBk5GzmSdK41fbfch=1=3dB=poH1iNoFgkzL6orQ@mail.gmail.com>
hi Shagor


On Sat, 8 Aug 2026 at 11:16, Shagor Khan <[email protected]> wrote:
>
> hey Ricardo,
>
> sorry to interrupt but it might be that the mail I've previously was left behind, it would be really kind if you see my previous mail about ensuring that there is no variables left that could be constified like that, I would be great if you accept the patch even if it's a very little one, I'd be grateful to become a little kernel contributor :p

Just give it a couple of weeks and if Hans or Laurent like the patch
they will take it.

Regards!

>
> Thanks
>
> On Thu, Aug 6, 2026, 15:45 イムティヤズ <[email protected]> wrote:
>>
>> Hi Ricardo,
>>
>> Thanks for the review.
>>
>> I went through the whole uvc driver looking for other variables that
>> could be constified the same way. The attrs array in uvc_status.c is
>> the only one of this kind (an array of string pointers). The other
>> data tables in the driver (exposure_auto_mapping, cros_colorfx_mapping
>> in uvc_ctrl.c) are already declared const, so there is no remaining
>> group of changes to batch together.
>>
>> Thanks,
>> イムティヤズ



-- 
Ricardo Ribalda
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.