Re: [PATCH RESEND 0/7] Introduce MEDIA_VERSION to end KENREL_VERSION abuse in media

2017-06-30 Thread Johannes Thumshirn
a new version, but kill all the versioning stuff out of media (and others) except for maybe the HW version. > Any upstream driver should never use KERNEL_VERSION(). Exactly my reasoning. -- Johannes Thumshirn Storage jthumsh...@suse.de

Re: [PATCH RESEND 0/7] Introduce MEDIA_VERSION to end KENREL_VERSION abuse in media

2017-06-29 Thread Johannes Thumshirn
) shouldn't have in-tree users IMHO. Yes there is _one_ other user of it in-tree which is EXT4 and I already talked to Jan Kara about it and we decided to leave it in until 4.20. Byte, Johannes -- Johannes Thumshirn

[PATCH RESEND 2/7] video: fbdev: don't use KERNEL_VERSION macro for MEDIA_REVISION

2017-06-21 Thread Johannes Thumshirn
Don't use the KERNEL_VERSION() macro for the v4l2 capabilities, use MEDIA_REVISION instead. Signed-off-by: Johannes Thumshirn --- drivers/video/fbdev/matrox/matroxfb_base.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/video/fbdev/matrox/matroxfb_base

[PATCH RESEND 6/7] [media] media: bcm2048: use MEDIA_REVISION isntead of KERNEL_VERSION

2017-06-21 Thread Johannes Thumshirn
Use MEDIA_REVISION isntead of KERNEL_VERSION to encode the bcm2048 driver version. Signed-off-by: Johannes Thumshirn --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers

[PATCH RESEND 7/7] staging/atomisp: use MEDIA_VERSION instead of KERNEL_VERSION

2017-06-21 Thread Johannes Thumshirn
Use MEDIA_VERSION instead of KERNEL_VERSION to encode the driver version of the Atom ISP driver. Signed-off-by: Johannes Thumshirn --- drivers/staging/media/atomisp/include/linux/atomisp.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/media/atomisp

[PATCH RESEND 5/7] [media] media: s3c-camif: Use MEDIA_REVISON instead of KERNEL_VERSION

2017-06-21 Thread Johannes Thumshirn
Use MEDIA_REVISON instead of KERNEL_VERSION to encode the driver version. Signed-off-by: Johannes Thumshirn --- drivers/media/platform/s3c-camif/camif-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/s3c-camif/camif-core.c b/drivers/media

[PATCH RESEND 3/7] [media] media: document the use of MEDIA_REVISION instead of KERNEL_VERSION

2017-06-21 Thread Johannes Thumshirn
Update the documentation to introduce the use of MEDIA_REVISON instead of KERNEL_VERSION for the verison triplets of a media drivers hardware revision or driver version. Signed-off-by: Johannes Thumshirn --- Documentation/media/uapi/cec/cec-ioc-adap-g-caps.rst| 2 +- Documentation/media

[PATCH RESEND 1/7] [media] media: introduce MEDIA_REVISION macro

2017-06-21 Thread Johannes Thumshirn
Currently the media code abuses the KERNEL_VERSION macro to encode a version triplet. Introduce a MEDIA_REVISION macro to get rid of the confusing and creative KERNEL_VERSION usage in the media subsystem. Signed-off-by: Johannes Thumshirn --- include/uapi/linux/media.h | 4 +++- 1 file changed

[PATCH RESEND 4/7] [media] cx25821: use MEDIA_REVISION instead of KERNEL_VERSION

2017-06-21 Thread Johannes Thumshirn
Use MEDIA_REVISION instead of KERNEL_VERSION to encode the CX25821_VERSION_CODE. Signed-off-by: Johannes Thumshirn --- drivers/media/pci/cx25821/cx25821.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/cx25821/cx25821.h b/drivers/media/pci/cx25821/cx25821

[PATCH RESEND 0/7] Introduce MEDIA_VERSION to end KENREL_VERSION abuse in media

2017-06-21 Thread Johannes Thumshirn
clearly has media centric semantics and doesn't fool someone into thinking specific parts are defined for a specific kernel version only like in out of tree drivers. Johannes Thumshirn (7): [media] media: introduce MEDIA_REVISION macro video: fbdev: don't use KERNEL_VERSION macro for MEDI

[PATCH 6/7] [media] media: bcm2048: use MEDIA_REVISION isntead of KERNEL_VERSION

