mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-10 22:22:51 -04:00
gpu: nova-core: vbios: use size_of instead of magic number
12 is identical to the value of `size_of::<BitHeader>()`, so use the latter instead. [acourbot@nvidia.com: remove `dead_code` expect to fix `unfulfilled_lint_expectations` lint] Reviewed-by: Alexandre Courbot <acourbot@nvidia.com> Signed-off-by: Rhys Lloyd <krakow20@gmail.com> Link: https://lore.kernel.org/r/20250718073633.194032-2-krakow20@gmail.com Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
This commit is contained in:
committed by
Alexandre Courbot
parent
8f5ae30d69
commit
0f85b6c1cb
@@ -345,7 +345,7 @@ fn image_size_bytes(&self) -> usize {
|
||||
/// its header) is in the [`PciAtBiosImage`] and the falcon data it is pointing to is in the
|
||||
/// [`FwSecBiosImage`].
|
||||
#[derive(Debug, Clone, Copy)]
|
||||
#[expect(dead_code)]
|
||||
#[repr(C)]
|
||||
struct BitHeader {
|
||||
/// 0h: BIT Header Identifier (BMP=0x7FFF/BIT=0xB8FF)
|
||||
id: u16,
|
||||
@@ -365,7 +365,7 @@ struct BitHeader {
|
||||
|
||||
impl BitHeader {
|
||||
fn new(data: &[u8]) -> Result<Self> {
|
||||
if data.len() < 12 {
|
||||
if data.len() < core::mem::size_of::<Self>() {
|
||||
return Err(EINVAL);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user