* [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS
@ 2016-06-05 9:30 Arend van Spriel
2016-06-05 9:30 ` [PATCH 2/2] backports: patches: update devcoredump patch Arend van Spriel
2016-07-10 21:16 ` [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS Hauke Mehrtens
0 siblings, 2 replies; 10+ messages in thread
From: Arend van Spriel @ 2016-06-05 9:30 UTC (permalink / raw)
To: Luis R. Rodriguez; +Cc: backports, Arend van Spriel
Speed definition for usb3.1 has been added in 4.6 kernel. Add
definiton for older kernels.
Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
---
backport/backport-include/linux/usb/ch9.h | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/backport/backport-include/linux/usb/ch9.h b/backport/backport-include/linux/usb/ch9.h
index 135e24c..c42fc30 100644
--- a/backport/backport-include/linux/usb/ch9.h
+++ b/backport/backport-include/linux/usb/ch9.h
@@ -21,4 +21,8 @@ static inline int usb_endpoint_maxp(const struct usb_endpoint_descriptor *epd)
}
#endif /* < 3.2 */
+#if LINUX_VERSION_CODE < KERNEL_VERSION(4,6,0)
+#define USB_SPEED_SUPER_PLUS (USB_SPEED_SUPER + 1)
+#endif /* < 4.6 */
+
#endif /* __BACKPORT__LINUX_USB_CH9_H */
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/2] backports: patches: update devcoredump patch
2016-06-05 9:30 [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS Arend van Spriel
@ 2016-06-05 9:30 ` Arend van Spriel
2016-06-05 15:00 ` Johannes Berg
2016-07-10 21:16 ` [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS Hauke Mehrtens
1 sibling, 1 reply; 10+ messages in thread
From: Arend van Spriel @ 2016-06-05 9:30 UTC (permalink / raw)
To: Luis R. Rodriguez; +Cc: backports, Arend van Spriel
Due to changes in include/linux/devcoredump.h the patch needs to
be updated.
Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
---
This patch is another shot at devcoredump patch. It just fixes
applying the patch. However, the resulting file now uses the
scatterlist function sg_pcopy_to_buffer() so that will need a
backport for kernels before v3.11.
Regards,
Arend
---
patches/backport-adjustments/devcoredump.patch | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/patches/backport-adjustments/devcoredump.patch b/patches/backport-adjustments/devcoredump.patch
index b438f32..397b317 100644
--- a/patches/backport-adjustments/devcoredump.patch
+++ b/patches/backport-adjustments/devcoredump.patch
@@ -97,18 +97,18 @@
-__exitcall(devcoredump_exit);
--- a/include/linux/backport-devcoredump.h
+++ b/include/linux/backport-devcoredump.h
-@@ -5,7 +5,7 @@
- #include <linux/module.h>
- #include <linux/vmalloc.h>
+@@ -66,7 +66,7 @@ static inline void _devcd_free_sgtable(struct scatterlist *tab
+ }
+
-#ifdef CONFIG_DEV_COREDUMP
+#ifdef CPTCFG_BPAUTO_WANT_DEV_COREDUMP
- void dev_coredumpv(struct device *dev, const void *data, size_t datalen,
- gfp_t gfp);
+ void dev_coredumpv(struct device *dev, void *data, size_t datalen,
+ gfp_t gfp);
-@@ -30,6 +30,6 @@ dev_coredumpm(struct device *dev, struct
+@@ -100,6 +100,6 @@ static inline void dev_coredumpsg(struct device *dev, struct
{
- free(data);
+ _devcd_free_sgtable(table);
}
-#endif /* CONFIG_DEV_COREDUMP */
+#endif /* CPTCFG_BPAUTO_WANT_DEV_COREDUMP */
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] backports: patches: update devcoredump patch
2016-06-05 9:30 ` [PATCH 2/2] backports: patches: update devcoredump patch Arend van Spriel
@ 2016-06-05 15:00 ` Johannes Berg
2016-06-06 8:58 ` Arend Van Spriel
0 siblings, 1 reply; 10+ messages in thread
From: Johannes Berg @ 2016-06-05 15:00 UTC (permalink / raw)
To: Arend van Spriel, Luis R. Rodriguez; +Cc: backports
On Sun, 2016-06-05 at 11:30 +0200, Arend van Spriel wrote:
> Due to changes in include/linux/devcoredump.h the patch needs to
> be updated.
>
> Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
> ---
> This patch is another shot at devcoredump patch. It just fixes
> applying the patch. However, the resulting file now uses the
> scatterlist function sg_pcopy_to_buffer() so that will need a
> backport for kernels before v3.11.
>
I think, in order to make the new dev_coredumpsg() [or whatever it's
called] function generally available, we have to also adjust the kernel
version where the backported one gets used.
johannes
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] backports: patches: update devcoredump patch
2016-06-05 15:00 ` Johannes Berg
@ 2016-06-06 8:58 ` Arend Van Spriel
2016-06-06 9:00 ` Johannes Berg
0 siblings, 1 reply; 10+ messages in thread
From: Arend Van Spriel @ 2016-06-06 8:58 UTC (permalink / raw)
To: Johannes Berg, Luis R. Rodriguez; +Cc: backports
On 5-6-2016 17:00, Johannes Berg wrote:
> On Sun, 2016-06-05 at 11:30 +0200, Arend van Spriel wrote:
>> Due to changes in include/linux/devcoredump.h the patch needs to
>> be updated.
>>
>> Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
>> ---
>> This patch is another shot at devcoredump patch. It just fixes
>> applying the patch. However, the resulting file now uses the
>> scatterlist function sg_pcopy_to_buffer() so that will need a
>> backport for kernels before v3.11.
>>
> I think, in order to make the new dev_coredumpsg() [or whatever it's
> called] function generally available, we have to also adjust the kernel
> version where the backported one gets used.
True. As there are no users right now we could opt to remove the new
function using devcoredump.patch. Or can we expect a driver using it soon.
Regards,
Arend
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] backports: patches: update devcoredump patch
2016-06-06 8:58 ` Arend Van Spriel
@ 2016-06-06 9:00 ` Johannes Berg
2016-06-06 9:10 ` Arend Van Spriel
0 siblings, 1 reply; 10+ messages in thread
From: Johannes Berg @ 2016-06-06 9:00 UTC (permalink / raw)
To: Arend Van Spriel, Luis R. Rodriguez; +Cc: backports
> > I think, in order to make the new dev_coredumpsg() [or whatever
> > it's
> > called] function generally available, we have to also adjust the
> > kernel
> > version where the backported one gets used.
> True. As there are no users right now we could opt to remove the new
> function using devcoredump.patch. Or can we expect a driver using it
> soon.
>
I have a patch for our driver to use it, which is basically only
waiting for this to get into backports :-)
johannes
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] backports: patches: update devcoredump patch
2016-06-06 9:00 ` Johannes Berg
@ 2016-06-06 9:10 ` Arend Van Spriel
2016-06-06 9:12 ` Johannes Berg
0 siblings, 1 reply; 10+ messages in thread
From: Arend Van Spriel @ 2016-06-06 9:10 UTC (permalink / raw)
To: Johannes Berg, Luis R. Rodriguez; +Cc: backports
On 6-6-2016 11:00, Johannes Berg wrote:
>
>>> I think, in order to make the new dev_coredumpsg() [or whatever
>>> it's
>>> called] function generally available, we have to also adjust the
>>> kernel
>>> version where the backported one gets used.
>> True. As there are no users right now we could opt to remove the new
>> function using devcoredump.patch. Or can we expect a driver using it
>> soon.
>>
>
> I have a patch for our driver to use it, which is basically only
> waiting for this to get into backports :-)
So how about incorporating lib/scatterlist.c in backports?
Regards,
Arend
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] backports: patches: update devcoredump patch
2016-06-06 9:10 ` Arend Van Spriel
@ 2016-06-06 9:12 ` Johannes Berg
2016-06-06 9:35 ` Arend Van Spriel
0 siblings, 1 reply; 10+ messages in thread
From: Johannes Berg @ 2016-06-06 9:12 UTC (permalink / raw)
To: Arend Van Spriel, Luis R. Rodriguez; +Cc: backports
> > I have a patch for our driver to use it, which is basically only
> > waiting for this to get into backports :-)
> So how about incorporating lib/scatterlist.c in backports?
>
We need the whole thing? I haven't really had time to look into this,
sorry.
johannes
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] backports: patches: update devcoredump patch
2016-06-06 9:12 ` Johannes Berg
@ 2016-06-06 9:35 ` Arend Van Spriel
0 siblings, 0 replies; 10+ messages in thread
From: Arend Van Spriel @ 2016-06-06 9:35 UTC (permalink / raw)
To: Johannes Berg, Luis R. Rodriguez; +Cc: backports
/On 6-6-2016 11:12, Johannes Berg wrote:
>
>>> I have a patch for our driver to use it, which is basically only
>>> waiting for this to get into backports :-)
>> So how about incorporating lib/scatterlist.c in backports?
>>
>
> We need the whole thing? I haven't really had time to look into this,
> sorry.
Here call chain:
dev_coredumpsg() ->
sg_pcopy_to_buffer() ->
sg_copy_buffer() ->
sg_miter_{start,skip,next,stop}() ->
sg_miter_get_next_page()
sg_page_iter_page()
__sg_page_iter_next()
__sg_page_iter_start()
So not everything is needed. About 350 lines from the 750 lines.
Regards,
Arend
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS
2016-06-05 9:30 [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS Arend van Spriel
2016-06-05 9:30 ` [PATCH 2/2] backports: patches: update devcoredump patch Arend van Spriel
@ 2016-07-10 21:16 ` Hauke Mehrtens
2016-07-15 10:56 ` Arend Van Spriel
1 sibling, 1 reply; 10+ messages in thread
From: Hauke Mehrtens @ 2016-07-10 21:16 UTC (permalink / raw)
To: Arend van Spriel, Luis R. Rodriguez; +Cc: backports
On 06/05/2016 11:30 AM, Arend van Spriel wrote:
> Speed definition for usb3.1 has been added in 4.6 kernel. Add
> definiton for older kernels.
>
> Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
> ---
> backport/backport-include/linux/usb/ch9.h | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/backport/backport-include/linux/usb/ch9.h b/backport/backport-include/linux/usb/ch9.h
> index 135e24c..c42fc30 100644
> --- a/backport/backport-include/linux/usb/ch9.h
> +++ b/backport/backport-include/linux/usb/ch9.h
> @@ -21,4 +21,8 @@ static inline int usb_endpoint_maxp(const struct usb_endpoint_descriptor *epd)
> }
> #endif /* < 3.2 */
>
> +#if LINUX_VERSION_CODE < KERNEL_VERSION(4,6,0)
> +#define USB_SPEED_SUPER_PLUS (USB_SPEED_SUPER + 1)
> +#endif /* < 4.6 */
> +
> #endif /* __BACKPORT__LINUX_USB_CH9_H */
>
This is in a emum in the mainline kernel. I think this will cause some
warnings when backporting it this way.
Hauke
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS
2016-07-10 21:16 ` [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS Hauke Mehrtens
@ 2016-07-15 10:56 ` Arend Van Spriel
0 siblings, 0 replies; 10+ messages in thread
From: Arend Van Spriel @ 2016-07-15 10:56 UTC (permalink / raw)
To: Hauke Mehrtens, Luis R. Rodriguez; +Cc: backports
On 10-7-2016 23:16, Hauke Mehrtens wrote:
> On 06/05/2016 11:30 AM, Arend van Spriel wrote:
>> Speed definition for usb3.1 has been added in 4.6 kernel. Add
>> definiton for older kernels.
>>
>> Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
>> ---
>> backport/backport-include/linux/usb/ch9.h | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/backport/backport-include/linux/usb/ch9.h b/backport/backport-include/linux/usb/ch9.h
>> index 135e24c..c42fc30 100644
>> --- a/backport/backport-include/linux/usb/ch9.h
>> +++ b/backport/backport-include/linux/usb/ch9.h
>> @@ -21,4 +21,8 @@ static inline int usb_endpoint_maxp(const struct usb_endpoint_descriptor *epd)
>> }
>> #endif /* < 3.2 */
>>
>> +#if LINUX_VERSION_CODE < KERNEL_VERSION(4,6,0)
>> +#define USB_SPEED_SUPER_PLUS (USB_SPEED_SUPER + 1)
>> +#endif /* < 4.6 */
>> +
>> #endif /* __BACKPORT__LINUX_USB_CH9_H */
>>
>
> This is in a emum in the mainline kernel. I think this will cause some
> warnings when backporting it this way.
The way it is used in brcmfmac I do not get a warning, but I agree it
would not work if this value is used calling a function that expects
enum usb_device_speed. So how would you backport this?
Regards,
Arend
--
To unsubscribe from this list: send the line "unsubscribe backports" in
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-07-15 10:56 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-05 9:30 [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS Arend van Spriel
2016-06-05 9:30 ` [PATCH 2/2] backports: patches: update devcoredump patch Arend van Spriel
2016-06-05 15:00 ` Johannes Berg
2016-06-06 8:58 ` Arend Van Spriel
2016-06-06 9:00 ` Johannes Berg
2016-06-06 9:10 ` Arend Van Spriel
2016-06-06 9:12 ` Johannes Berg
2016-06-06 9:35 ` Arend Van Spriel
2016-07-10 21:16 ` [PATCH 1/2] backports: add definition for USB_SPEED_SUPER_PLUS Hauke Mehrtens
2016-07-15 10:56 ` Arend Van Spriel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).