Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Julian Calabywrites: > Hi Kalle, > > On Sat, Apr 16, 2016 at 4:25 AM, Kalle Valo wrote: >> Byeoungwook Kim writes: >> >>> rtl_*_delay() functions were reused same codes about addr variable. >>> So i have converted to rtl_addr_delay() from code about addr variable. >>> >>> Signed-off-by: Byeoungwook Kim >>> Reviewed-by: Julian Calaby >> >> Doesn't apply: >> >> Applying: rtlwifi: Fix reusable codes in core.c >> fatal: sha1 information is lacking or useless >> (drivers/net/wireless/realtek/rtlwifi/core.c). >> Repository lacks necessary blobs to fall back on 3-way merge. >> Cannot fall back to three-way merge. >> Patch failed at 0001 rtlwifi: Fix reusable codes in core.c >> >> Please rebase and resend. > > This one is already applied in some form. I thought I'd listed it in > my big list of superseded patches, however I must have missed it. Or I missed it :) But good to know, so no actions needed anymore. -- Kalle Valo
Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Julian Calaby writes: > Hi Kalle, > > On Sat, Apr 16, 2016 at 4:25 AM, Kalle Valo wrote: >> Byeoungwook Kim writes: >> >>> rtl_*_delay() functions were reused same codes about addr variable. >>> So i have converted to rtl_addr_delay() from code about addr variable. >>> >>> Signed-off-by: Byeoungwook Kim >>> Reviewed-by: Julian Calaby >> >> Doesn't apply: >> >> Applying: rtlwifi: Fix reusable codes in core.c >> fatal: sha1 information is lacking or useless >> (drivers/net/wireless/realtek/rtlwifi/core.c). >> Repository lacks necessary blobs to fall back on 3-way merge. >> Cannot fall back to three-way merge. >> Patch failed at 0001 rtlwifi: Fix reusable codes in core.c >> >> Please rebase and resend. > > This one is already applied in some form. I thought I'd listed it in > my big list of superseded patches, however I must have missed it. Or I missed it :) But good to know, so no actions needed anymore. -- Kalle Valo
Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Hi Kalle, On Sat, Apr 16, 2016 at 4:25 AM, Kalle Valowrote: > Byeoungwook Kim writes: > >> rtl_*_delay() functions were reused same codes about addr variable. >> So i have converted to rtl_addr_delay() from code about addr variable. >> >> Signed-off-by: Byeoungwook Kim >> Reviewed-by: Julian Calaby > > Doesn't apply: > > Applying: rtlwifi: Fix reusable codes in core.c > fatal: sha1 information is lacking or useless > (drivers/net/wireless/realtek/rtlwifi/core.c). > Repository lacks necessary blobs to fall back on 3-way merge. > Cannot fall back to three-way merge. > Patch failed at 0001 rtlwifi: Fix reusable codes in core.c > > Please rebase and resend. This one is already applied in some form. I thought I'd listed it in my big list of superseded patches, however I must have missed it. Thanks, -- Julian Calaby Email: julian.cal...@gmail.com Profile: http://www.google.com/profiles/julian.calaby/
Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Hi Kalle, On Sat, Apr 16, 2016 at 4:25 AM, Kalle Valo wrote: > Byeoungwook Kim writes: > >> rtl_*_delay() functions were reused same codes about addr variable. >> So i have converted to rtl_addr_delay() from code about addr variable. >> >> Signed-off-by: Byeoungwook Kim >> Reviewed-by: Julian Calaby > > Doesn't apply: > > Applying: rtlwifi: Fix reusable codes in core.c > fatal: sha1 information is lacking or useless > (drivers/net/wireless/realtek/rtlwifi/core.c). > Repository lacks necessary blobs to fall back on 3-way merge. > Cannot fall back to three-way merge. > Patch failed at 0001 rtlwifi: Fix reusable codes in core.c > > Please rebase and resend. This one is already applied in some form. I thought I'd listed it in my big list of superseded patches, however I must have missed it. Thanks, -- Julian Calaby Email: julian.cal...@gmail.com Profile: http://www.google.com/profiles/julian.calaby/
Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Byeoungwook Kimwrites: > rtl_*_delay() functions were reused same codes about addr variable. > So i have converted to rtl_addr_delay() from code about addr variable. > > Signed-off-by: Byeoungwook Kim > Reviewed-by: Julian Calaby Doesn't apply: Applying: rtlwifi: Fix reusable codes in core.c fatal: sha1 information is lacking or useless (drivers/net/wireless/realtek/rtlwifi/core.c). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 rtlwifi: Fix reusable codes in core.c Please rebase and resend. -- Kalle Valo
Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Byeoungwook Kim writes: > rtl_*_delay() functions were reused same codes about addr variable. > So i have converted to rtl_addr_delay() from code about addr variable. > > Signed-off-by: Byeoungwook Kim > Reviewed-by: Julian Calaby Doesn't apply: Applying: rtlwifi: Fix reusable codes in core.c fatal: sha1 information is lacking or useless (drivers/net/wireless/realtek/rtlwifi/core.c). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 rtlwifi: Fix reusable codes in core.c Please rebase and resend. -- Kalle Valo
Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Hi Julian, I sent to modify the patch again. Thanks for your assists! Regards, Byeoungwook 2016-02-03 11:06 GMT+09:00 Julian Calaby : > Hi Byeoungwook, > > On Wed, Feb 3, 2016 at 1:01 PM, Byeoungwook Kim wrote: >> rtl_*_delay() functions were reused same codes about addr variable. >> So i have converted to rtl_addr_delay() from code about addr variable. >> >> Signed-off-by: Byeoungwook Kim >> Reviewed-by: Julian Calaby > > Just a note for the future, you have to explicitly be given a > reviewed-by, you can't just assume that someone who has made comments > on a patch has reviewed it. > > In this case, I have reviewed it, so formally: > > Reviewed-by: Julian Calaby > > Thanks, > > Julian Calaby > > >> --- >> drivers/net/wireless/realtek/rtlwifi/core.c | 28 >> >> 1 file changed, 4 insertions(+), 24 deletions(-) >> >> diff --git a/drivers/net/wireless/realtek/rtlwifi/core.c >> b/drivers/net/wireless/realtek/rtlwifi/core.c >> index 05f432c..c1193d1 100644 >> --- a/drivers/net/wireless/realtek/rtlwifi/core.c >> +++ b/drivers/net/wireless/realtek/rtlwifi/core.c >> @@ -63,18 +63,8 @@ EXPORT_SYMBOL(rtl_addr_delay); >> void rtl_rfreg_delay(struct ieee80211_hw *hw, enum radio_path rfpath, u32 >> addr, >> u32 mask, u32 data) >> { >> - if (addr == 0xfe) { >> - mdelay(50); >> - } else if (addr == 0xfd) { >> - mdelay(5); >> - } else if (addr == 0xfc) { >> - mdelay(1); >> - } else if (addr == 0xfb) { >> - udelay(50); >> - } else if (addr == 0xfa) { >> - udelay(5); >> - } else if (addr == 0xf9) { >> - udelay(1); >> + if (addr >= 0xf9 && addr <= 0xfe) { >> + rtl_addr_delay(addr); >> } else { >> rtl_set_rfreg(hw, rfpath, addr, mask, data); >> udelay(1); >> @@ -84,18 +74,8 @@ EXPORT_SYMBOL(rtl_rfreg_delay); >> >> void rtl_bb_delay(struct ieee80211_hw *hw, u32 addr, u32 data) >> { >> - if (addr == 0xfe) { >> - mdelay(50); >> - } else if (addr == 0xfd) { >> - mdelay(5); >> - } else if (addr == 0xfc) { >> - mdelay(1); >> - } else if (addr == 0xfb) { >> - udelay(50); >> - } else if (addr == 0xfa) { >> - udelay(5); >> - } else if (addr == 0xf9) { >> - udelay(1); >> + if (addr >= 0xf9 && addr <= 0xfe) { >> + rtl_addr_delay(addr); >> } else { >> rtl_set_bbreg(hw, addr, MASKDWORD, data); >> udelay(1); >> -- >> 2.5.0 >> > > > > -- > Julian Calaby > > Email: julian.cal...@gmail.com > Profile: http://www.google.com/profiles/julian.calaby/
Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Hi Byeoungwook, On Wed, Feb 3, 2016 at 1:01 PM, Byeoungwook Kim wrote: > rtl_*_delay() functions were reused same codes about addr variable. > So i have converted to rtl_addr_delay() from code about addr variable. > > Signed-off-by: Byeoungwook Kim > Reviewed-by: Julian Calaby Just a note for the future, you have to explicitly be given a reviewed-by, you can't just assume that someone who has made comments on a patch has reviewed it. In this case, I have reviewed it, so formally: Reviewed-by: Julian Calaby Thanks, Julian Calaby > --- > drivers/net/wireless/realtek/rtlwifi/core.c | 28 > 1 file changed, 4 insertions(+), 24 deletions(-) > > diff --git a/drivers/net/wireless/realtek/rtlwifi/core.c > b/drivers/net/wireless/realtek/rtlwifi/core.c > index 05f432c..c1193d1 100644 > --- a/drivers/net/wireless/realtek/rtlwifi/core.c > +++ b/drivers/net/wireless/realtek/rtlwifi/core.c > @@ -63,18 +63,8 @@ EXPORT_SYMBOL(rtl_addr_delay); > void rtl_rfreg_delay(struct ieee80211_hw *hw, enum radio_path rfpath, u32 > addr, > u32 mask, u32 data) > { > - if (addr == 0xfe) { > - mdelay(50); > - } else if (addr == 0xfd) { > - mdelay(5); > - } else if (addr == 0xfc) { > - mdelay(1); > - } else if (addr == 0xfb) { > - udelay(50); > - } else if (addr == 0xfa) { > - udelay(5); > - } else if (addr == 0xf9) { > - udelay(1); > + if (addr >= 0xf9 && addr <= 0xfe) { > + rtl_addr_delay(addr); > } else { > rtl_set_rfreg(hw, rfpath, addr, mask, data); > udelay(1); > @@ -84,18 +74,8 @@ EXPORT_SYMBOL(rtl_rfreg_delay); > > void rtl_bb_delay(struct ieee80211_hw *hw, u32 addr, u32 data) > { > - if (addr == 0xfe) { > - mdelay(50); > - } else if (addr == 0xfd) { > - mdelay(5); > - } else if (addr == 0xfc) { > - mdelay(1); > - } else if (addr == 0xfb) { > - udelay(50); > - } else if (addr == 0xfa) { > - udelay(5); > - } else if (addr == 0xf9) { > - udelay(1); > + if (addr >= 0xf9 && addr <= 0xfe) { > + rtl_addr_delay(addr); > } else { > rtl_set_bbreg(hw, addr, MASKDWORD, data); > udelay(1); > -- > 2.5.0 > -- Julian Calaby Email: julian.cal...@gmail.com Profile: http://www.google.com/profiles/julian.calaby/
Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Hi Byeoungwook, On Wed, Feb 3, 2016 at 1:01 PM, Byeoungwook Kimwrote: > rtl_*_delay() functions were reused same codes about addr variable. > So i have converted to rtl_addr_delay() from code about addr variable. > > Signed-off-by: Byeoungwook Kim > Reviewed-by: Julian Calaby Just a note for the future, you have to explicitly be given a reviewed-by, you can't just assume that someone who has made comments on a patch has reviewed it. In this case, I have reviewed it, so formally: Reviewed-by: Julian Calaby Thanks, Julian Calaby > --- > drivers/net/wireless/realtek/rtlwifi/core.c | 28 > 1 file changed, 4 insertions(+), 24 deletions(-) > > diff --git a/drivers/net/wireless/realtek/rtlwifi/core.c > b/drivers/net/wireless/realtek/rtlwifi/core.c > index 05f432c..c1193d1 100644 > --- a/drivers/net/wireless/realtek/rtlwifi/core.c > +++ b/drivers/net/wireless/realtek/rtlwifi/core.c > @@ -63,18 +63,8 @@ EXPORT_SYMBOL(rtl_addr_delay); > void rtl_rfreg_delay(struct ieee80211_hw *hw, enum radio_path rfpath, u32 > addr, > u32 mask, u32 data) > { > - if (addr == 0xfe) { > - mdelay(50); > - } else if (addr == 0xfd) { > - mdelay(5); > - } else if (addr == 0xfc) { > - mdelay(1); > - } else if (addr == 0xfb) { > - udelay(50); > - } else if (addr == 0xfa) { > - udelay(5); > - } else if (addr == 0xf9) { > - udelay(1); > + if (addr >= 0xf9 && addr <= 0xfe) { > + rtl_addr_delay(addr); > } else { > rtl_set_rfreg(hw, rfpath, addr, mask, data); > udelay(1); > @@ -84,18 +74,8 @@ EXPORT_SYMBOL(rtl_rfreg_delay); > > void rtl_bb_delay(struct ieee80211_hw *hw, u32 addr, u32 data) > { > - if (addr == 0xfe) { > - mdelay(50); > - } else if (addr == 0xfd) { > - mdelay(5); > - } else if (addr == 0xfc) { > - mdelay(1); > - } else if (addr == 0xfb) { > - udelay(50); > - } else if (addr == 0xfa) { > - udelay(5); > - } else if (addr == 0xf9) { > - udelay(1); > + if (addr >= 0xf9 && addr <= 0xfe) { > + rtl_addr_delay(addr); > } else { > rtl_set_bbreg(hw, addr, MASKDWORD, data); > udelay(1); > -- > 2.5.0 > -- Julian Calaby Email: julian.cal...@gmail.com Profile: http://www.google.com/profiles/julian.calaby/
Re: [PATCH 2/2] rtlwifi: Fix reusable codes in core.c
Hi Julian, I sent to modify the patch again. Thanks for your assists! Regards, Byeoungwook 2016-02-03 11:06 GMT+09:00 Julian Calaby: > Hi Byeoungwook, > > On Wed, Feb 3, 2016 at 1:01 PM, Byeoungwook Kim wrote: >> rtl_*_delay() functions were reused same codes about addr variable. >> So i have converted to rtl_addr_delay() from code about addr variable. >> >> Signed-off-by: Byeoungwook Kim >> Reviewed-by: Julian Calaby > > Just a note for the future, you have to explicitly be given a > reviewed-by, you can't just assume that someone who has made comments > on a patch has reviewed it. > > In this case, I have reviewed it, so formally: > > Reviewed-by: Julian Calaby > > Thanks, > > Julian Calaby > > >> --- >> drivers/net/wireless/realtek/rtlwifi/core.c | 28 >> >> 1 file changed, 4 insertions(+), 24 deletions(-) >> >> diff --git a/drivers/net/wireless/realtek/rtlwifi/core.c >> b/drivers/net/wireless/realtek/rtlwifi/core.c >> index 05f432c..c1193d1 100644 >> --- a/drivers/net/wireless/realtek/rtlwifi/core.c >> +++ b/drivers/net/wireless/realtek/rtlwifi/core.c >> @@ -63,18 +63,8 @@ EXPORT_SYMBOL(rtl_addr_delay); >> void rtl_rfreg_delay(struct ieee80211_hw *hw, enum radio_path rfpath, u32 >> addr, >> u32 mask, u32 data) >> { >> - if (addr == 0xfe) { >> - mdelay(50); >> - } else if (addr == 0xfd) { >> - mdelay(5); >> - } else if (addr == 0xfc) { >> - mdelay(1); >> - } else if (addr == 0xfb) { >> - udelay(50); >> - } else if (addr == 0xfa) { >> - udelay(5); >> - } else if (addr == 0xf9) { >> - udelay(1); >> + if (addr >= 0xf9 && addr <= 0xfe) { >> + rtl_addr_delay(addr); >> } else { >> rtl_set_rfreg(hw, rfpath, addr, mask, data); >> udelay(1); >> @@ -84,18 +74,8 @@ EXPORT_SYMBOL(rtl_rfreg_delay); >> >> void rtl_bb_delay(struct ieee80211_hw *hw, u32 addr, u32 data) >> { >> - if (addr == 0xfe) { >> - mdelay(50); >> - } else if (addr == 0xfd) { >> - mdelay(5); >> - } else if (addr == 0xfc) { >> - mdelay(1); >> - } else if (addr == 0xfb) { >> - udelay(50); >> - } else if (addr == 0xfa) { >> - udelay(5); >> - } else if (addr == 0xf9) { >> - udelay(1); >> + if (addr >= 0xf9 && addr <= 0xfe) { >> + rtl_addr_delay(addr); >> } else { >> rtl_set_bbreg(hw, addr, MASKDWORD, data); >> udelay(1); >> -- >> 2.5.0 >> > > > > -- > Julian Calaby > > Email: julian.cal...@gmail.com > Profile: http://www.google.com/profiles/julian.calaby/