RE: [EXT] Re: [PATCH] media: amphion: fix some issues reported by coverity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



>From: Markus Elfring <Markus.Elfring@xxxxxx>
>Sent: 2023年7月18日 1:26
>To: Ming Qian <ming.qian@xxxxxxx>; linux-media@xxxxxxxxxxxxxxx; kernel-
>janitors@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; dl-linux-imx
><linux-imx@xxxxxxx>; kernel@xxxxxxxxxxxxxx; Fabio Estevam
><festevam@xxxxxxxxx>; Hans Verkuil <hverkuil-cisco@xxxxxxxxx>; Mauro
>Carvalho Chehab <mchehab@xxxxxxxxxx>; Nicolas Dufresne
><nicolas@xxxxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Sascha Hauer
><s.hauer@xxxxxxxxxxxxxx>; Shawn Guo <shawnguo@xxxxxxxxxx>; Tao Jiang
><tao.jiang_2@xxxxxxx>; X.H. Bao <xiahong.bao@xxxxxxx>; Eagle Zhou
><eagle.zhou@xxxxxxx>
>Cc: LKML <linux-kernel@xxxxxxxxxxxxxxx>
>Subject: [EXT] Re: [PATCH] media: amphion: fix some issues reported by
>coverity
>
>Caution: This is an external email. Please take care when clicking links or
>opening attachments. When in doubt, report the message using the 'Report
>this email' button
>
>
>> CHECKED_RETURN: 4 case
>> REVERSE_INULL: 2 case
>> UNINIT: 6 case
>> UNUSED_VALUE: 1 case
>
>How does such a change combination fit to the development requirement
>“Solve only one problem per patch.”?
>
>See also:
>https://git.ker/
>nel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ftorvalds%2Flinux.git%2Ft
>ree%2FDocumentation%2Fprocess%2Fsubmitting-patches.rst%3Fh%3Dv6.5-
>rc2%23n81&data=05%7C01%7Cming.qian%40nxp.com%7C94575f499df74c702
>d8308db86ead678%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C63
>8252115406304424%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAi
>LCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sd
>ata=nr7bqZMPNvrECSOMJsCzn3aU8UWdinBG4sHRRGRJJ1E%3D&reserved=0
>
>
>Will further imperative change descriptions become more helpful?
>
>Regards,
>Markus

Hi Markus,
    Thanks for your comments, I'll split this patch into 4 patches, one topic one patch.

Ming





[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux