Hi,
On Thu, Sep 15, 2016 at 02:22:23PM +0300, Sakari Ailus wrote:
> The information gathered during frame format reading will be required
> earlier in the initialisation when it was available. Also return an error
> if frame format cannot be obtained.
>
> Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
> ---
> drivers/media/i2c/smiapp/smiapp-core.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/i2c/smiapp/smiapp-core.c
> b/drivers/media/i2c/smiapp/smiapp-core.c
> index 0b5671c..c9aee83 100644
> --- a/drivers/media/i2c/smiapp/smiapp-core.c
> +++ b/drivers/media/i2c/smiapp/smiapp-core.c
> @@ -2890,6 +2890,12 @@ static int smiapp_probe(struct i2c_client *client,
> goto out_power_off;
> }
>
> + rval = smiapp_read_frame_fmt(sensor);
> + if (rval) {
> + rval = -ENODEV;
> + goto out_power_off;
> + }
> +
> /*
> * Handle Sensor Module orientation on the board.
> *
> @@ -3013,8 +3019,6 @@ static int smiapp_probe(struct i2c_client *client,
>
> sensor->pixel_array->sd.entity.function = MEDIA_ENT_F_CAM_SENSOR;
>
> - /* final steps */
> - smiapp_read_frame_fmt(sensor);
> rval = smiapp_init_controls(sensor);
> if (rval < 0)
> goto out_cleanup;Is this missing a Fixes tag, or will it only be required earlier for future patches? -- Sebastian
signature.asc
Description: PGP signature
