2023-10-25 18:50:48

by Andy Shevchenko

[permalink] [raw]
Subject: [PATCH v1 0/3] gpio: mmio: Make driver agnostic

Driver uses so far some OF APIs when generic fwnode ones can be used.
Replace these APIs accordingly. Note, this will help to clean up OF
headers even more.

Andy Shevchenko (3):
device property: Implement device_is_big_endian()
gpio: mmio: Make use of device properties
gpio: mmio: Clean up headers

drivers/gpio/gpio-mmio.c | 53 +++++++++++++++-------------------------
include/linux/property.h | 26 ++++++++++++++++++++
2 files changed, 46 insertions(+), 33 deletions(-)

--
2.40.0.1.gaa8946217a0b


2023-10-25 19:55:52

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH v1 0/3] gpio: mmio: Make driver agnostic

On Wed, Oct 25, 2023 at 8:50 PM Andy Shevchenko
<[email protected]> wrote:

> Driver uses so far some OF APIs when generic fwnode ones can be used.
> Replace these APIs accordingly. Note, this will help to clean up OF
> headers even more.

Clean, elegant and does exactly what we want.
Reviewed-by: Linus Walleij <[email protected]>

Yours,
Linus Walleij

2023-12-18 12:45:52

by Bartosz Golaszewski

[permalink] [raw]
Subject: Re: [PATCH v1 0/3] gpio: mmio: Make driver agnostic

On Wed, Oct 25, 2023 at 8:50 PM Andy Shevchenko
<[email protected]> wrote:
>
> Driver uses so far some OF APIs when generic fwnode ones can be used.
> Replace these APIs accordingly. Note, this will help to clean up OF
> headers even more.
>
> Andy Shevchenko (3):
> device property: Implement device_is_big_endian()
> gpio: mmio: Make use of device properties
> gpio: mmio: Clean up headers
>
> drivers/gpio/gpio-mmio.c | 53 +++++++++++++++-------------------------
> include/linux/property.h | 26 ++++++++++++++++++++
> 2 files changed, 46 insertions(+), 33 deletions(-)
>
> --
> 2.40.0.1.gaa8946217a0b
>

Applied patches 2 and 3.

Bart

2023-12-18 13:36:23

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH v1 0/3] gpio: mmio: Make driver agnostic

On Mon, Dec 18, 2023 at 03:35:28PM +0200, Andy Shevchenko wrote:
> On Mon, Dec 18, 2023 at 01:43:40PM +0100, Bartosz Golaszewski wrote:

...

> Thank you! I guess you forgot to add --sign-off to the `git merge ...`

Sorry, it's --signoff.

--
With Best Regards,
Andy Shevchenko



2023-12-18 13:38:49

by Bartosz Golaszewski

[permalink] [raw]
Subject: Re: [PATCH v1 0/3] gpio: mmio: Make driver agnostic

On Mon, Dec 18, 2023 at 2:36 PM Andy Shevchenko
<[email protected]> wrote:
>
> On Mon, Dec 18, 2023 at 03:35:28PM +0200, Andy Shevchenko wrote:
> > On Mon, Dec 18, 2023 at 01:43:40PM +0100, Bartosz Golaszewski wrote:
>
> ...
>
> > Thank you! I guess you forgot to add --sign-off to the `git merge ...`
>
> Sorry, it's --signoff.
>

I never sign-off on merges though, is it mandatory? Probably not as
there are no warnings from next like for regular commits about missing
sign-offs.

Bart

> --
> With Best Regards,
> Andy Shevchenko
>
>

2023-12-18 15:02:45

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH v1 0/3] gpio: mmio: Make driver agnostic

On Mon, Dec 18, 2023 at 02:37:48PM +0100, Bartosz Golaszewski wrote:
> On Mon, Dec 18, 2023 at 2:36 PM Andy Shevchenko
> <[email protected]> wrote:
> > On Mon, Dec 18, 2023 at 03:35:28PM +0200, Andy Shevchenko wrote:
> > > On Mon, Dec 18, 2023 at 01:43:40PM +0100, Bartosz Golaszewski wrote:

...

> > > Thank you! I guess you forgot to add --sign-off to the `git merge ...`
> >
> > Sorry, it's --signoff.
>
> I never sign-off on merges though, is it mandatory? Probably not as
> there are no warnings from next like for regular commits about missing
> sign-offs.

No idea. I do it as it is harmless, and makes commit more robust.

For me it's fine, but I leave for me to exclaim later "Yay, had told ya!" :-)

--
With Best Regards,
Andy Shevchenko