2017-06-10 Thread Johannes Thumshirn
Use MEDIA_REVISION isntead of KERNEL_VERSION to encode the bcm2048 driver version. Signed-off-by: Johannes Thumshirn --- drivers/staging/media/bcm2048/radio-bcm2048.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers

[PATCH 7/7] staging/atomisp: use MEDIA_VERSION instead of KERNEL_VERSION

2017-06-10 Thread Johannes Thumshirn
Use MEDIA_VERSION instead of KERNEL_VERSION to encode the driver version of the Atom ISP driver. Signed-off-by: Johannes Thumshirn --- drivers/staging/media/atomisp/include/linux/atomisp.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/media/atomisp

[PATCH 5/7] [media] media: s3c-camif: Use MEDIA_REVISON instead of KERNEL_VERSION

2017-06-10 Thread Johannes Thumshirn
Use MEDIA_REVISON instead of KERNEL_VERSION to encode the driver version. Signed-off-by: Johannes Thumshirn --- drivers/media/platform/s3c-camif/camif-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/s3c-camif/camif-core.c b/drivers/media

[PATCH 4/7] [media] cx25821: use MEDIA_REVISION instead of KERNEL_VERSION

2017-06-10 Thread Johannes Thumshirn
Use MEDIA_REVISION instead of KERNEL_VERSION to encode the CX25821_VERSION_CODE. Signed-off-by: Johannes Thumshirn --- drivers/media/pci/cx25821/cx25821.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/cx25821/cx25821.h b/drivers/media/pci/cx25821/cx25821

[PATCH 0/7] Introduce MEDIA_VERSION to end KENREL_VERSION abuse in media

2017-06-10 Thread Johannes Thumshirn
clearly has media centric semantics and doesn't fool someone into thinking specific parts are defined for a specific kernel version only like in out of tree drivers. Johannes Thumshirn (7): [media] media: introduce MEDIA_REVISION macro video: fbdev: don't use KERNEL_VERSION macro for MEDI

[PATCH 2/7] video: fbdev: don't use KERNEL_VERSION macro for MEDIA_REVISION

2017-06-10 Thread Johannes Thumshirn
Don't use the KERNEL_VERSION() macro for the v4l2 capabilities, use MEDIA_REVISION instead. Signed-off-by: Johannes Thumshirn --- drivers/video/fbdev/matrox/matroxfb_base.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/video/fbdev/matrox/matroxfb_base

[PATCH 3/7] [media] media: document the use of MEDIA_REVISION instead of KERNEL_VERSION

2017-06-10 Thread Johannes Thumshirn
Update the documentation to introduce the use of MEDIA_REVISON instead of KERNEL_VERSION for the verison triplets of a media drivers hardware revision or driver version. Signed-off-by: Johannes Thumshirn --- Documentation/media/uapi/cec/cec-ioc-adap-g-caps.rst| 2 +- Documentation/media

[PATCH 1/7] [media] media: introduce MEDIA_REVISION macro

2017-06-10 Thread Johannes Thumshirn
Currently the media code abuses the KERNEL_VERSION macro to encode a version triplet. Introduce a MEDIA_REVISION macro to get rid of the confusing and creative KERNEL_VERSION usage in the media subsystem. Signed-off-by: Johannes Thumshirn --- include/uapi/linux/media.h | 4 +++- 1 file changed

Re: [PATCH 22/29] drivers, scsi: convert iscsi_task.refcount from atomic_t to refcount_t

2017-03-09 Thread Johannes Thumshirn
r review! Do you have a tree that can take this change? >> >> Hi Elena, >> >> iscsi like fcoe should go via the SCSI tree. > > Thanks Johannes! Should I resend with "Acked-by" added in order for it to be > picked up? Yes I think this would be a good way

Re: [PATCH 22/29] drivers, scsi: convert iscsi_task.refcount from atomic_t to refcount_t

2017-03-09 Thread Johannes Thumshirn
>> >> This looks OK to me. >> >> Acked-by: Chris Leech > > Thank you for review! Do you have a tree that can take this change? Hi Elena, iscsi like fcoe should go via the SCSI tree. Byte, Johannes -- Johannes Thumshirn

Re: [PATCH 21/29] drivers, s390: convert fc_fcp_pkt.ref_cnt from atomic_t to refcount_t

2017-03-08 Thread Johannes Thumshirn
-after-free situations. The subject is wrong, should be something like "scsi: libfc convert fc_fcp_pkt.ref_cnt from atomic_t to refcount_t" but not s390. Other than that Acked-by: Johannes Thumshirn Turns out that it is better that all these patches go through the respective mainta

