NekoCWD/libcamera
NekoCWD/libcamera
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 7 Wiki Activity Actions

libcamera: mali-c55: Remove tpgSizes_ member from MaliC55CameraData

Browse source 
The tpgSizes_ vector is only used within the initTPGData() function.
Drop it and use a local variable instead.

Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Daniel Scally <dan.scally@ideasonboard.com>
This commit is contained in:
Daniel Scally 2025-02-20 22:05:32 +00:00
parent b55943714f
commit f58077f073
1 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

7
src/libcamera/pipeline/mali-c55/mali-c55.cpp
View file

@ -131,7 +131,6 @@ private:
void setSensorControls(const ControlList &sensorControls); void setSensorControls(const ControlList &sensorControls);
std::string id_; std::string id_;
std::vector<Size> tpgSizes_;
Size tpgResolution_; Size tpgResolution_;
}; };
@ -179,13 +178,15 @@ void MaliC55CameraData::initTPGData()
if (formats.empty()) if (formats.empty())
return; return;
std::vector<Size> tpgSizes;
for (const auto &format : formats) { for (const auto &format : formats) {
const std::vector<SizeRange> &ranges = format.second; const std::vector<SizeRange> &ranges = format.second;
std::transform(ranges.begin(), ranges.end(), std::back_inserter(tpgSizes_), std::transform(ranges.begin(), ranges.end(), std::back_inserter(tpgSizes),
[](const SizeRange &range) { return range.max; }); [](const SizeRange &range) { return range.max; });
} }
tpgResolution_ = tpgSizes_.back(); tpgResolution_ = tpgSizes.back();
} }
void MaliC55CameraData::setSensorControls(const ControlList &sensorControls) void MaliC55CameraData::setSensorControls(const ControlList &sensorControls)