[U-Boot] [PATCH 05/14] serial: 16550: Add port type as driver data

[Marek Vasut]

On 11/27/2016 06:03 PM, Simon Glass wrote:
> Hi Marex,
> 
> On 25 November 2016 at 15:32, Marek Vasut <marex at denx.de> wrote:
>> Add driver data to each compatible string to identify the type of
>> the port. Since all the ports in the driver are entirely compatible
>> with 16550 for now, all are marked with PORT_NS16550. But, there
>> are ports which have specific quirks, like the JZ4780 UART, which
>> do not have any DT property to denote the quirks. Instead, Linux
>> uses the compatible string to discern such ports and enable the
>> necessary quirks.
>>
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> Cc: Tom Rini <trini at konsulko.com>
>> Cc: Simon Glass <sjg at chromium.org>
>> ---
>>  drivers/serial/ns16550.c | 26 ++++++++++++++++----------
>>  1 file changed, 16 insertions(+), 10 deletions(-)
>>
>> diff --git a/drivers/serial/ns16550.c b/drivers/serial/ns16550.c
>> index 3c9f3b0..3130a1d 100644
>> --- a/drivers/serial/ns16550.c
>> +++ b/drivers/serial/ns16550.c
>> @@ -360,6 +360,12 @@ int ns16550_serial_probe(struct udevice *dev)
>>         return 0;
>>  }
>>
>> +#if CONFIG_IS_ENABLED(OF_CONTROL)
>> +enum {
>> +       PORT_NS16550 = 0,
>> +};
>> +#endif
>> +
>>  #if CONFIG_IS_ENABLED(OF_CONTROL) && !CONFIG_IS_ENABLED(OF_PLATDATA)
>>  int ns16550_serial_ofdata_to_platdata(struct udevice *dev)
>>  {
>> @@ -453,16 +459,16 @@ const struct dm_serial_ops ns16550_serial_ops = {
>>   * compatible string to your dts.
>>   */
>>  static const struct udevice_id ns16550_serial_ids[] = {
>> -       { .compatible = "ns16550" },
>> -       { .compatible = "ns16550a" },
>> -       { .compatible = "nvidia,tegra20-uart" },
>> -       { .compatible = "snps,dw-apb-uart" },
>> -       { .compatible = "ti,omap2-uart" },
>> -       { .compatible = "ti,omap3-uart" },
>> -       { .compatible = "ti,omap4-uart" },
>> -       { .compatible = "ti,am3352-uart" },
>> -       { .compatible = "ti,am4372-uart" },
>> -       { .compatible = "ti,dra742-uart" },
>> +       { .compatible = "ns16550",              .data = PORT_NS16550 },
>> +       { .compatible = "ns16550a",             .data = PORT_NS16550 },
>> +       { .compatible = "nvidia,tegra20-uart",  .data = PORT_NS16550 },
>> +       { .compatible = "snps,dw-apb-uart",     .data = PORT_NS16550 },
>> +       { .compatible = "ti,omap2-uart",        .data = PORT_NS16550 },
>> +       { .compatible = "ti,omap3-uart",        .data = PORT_NS16550 },
>> +       { .compatible = "ti,omap4-uart",        .data = PORT_NS16550 },
>> +       { .compatible = "ti,am3352-uart",       .data = PORT_NS16550 },
>> +       { .compatible = "ti,am4372-uart",       .data = PORT_NS16550 },
>> +       { .compatible = "ti,dra742-uart",       .data = PORT_NS16550 },
> 
> But can we have 0 as the default so we don't need these values?

PORT_NS16550 is zero anyway, it's just explicitly spelled here.

-- 
Best regards,
Marek Vasut