Re: [PATCH 21/29] drivers, s390: convert fc_fcp_pkt.ref_cnt from atomic_t to refcount_t

2017-03-06 Thread Johannes Thumshirn
The subject is wrong, should be something like "scsi: libfc convert fc_fcp_pkt.ref_cnt from atomic_t to refcount_t" but not s390. Other than that Acked-by: Johannes Thumshirn -- Johannes Thumshirn Storage jthumsh...@suse.de

Re: [PATCHv1] net-next: treewide use is_vlan_dev() helper function.

2017-02-05 Thread Johannes Thumshirn
On 02/04/2017 06:00 PM, Parav Pandit wrote: This patch makes use of is_vlan_dev() function instead of flag comparison which is exactly done by is_vlan_dev() helper function. Signed-off-by: Parav Pandit Reviewed-by: Daniel Jurgens --- For drivers/scsi/fcoe/fcoe.c: Acked-by: Johannes

Re: [PATCH 1/3] storvsc: use tagged SRB requests if supported by the device

2016-09-07 Thread Johannes Thumshirn
ction = SRB_SIMPLE_TAG_REQUEST; > + } > + > /* Build the SRB */ > switch (scmnd->sc_data_direction) { > case DMA_TO_DEVICE: > -- > 1.8.5.6 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-sc

Re: [PATCH V2 2/2] scsi: storvsc: Use the specified target ID in device lookup

2016-01-28 Thread Johannes Thumshirn
srb->target_id; > INIT_WORK(&wrk->work, process_err_fn); > schedule_work(&wrk->work); > } > -- > 1.7.4.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-scsi" in > the body of a message to majord...@vge

Re: [PATCH V2 1/2] scsi: storvsc: Install the storvsc specific timeout handler for FC devices

2016-01-28 Thread Johannes Thumshirn
d the line "unsubscribe linux-scsi" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html Reviewed-by: Johannes Thumshirn -- Johannes Thumshirn Storage jthumsh...@suse.de

Re: [PATCH 4/4] scsi: storvsc: Tighten up the interrupt path

2015-12-11 Thread Johannes Thumshirn
vmscsi_size_delta)); > complete(&request->wait_event); > } else { > - storvsc_on_receive(device, > + storvsc_on_receive(stor_device, >

Re: [PATCH 3/4] scsi: storvsc: Refactor the code in storvsc_channel_init()

2015-12-11 Thread Johannes Thumshirn
e_delta), > -(unsigned long)request, > -VM_PKT_DATA_INBAND, > -VMBUS_DATA_PACKET_FLAG_COMPLETION_REQUESTED); > - > - if (ret != 0) > - goto cleanup; > - > - t = wait_f

Re: [PATCH 2/4] scsi: storvsc: Properly support Fibre Channel devices

2015-12-11 Thread Johannes Thumshirn
))); > > - return vmbus_driver_register(&storvsc_drv); > + fc_transport_template = fc_attach_transport(&fc_transport_functions); > + if (!fc_transport_template) > + return -ENODEV; > + > + ret = vmbus_driver_register(&storvsc_drv); >

Re: [PATCH 1/4] scsi: storvsc: Fix a bug in the layout of the hv_fc_wwn_packet

2015-12-11 Thread Johannes Thumshirn
/vger.kernel.org/majordomo-info.html Should this go through stable as well? If yes: Fixes: 8b612fa23 '[SCSI] storvsc: Update the storage protocol to win8 level' Cc: sta...@vger.kernel.org # v3.11+ Reviewed-by: Johannes Thumshirn -- Johannes Thumshirn

Re: [PATCH RESEND V2 3/7] scsi: storvsc: Untangle the storage protocol negotiation from the vmbus protocol negotiation.

2015-08-13 Thread Johannes Thumshirn
KY Srinivasan writes: >> -Original Message- >> From: Johannes Thumshirn [mailto:jthumsh...@suse.de] >> Sent: Thursday, August 13, 2015 7:34 AM >> To: KY Srinivasan ; Keith Mange >> >> Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; &

Re: [PATCH RESEND V2 3/7] scsi: storvsc: Untangle the storage protocol negotiation from the vmbus protocol negotiation.

2015-08-13 Thread Johannes Thumshirn
POST_WIN7_STORVSC_SENSE_BUFFER_SIZE, > + 0 > + }, > + { > + VMSTOR_PROTO_VERSION_WIN7, > + PRE_WIN8_STORVSC_SENSE_BUFFER_SIZE, > + sizeof(struct vmscsi_win8_extension), > + }, > + { > + VMSTOR_PROT