[PATCH v5] serial: sh-sci: Add OF support - incremental

Arnd Bergmann arnd at arndb.de
Tue Mar 5 14:28:50 EST 2013


On Tuesday 05 March 2013, Bastian Hecht wrote:
> This is an incremental patch to the "[PATCH v4 1/3] serial: sh-sci: Add
> OF support" implementing suggestions from Arnd.
> 
> - The compatible pattern has been reworked to include devices with
>   legacy register sets.
> - DT property interrupt-names added
> - Renamed renesas,scbrr-algo-id into renesas,clock-algorithm
> - Replaced #ifdef CONFIG_OF with
>   +       if (!IS_ENABLED(CONFIG_OF) || !np)
>   +                               return NULL;
> 
> I opted to show the changes in an incremental patch as it may be easier
> to follow the changes and as Paul has the v3 already in his repo.

Ok, makes sense.

> -	sci at 0xe6c50000 {
> -		compatible = "renesas,sci-SCIFA-uart";
> +	sci at e6c50000 {
> +		compatible = "renesas,sci-scifa-serial";

As mentioned, I meant the name to be changed, not the compatible value.

> +	{ .compatible = "renesas,sci-sci-serial",
> +		.data = (void *)SCIx_SCI_REGTYPE },
> +	{ .compatible = "renesas,sci-scif-serial",
> +		.data = (void *)SCIx_SH4_SCIF_REGTYPE, },
> +	{ .compatible = "renesas,sci-irda-serial",
> +		.data = (void *)SCIx_IRDA_REGTYPE },
> +	{ .compatible = "renesas,sci-scifa-serial",
> +		.data = (void *)SCIx_SCIFA_REGTYPE },
> +	{ .compatible = "renesas,sci-scifb-serial",
> +		.data = (void *)SCIx_SCIFB_REGTYPE },
> +	{ .compatible = "renesas,sci-sh2_scif_fifodata-serial",
> +		.data = (void *)SCIx_SH2_SCIF_FIFODATA_REGTYPE },
> +	{ .compatible = "renesas,sci-sh3_scif-serial",
> +		.data = (void *)SCIx_SH3_SCIF_REGTYPE },
> +	{ .compatible = "renesas,sci-sh4_scif-serial",
> +		.data = (void *)SCIx_SH4_SCIF_REGTYPE },
> +	{ .compatible = "renesas,sci-sh4_scif_no_scsptr-serial",
> +		.data = (void *)SCIx_SH4_SCIF_NO_SCSPTR_REGTYPE },
> +	{ .compatible = "renesas,sci-sh4_scif_fifodata-serial",
> +		.data = (void *)SCIx_SH4_SCIF_FIFODATA_REGTYPE },
> +	{ .compatible = "renesas,sci-sh7705_scif-serial",
> +		.data = (void *)SCIx_SH7705_SCIF_REGTYPE },
>  	{},

Ah nice, it's not even more complex than before.

Looks good,

Acked-by: Arnd Bergmann <arnd at arndb.de>



More information about the linux-arm-kernel mailing list