Re: [PATCH] serial: imx: Simplify imx_uart_type

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

On Thu, Mar 02, 2023 at 01:20:25PM +0100, Alexander Stein wrote:
> Am Donnerstag, 2. März 2023, 12:55:47 CET schrieb Uwe Kleine-König:
> > * PGP Signed by an unknown key
> > 
> > Hello,
> > 
> > On Thu, Mar 02, 2023 at 12:48:10PM +0100, Uwe Kleine-König wrote:
> > > There is no point in casting a struct uart_port to a struct imx_port
> > > just to access the first member of the latter (a struct uart_port).
> > > 
> > > This introduces code changes.
> > 
> > Please disregard this patch. While the patch is fine as is, I just
> > noticed a few more functions in this driver that can be simplified
> > accordingly. I'm about to send out an improved version (with a different
> > subject, so not called v2).
> 
> Shouldn't this cast on the other locations be converted into a to_imx_port() 
> function using container_of as well?

Yeah, I wondered about that, too, but didn't tackle that (yet). I'll
keep that for another day (or someone else, don't hesitate if you're
motivated).

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux