[PATCH 1/2] serial: sh: Fix compile error when lacking HSCIF support

[Paul Barker]

On 23/08/2023 12:22, Marek Vasut wrote:
> On 8/23/23 08:25, Paul Barker wrote:
>> On 22/08/2023 22:38, Marek Vasut wrote:
>>> On 8/22/23 20:10, Paul Barker wrote:
>>>> If we attempt to compile serial_sh.c for a system which lacks HSCIF
>>>> support (e.g. R8A7740), we see the following compilation error:
>>>>
>>>>       In file included from drivers/serial/serial_sh.c:20:
>>>>       drivers/serial/serial_sh.c: In function ‘sh_serial_init_generic’:
>>>>       drivers/serial/serial_sh.h:429:35: warning: implicit declaration of function ‘sci_HSSRR_out’; did you mean ‘sci_SCSCR_out’? [-Wimplicit-function-declaration]
>>>>         429 | #define sci_out(port, reg, value) sci_##reg##_out(port, value)
>>>>             |                                   ^~~~
>>>>       drivers/serial/serial_sh.c:62:17: note: in expansion of macro ‘sci_out’
>>>>          62 |                 sci_out(port, HSSRR, HSSRR_SRE | HSSRR_SRCYC8);
>>>>             |                 ^~~~~~~
>>>>
>>>> To fix this, only try to support access to the HSSRR register for SoCs
>>>> where it actually exists.
>>>>
>>>> Fixes: bbe36e29ca2c ('serial: sh: Add HSCIF support for R-Car SoC')
>>>> Signed-off-by: Paul Barker <paul.barker.ct at bp.renesas.com>
>>>> Cc: Hai Pham <hai.pham.ud at renesas.com>
>>>> Cc: Marek Vasut <marek.vasut+renesas at mailbox.org>
>>>> Cc: Simon Glass <sjg at chromium.org>
>>>> ---
>>>>    drivers/serial/serial_sh.c | 2 ++
>>>>    1 file changed, 2 insertions(+)
>>>>
>>>> diff --git a/drivers/serial/serial_sh.c b/drivers/serial/serial_sh.c
>>>> index 20cda5dbe272..cd9bcf4f0a25 100644
>>>> --- a/drivers/serial/serial_sh.c
>>>> +++ b/drivers/serial/serial_sh.c
>>>> @@ -58,8 +58,10 @@ static void sh_serial_init_generic(struct uart_port *port)
>>>>    	sci_out(port, SCSPTR, 0x0003);
>>>>    #endif
>>>>    
>>>> +#if defined(CONFIG_RCAR_GEN2) || defined(CONFIG_RCAR_GEN3) || defined(CONFIG_RCAR_GEN4)
>>>
>>> You can use CONFIG_RCAR_64 instead of GEN3+GEN4 test.
>>
>> I'm going to be sending patches for the RZ/G2L shortly which enables
>> CONFIG_RCAR_64 but lacks HSCIF support. Testing for RCAR gen3/gen4 now
>> will avoid the need to make another change here.
> 
> The G2L has no HSCIF ?

No, it has SCIF and SCI interfaces but no HSCIF.

Thanks,
Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0x27F4B3459F002257.asc
Type: application/pgp-keys
Size: 3520 bytes
Desc: OpenPGP public key
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230826/10649cce/attachment.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 236 bytes
Desc: OpenPGP digital signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230826/10649cce/attachment.sig>