Hi Jacopo,
On 2018-05-16 14:16:56 +0200, Jacopo Mondi wrote:
> Add R-Car R8A77995 SoC to the rcar-vin supported ones.
>
> Signed-off-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx>
> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> ---
> drivers/media/platform/rcar-vin/rcar-core.c | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/drivers/media/platform/rcar-vin/rcar-core.c b/drivers/media/platform/rcar-vin/rcar-core.c
> index ea74c55..fba8610 100644
> --- a/drivers/media/platform/rcar-vin/rcar-core.c
> +++ b/drivers/media/platform/rcar-vin/rcar-core.c
> @@ -1104,6 +1104,10 @@ static const struct rvin_group_route _rcar_info_r8a77970_routes[] = {
> { /* Sentinel */ }
> };
>
> +static const struct rvin_group_route _rcar_info_r8a77995_routes[] = {
Nitpick and I see that the error is not yours but present in other but
not all route declarations. But the intention is not to have a leading _
here. A patch to clean that up for _rcar_info_r8a77965_routes[] and
_rcar_info_r8a77970_routes[] would be welcome, or I can do it if you are
over loaded.
> + { /* Sentinel */ }
> +};
> +
> static const struct rvin_info rcar_info_r8a77970 = {
> .model = RCAR_GEN3,
> .use_mc = true,
> @@ -1112,6 +1116,14 @@ static const struct rvin_info rcar_info_r8a77970 = {
> .routes = _rcar_info_r8a77970_routes,
> };
>
> +static const struct rvin_info rcar_info_r8a77995 = {
> + .model = RCAR_GEN3,
> + .use_mc = true,
> + .max_width = 4096,
> + .max_height = 4096,
> + .routes = _rcar_info_r8a77995_routes,
> +};
> +
> static const struct of_device_id rvin_of_id_table[] = {
> {
> .compatible = "renesas,vin-r8a7778",
> @@ -1153,6 +1165,10 @@ static const struct of_device_id rvin_of_id_table[] = {
> .compatible = "renesas,vin-r8a77970",
> .data = &rcar_info_r8a77970,
> },
> + {
> + .compatible = "renesas,vin-r8a77995",
> + .data = &rcar_info_r8a77995,
> + },
> { /* Sentinel */ },
> };
> MODULE_DEVICE_TABLE(of, rvin_of_id_table);
> --
> 2.7.4
>
--
Regards,
Niklas Söderlund
