diff mbox series

[v9,03/12] serial: actions: add compatible string

Message ID 1585725575-8050-4-git-send-email-amittomer25@gmail.com
State Superseded
Headers show
Series Actions S700 SoC support | expand

Commit Message

Amit Tomer April 1, 2020, 7:19 a.m. UTC
This patch adds "actions,owl-uart" string to the owl uart driver. It
is also defined in Linux kernel.

Reviewed-by: Andre Przywara <andre.przywara at arm.com>
Signed-off-by: Amit Singh Tomar <amittomer25 at gmail.com>
---
Changes since v8:
        * No changes.
Changes since v7:
        * No changes.
Changes since v6:
        * Added Reviewd-by tag.
Changes since v5:
        * Moved it to from 06/11 to 03/11.
        * Used appropriate commit message.
        * Removed the reviwed-by tag.
Changes since v4:
        * Moved it to from 09/11 to 06/11.
Changes since v3:
        * Used only owl-uart for compatible string.
Changes since v2:
        * No changes.
Changes since v1:
        * No changes.
---
 drivers/serial/serial_owl.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Manivannan Sadhasivam April 5, 2020, 6:59 a.m. UTC | #1
On Wed, Apr 01, 2020 at 12:49:26PM +0530, Amit Singh Tomar wrote:
> This patch adds "actions,owl-uart" string to the owl uart driver. It
> is also defined in Linux kernel.
> 
> Reviewed-by: Andre Przywara <andre.przywara at arm.com>
> Signed-off-by: Amit Singh Tomar <amittomer25 at gmail.com>
> ---
> Changes since v8:
>         * No changes.
> Changes since v7:
>         * No changes.
> Changes since v6:
>         * Added Reviewd-by tag.
> Changes since v5:
>         * Moved it to from 06/11 to 03/11.
>         * Used appropriate commit message.
>         * Removed the reviwed-by tag.
> Changes since v4:
>         * Moved it to from 09/11 to 06/11.
> Changes since v3:
>         * Used only owl-uart for compatible string.
> Changes since v2:
>         * No changes.
> Changes since v1:
>         * No changes.
> ---
>  drivers/serial/serial_owl.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/serial/serial_owl.c b/drivers/serial/serial_owl.c
> index 7ead73e..539acdc 100644
> --- a/drivers/serial/serial_owl.c
> +++ b/drivers/serial/serial_owl.c
> @@ -121,6 +121,7 @@ static const struct dm_serial_ops owl_serial_ops = {
>  
>  static const struct udevice_id owl_serial_ids[] = {
>  	{ .compatible =	"actions,s900-serial" },

This should've been removed here itself as opposed to in devicetree sync patch.
But I don't warrant a new series for this change.

Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam at linaro.org>

Thanks,
Mani

> +	{ .compatible = "actions,owl-uart" },
>  	{ }
>  };
>  
> -- 
> 2.7.4
>
Andre Przywara April 5, 2020, 10:08 a.m. UTC | #2
On 05/04/2020 07:59, Manivannan Sadhasivam wrote:
> On Wed, Apr 01, 2020 at 12:49:26PM +0530, Amit Singh Tomar wrote:
>> This patch adds "actions,owl-uart" string to the owl uart driver. It
>> is also defined in Linux kernel.
>>
>> Reviewed-by: Andre Przywara <andre.przywara at arm.com>
>> Signed-off-by: Amit Singh Tomar <amittomer25 at gmail.com>
>> ---
>> Changes since v8:
>>         * No changes.
>> Changes since v7:
>>         * No changes.
>> Changes since v6:
>>         * Added Reviewd-by tag.
>> Changes since v5:
>>         * Moved it to from 06/11 to 03/11.
>>         * Used appropriate commit message.
>>         * Removed the reviwed-by tag.
>> Changes since v4:
>>         * Moved it to from 09/11 to 06/11.
>> Changes since v3:
>>         * Used only owl-uart for compatible string.
>> Changes since v2:
>>         * No changes.
>> Changes since v1:
>>         * No changes.
>> ---
>>  drivers/serial/serial_owl.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/serial/serial_owl.c b/drivers/serial/serial_owl.c
>> index 7ead73e..539acdc 100644
>> --- a/drivers/serial/serial_owl.c
>> +++ b/drivers/serial/serial_owl.c
>> @@ -121,6 +121,7 @@ static const struct dm_serial_ops owl_serial_ops = {
>>  
>>  static const struct udevice_id owl_serial_ids[] = {
>>  	{ .compatible =	"actions,s900-serial" },
> 
> This should've been removed here itself as opposed to in devicetree sync patch.

But this would break S900, because the Bubblegum DT still uses that string.
If you sync the DT before, you have a similar problem, so we have to
change the DT and the driver string in *one* patch, which is what patch
04/12 does.

Cheers,
Andre

> But I don't warrant a new series for this change.
> 
> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam at linaro.org>
> 
> Thanks,
> Mani
> 
>> +	{ .compatible = "actions,owl-uart" },
>>  	{ }
>>  };
>>  
>> -- 
>> 2.7.4
>>
Manivannan Sadhasivam April 5, 2020, 10:54 a.m. UTC | #3
On Sun, Apr 05, 2020 at 11:08:21AM +0100, Andr? Przywara wrote:
> On 05/04/2020 07:59, Manivannan Sadhasivam wrote:
> > On Wed, Apr 01, 2020 at 12:49:26PM +0530, Amit Singh Tomar wrote:
> >> This patch adds "actions,owl-uart" string to the owl uart driver. It
> >> is also defined in Linux kernel.
> >>
> >> Reviewed-by: Andre Przywara <andre.przywara at arm.com>
> >> Signed-off-by: Amit Singh Tomar <amittomer25 at gmail.com>
> >> ---
> >> Changes since v8:
> >>         * No changes.
> >> Changes since v7:
> >>         * No changes.
> >> Changes since v6:
> >>         * Added Reviewd-by tag.
> >> Changes since v5:
> >>         * Moved it to from 06/11 to 03/11.
> >>         * Used appropriate commit message.
> >>         * Removed the reviwed-by tag.
> >> Changes since v4:
> >>         * Moved it to from 09/11 to 06/11.
> >> Changes since v3:
> >>         * Used only owl-uart for compatible string.
> >> Changes since v2:
> >>         * No changes.
> >> Changes since v1:
> >>         * No changes.
> >> ---
> >>  drivers/serial/serial_owl.c | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/drivers/serial/serial_owl.c b/drivers/serial/serial_owl.c
> >> index 7ead73e..539acdc 100644
> >> --- a/drivers/serial/serial_owl.c
> >> +++ b/drivers/serial/serial_owl.c
> >> @@ -121,6 +121,7 @@ static const struct dm_serial_ops owl_serial_ops = {
> >>  
> >>  static const struct udevice_id owl_serial_ids[] = {
> >>  	{ .compatible =	"actions,s900-serial" },
> > 
> > This should've been removed here itself as opposed to in devicetree sync patch.
> 
> But this would break S900, because the Bubblegum DT still uses that string.
> If you sync the DT before, you have a similar problem, so we have to
> change the DT and the driver string in *one* patch, which is what patch
> 04/12 does.
> 

Hmm... chicken & egg problem :) Let's keep it as it is.

Thanks,
Mani

> Cheers,
> Andre
> 
> > But I don't warrant a new series for this change.
> > 
> > Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam at linaro.org>
> > 
> > Thanks,
> > Mani
> > 
> >> +	{ .compatible = "actions,owl-uart" },
> >>  	{ }
> >>  };
> >>  
> >> -- 
> >> 2.7.4
> >>
>
diff mbox series

Patch

diff --git a/drivers/serial/serial_owl.c b/drivers/serial/serial_owl.c
index 7ead73e..539acdc 100644
--- a/drivers/serial/serial_owl.c
+++ b/drivers/serial/serial_owl.c
@@ -121,6 +121,7 @@  static const struct dm_serial_ops owl_serial_ops = {
 
 static const struct udevice_id owl_serial_ids[] = {
 	{ .compatible =	"actions,s900-serial" },
+	{ .compatible = "actions,owl-uart" },
 	{ }
 };