Hi Geert,
geert@xxxxxxxxxxxxxx wrote on Tue, 29 Mar 2022 17:58:40 +0200:
> Hi Miquel,
>
> On Tue, Mar 29, 2022 at 5:24 PM Miquel Raynal <miquel.raynal@xxxxxxxxxxx> wrote:
> > Unsigned int is better than unsigned long as its size does not change
> > between setups and we don't really need this variable to be more than a
> > few bits wide for now.
> >
> > Suggested-by: Suggested-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
>
> Thanks for your patch!
>
> > --- a/drivers/tty/serial/8250/8250_dw.c
> > +++ b/drivers/tty/serial/8250/8250_dw.c
> > @@ -376,7 +376,7 @@ static bool dw8250_idma_filter(struct dma_chan *chan, void *param)
> > static void dw8250_quirks(struct uart_port *p, struct dw8250_data *data)
> > {
> > struct device_node *np = p->dev->of_node;
> > - unsigned long quirks = (unsigned long)device_get_match_data(p->dev);
> > + unsigned int quirks = (unsigned int)device_get_match_data(p->dev);
>
> "(uintptr_t)" (or "(unsigned long)" ;-), else you'll get a "cast from
> pointer to integer of different size" warning on 64-bit builds.
Actually as I'm changing that quirks entry from being the entire device
data to be a member of a wider structure, I'll change its type when
doing the switch to avoid such a warning.
Thanks,
Miquèl
