drivers/staging/rtl8723bs/include/hal_pwr_seq.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Remove trailing whitespace to comply with kernel coding style.
Signed-off-by: Gabriel Shahrouzi <gshahrouzi@gmail.com>
---
Changes in v2:
- Resend using git send-email to fix formatting issues in email body.
---
drivers/staging/rtl8723bs/include/hal_pwr_seq.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git drivers/staging/rtl8723bs/include/hal_pwr_seq.h drivers/staging/rtl8723bs/include/hal_pwr_seq.h
index b93d74a5b9a5..48bf7f66a06e 100644
--- drivers/staging/rtl8723bs/include/hal_pwr_seq.h
+++ drivers/staging/rtl8723bs/include/hal_pwr_seq.h
@@ -209,7 +209,7 @@
#define RTL8723B_TRANS_END \
/* format */ \
/* { offset, cut_msk, fab_msk|interface_msk, base|cmd, msk, value }, comments here*/ \
- {0xFFFF, PWR_CUT_ALL_MSK, PWR_FAB_ALL_MSK, PWR_INTF_ALL_MSK, 0, PWR_CMD_END, 0, 0},
+ {0xFFFF, PWR_CUT_ALL_MSK, PWR_FAB_ALL_MSK, PWR_INTF_ALL_MSK, 0, PWR_CMD_END, 0, 0},
extern struct wlan_pwr_cfg rtl8723B_power_on_flow[RTL8723B_TRANS_CARDEMU_TO_ACT_STEPS+RTL8723B_TRANS_END_STEPS];
--
2.43.0
On Wed, Apr 02, 2025 at 08:42:07AM -0400, Gabriel Shahrouzi wrote: > Remove trailing whitespace to comply with kernel coding style. > > Signed-off-by: Gabriel Shahrouzi <gshahrouzi@gmail.com> > --- > Changes in v2: > - Resend using git send-email to fix formatting issues in email body. > --- > drivers/staging/rtl8723bs/include/hal_pwr_seq.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git drivers/staging/rtl8723bs/include/hal_pwr_seq.h drivers/staging/rtl8723bs/include/hal_pwr_seq.h > index b93d74a5b9a5..48bf7f66a06e 100644 > --- drivers/staging/rtl8723bs/include/hal_pwr_seq.h > +++ drivers/staging/rtl8723bs/include/hal_pwr_seq.h This wasn't made with git, was it? You are "one" indent level off, the diff should say: --- a/drivers/staging/rtl8723bs/include/hal_pwr_seq.h +++ b/drivers/staging/rtl8723bs/include/hal_pwr_seq.h here, right? Anyway, because of that, this does not apply to the tree at all :( Please fix and send a v3. thanks, greg k-h
On Thu, Apr 3, 2025 at 10:22 AM Greg KH <gregkh@linuxfoundation.org> wrote: > > On Wed, Apr 02, 2025 at 08:42:07AM -0400, Gabriel Shahrouzi wrote: > > Remove trailing whitespace to comply with kernel coding style. > > > > Signed-off-by: Gabriel Shahrouzi <gshahrouzi@gmail.com> > > --- > > Changes in v2: > > - Resend using git send-email to fix formatting issues in the email body. > > --- > > drivers/staging/rtl8723bs/include/hal_pwr_seq.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git drivers/staging/rtl8723bs/include/hal_pwr_seq.h drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > index b93d74a5b9a5..48bf7f66a06e 100644 > > --- drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > +++ drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > This wasn't made with git, was it? You are "one" indent level off, the > diff should say: Interesting. Not entirely certain how this happened. Since it was from an earlier commit I made, I rebased it, amended the changes, and then formatted another patch using git. Apparently one of my other patches for an earlier version had the same problem but the subsequent version has the correct indent. > > --- a/drivers/staging/rtl8723bs/include/hal_pwr_seq.h > +++ b/drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > here, right? Yes. > > Anyway, because of that, this does not apply to the tree at all :( > > Please fix and send a v3. Got it. > > thanks, > > greg k-h On Thu, Apr 3, 2025 at 10:22 AM Greg KH <gregkh@linuxfoundation.org> wrote: > > On Wed, Apr 02, 2025 at 08:42:07AM -0400, Gabriel Shahrouzi wrote: > > Remove trailing whitespace to comply with kernel coding style. > > > > Signed-off-by: Gabriel Shahrouzi <gshahrouzi@gmail.com> > > --- > > Changes in v2: > > - Resend using git send-email to fix formatting issues in email body. > > --- > > drivers/staging/rtl8723bs/include/hal_pwr_seq.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git drivers/staging/rtl8723bs/include/hal_pwr_seq.h drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > index b93d74a5b9a5..48bf7f66a06e 100644 > > --- drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > +++ drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > This wasn't made with git, was it? You are "one" indent level off, the > diff should say: > > --- a/drivers/staging/rtl8723bs/include/hal_pwr_seq.h > +++ b/drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > here, right? > > Anyway, because of that, this does not apply to the tree at all :( > > Please fix and send a v3. > > thanks, > > greg k-h
On Thu, Apr 3, 2025 at 11:03 PM Gabriel <gshahrouzi@gmail.com> wrote: > > On Thu, Apr 3, 2025 at 10:22 AM Greg KH <gregkh@linuxfoundation.org> wrote: > > > > On Wed, Apr 02, 2025 at 08:42:07AM -0400, Gabriel Shahrouzi wrote: > > > Remove trailing whitespace to comply with kernel coding style. > > > > > > Signed-off-by: Gabriel Shahrouzi <gshahrouzi@gmail.com> > > > --- > > > Changes in v2: > > > - Resend using git send-email to fix formatting issues in the email body. > > > --- > > > drivers/staging/rtl8723bs/include/hal_pwr_seq.h | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git drivers/staging/rtl8723bs/include/hal_pwr_seq.h drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > > index b93d74a5b9a5..48bf7f66a06e 100644 > > > --- drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > > +++ drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > > > This wasn't made with git, was it? You are "one" indent level off, the > > diff should say: > Interesting. Not entirely certain how this happened. Since it was from > an earlier commit I made, I rebased it, amended the changes, and then > formatted another patch using git. Apparently one of my other patches > for an earlier version had the same problem but the subsequent version > has the correct indent. > > > > --- a/drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > +++ b/drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > > > here, right? > Yes. > > > > Anyway, because of that, this does not apply to the tree at all :( > > > > Please fix and send a v3. > Got it. > > > > thanks, > > > > greg k-h > > On Thu, Apr 3, 2025 at 10:22 AM Greg KH <gregkh@linuxfoundation.org> wrote: > > > > On Wed, Apr 02, 2025 at 08:42:07AM -0400, Gabriel Shahrouzi wrote: > > > Remove trailing whitespace to comply with kernel coding style. > > > > > > Signed-off-by: Gabriel Shahrouzi <gshahrouzi@gmail.com> > > > --- > > > Changes in v2: > > > - Resend using git send-email to fix formatting issues in email body. > > > --- > > > drivers/staging/rtl8723bs/include/hal_pwr_seq.h | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git drivers/staging/rtl8723bs/include/hal_pwr_seq.h drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > > index b93d74a5b9a5..48bf7f66a06e 100644 > > > --- drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > > +++ drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > > > This wasn't made with git, was it? You are "one" indent level off, the > > diff should say: > > > > --- a/drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > +++ b/drivers/staging/rtl8723bs/include/hal_pwr_seq.h > > > > here, right? > > > > Anyway, because of that, this does not apply to the tree at all :( > > > > Please fix and send a v3. > > > > thanks, > > > > greg k-h Apologies for the duplicate post. I initially replied only to one person by accident and have resent it using 'reply all'
© 2016 - 2025 Red Hat, Inc.