On 22/09/2023 16:11, Stephan Gerhold wrote:
Add "regulator" to the node names of the fixed regulators, and drop the "_rear" part of the camera node name since it is not part of the class of the device (which is simply "camera"). Signed-off-by: Stephan Gerhold <stephan@xxxxxxxxxxx> ---
&cci_i2c0 { - camera_rear@3b { + camera@3b { compatible = "ovti,ov5640"; reg = <0x3b>;
You could consider a follow-up patch here along the lines of orientation = <2> to indicate the camera is External. This shows up with a pretty name in libcamera based applications then. Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>