mbox series

[v1,0/2] bitmap-str: A couple of fixes

Message ID 20231215184218.2005611-1-andriy.shevchenko@linux.intel.com
Headers
Series bitmap-str: A couple of fixes |

Message

Andy Shevchenko Dec. 15, 2023, 6:41 p.m. UTC 
  Imtended to be folded into original code, hence no proper commit messages.

Andy Shevchenko (2):
  bitmap-str: Get rid of extern
  bitmap-str: Add missing header(s)

 include/linux/bitmap-str.h | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

Comments

Yury Norov Dec. 15, 2023, 7:01 p.m. UTC | #1 
On Fri, Dec 15, 2023 at 08:41:07PM +0200, Andy Shevchenko wrote:
> Imtended to be folded into original code, hence no proper commit messages.

s/Imtended/Intended

Didn't understand why you didn't write commit messages. It breaks the
very basic rule. Can you elaborate?

> Andy Shevchenko (2):
>   bitmap-str: Get rid of extern
>   bitmap-str: Add missing header(s)
> 
>  include/linux/bitmap-str.h | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
> 
> -- 
> 2.43.0.rc1.1.gbec44491f096
Andy Shevchenko Dec. 15, 2023, 7:14 p.m. UTC | #2 
On Fri, Dec 15, 2023 at 11:01:12AM -0800, Yury Norov wrote:
> On Fri, Dec 15, 2023 at 08:41:07PM +0200, Andy Shevchenko wrote:
> > Imtended to be folded into original code, hence no proper commit messages.
> 
> s/Imtended/Intended
> 
> Didn't understand why you didn't write commit messages. It breaks the
> very basic rule. Can you elaborate?

I expected you to rebase your original commit.
Or based on my understanding of your comments I can submit a revert, if you wish.
Yury Norov Dec. 15, 2023, 8:04 p.m. UTC | #3 
On Fri, Dec 15, 2023 at 09:14:19PM +0200, Andy Shevchenko wrote:
> On Fri, Dec 15, 2023 at 11:01:12AM -0800, Yury Norov wrote:
> > On Fri, Dec 15, 2023 at 08:41:07PM +0200, Andy Shevchenko wrote:
> > > Imtended to be folded into original code, hence no proper commit messages.
> > 
> > s/Imtended/Intended
> > 
> > Didn't understand why you didn't write commit messages. It breaks the
> > very basic rule. Can you elaborate?
> 
> I expected you to rebase your original commit.
> Or based on my understanding of your comments I can submit a revert, if you wish.
 
It's merged in v6.7-rc1. Can you send #1 as a proper patch? Regarding #2,
it's not needed because bitmap-str.h is a local header for bitmap.h.

Thanks,
        Yury