[PATCH v3 1/2] staging: dgap: tty.c: adds error handing in tty driver allocations

[Lidza Louina]

On Tue, Oct 1, 2013 at 12:09 PM, Dan Carpenter <dan.carpenter at oracle.com> wrote:
> On Tue, Oct 01, 2013 at 12:00:23PM -0400, Lidza Louina wrote:
>> @@ -306,7 +316,16 @@ int dgap_tty_register(struct board_t *brd)
>>
>>       DPR_INIT(("DGAP REGISTER TTY: MAJORS: %d %d\n", brd->SerialDriver->major,
>>               brd->PrintDriver->major));
>> -
>
> You need a "return 0;" here otherwise we release everything on success
> when we worked so hard to allocate it.
>
> regards,
> dan carpenter

When I add that I get these smatch warnings:

drivers/staging/dgap/dgap_tty.c:323 dgap_tty_register() info: ignoring
unreachable code.
drivers/staging/dgap/dgap_tty.c:323 dgap_tty_register() info: ignoring
unreachable code.

Ignore these warnings?

Thanks,
Lidza