Re: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7
Hi Sachin, On Tue, Jun 25, 2013 at 4:54 PM, Sachin Kamat sachin.ka...@linaro.org wrote: Hi Arun, @@ -684,5 +685,6 @@ void set_work_bit_irqsave(struct s5p_mfc_ctx *ctx); (dev-variant-port_num ? 1 : 0) : 0) : 0) #define IS_TWOPORT(dev)(dev-variant-port_num == 2 ? 1 : 0) #define IS_MFCV6_PLUS(dev) (dev-variant-version = 0x60 ? 1 : 0) +#define IS_MFCV7(dev) (dev-variant-version = 0x70 ? 1 : 0) Considering the definition and pattern, wouldn't it be appropriate to call this IS_MFCV7_PLUS? We are still not sure about MFCv8 if it can re-use v7 stuff or not. Regards Arun -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7
Hi Arun, On 26 June 2013 12:18, Arun Kumar K arunkk.sams...@gmail.com wrote: Hi Sachin, On Tue, Jun 25, 2013 at 4:54 PM, Sachin Kamat sachin.ka...@linaro.org wrote: Hi Arun, @@ -684,5 +685,6 @@ void set_work_bit_irqsave(struct s5p_mfc_ctx *ctx); (dev-variant-port_num ? 1 : 0) : 0) : 0) #define IS_TWOPORT(dev)(dev-variant-port_num == 2 ? 1 : 0) #define IS_MFCV6_PLUS(dev) (dev-variant-version = 0x60 ? 1 : 0) +#define IS_MFCV7(dev) (dev-variant-version = 0x70 ? 1 : 0) Considering the definition and pattern, wouldn't it be appropriate to call this IS_MFCV7_PLUS? We are still not sure about MFCv8 if it can re-use v7 stuff or not. OK. In that case probably we can restrict the definition to (dev-variant-version == 0x70 ? 1 : 0). -- With warm regards, Sachin -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7
Hi, -Original Message- From: Sachin Kamat [mailto:sachin.ka...@linaro.org] Sent: Wednesday, June 26, 2013 9:05 AM To: Arun Kumar K Cc: Arun Kumar K; LMML; Kamil Debski; jtp.p...@samsung.com; Sylwester Nawrocki; Hans Verkuil; avnd.ki...@samsung.com Subject: Re: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7 Hi Arun, On 26 June 2013 12:18, Arun Kumar K arunkk.sams...@gmail.com wrote: Hi Sachin, On Tue, Jun 25, 2013 at 4:54 PM, Sachin Kamat sachin.ka...@linaro.org wrote: Hi Arun, @@ -684,5 +685,6 @@ void set_work_bit_irqsave(struct s5p_mfc_ctx *ctx); (dev-variant-port_num ? 1 : 0) : 0) : 0) #define IS_TWOPORT(dev)(dev-variant-port_num == 2 ? 1 : 0) #define IS_MFCV6_PLUS(dev) (dev-variant-version = 0x60 ? 1 : 0) +#define IS_MFCV7(dev) (dev-variant-version = 0x70 ? 1 : 0) Considering the definition and pattern, wouldn't it be appropriate to call this IS_MFCV7_PLUS? We are still not sure about MFCv8 if it can re-use v7 stuff or not. OK. In that case probably we can restrict the definition to (dev- variant-version == 0x70 ? 1 : 0). Guys, I think that simple ((dev-variant-version 0xF0) == 0x70) would cover every 7.x version. Same could apply to versions 6.x and 5.x. Then instead of using IS_MFCV6_PLUS(dev) one would use IS_MFCV6(dev) || IS_MFCV7(dev). This is a bit longer, but if version 8 will be totally different from v7 then it is much better to use v6||v7 instead of v6_plus. Best wishes, -- Kamil Debski Linux Kernel Developer Samsung RD Institute Poland -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7
Hi Kamil, On Wed, Jun 26, 2013 at 1:50 PM, Kamil Debski k.deb...@samsung.com wrote: Hi, -Original Message- From: Sachin Kamat [mailto:sachin.ka...@linaro.org] Sent: Wednesday, June 26, 2013 9:05 AM To: Arun Kumar K Cc: Arun Kumar K; LMML; Kamil Debski; jtp.p...@samsung.com; Sylwester Nawrocki; Hans Verkuil; avnd.ki...@samsung.com Subject: Re: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7 Hi Arun, On 26 June 2013 12:18, Arun Kumar K arunkk.sams...@gmail.com wrote: Hi Sachin, On Tue, Jun 25, 2013 at 4:54 PM, Sachin Kamat sachin.ka...@linaro.org wrote: Hi Arun, @@ -684,5 +685,6 @@ void set_work_bit_irqsave(struct s5p_mfc_ctx *ctx); (dev-variant-port_num ? 1 : 0) : 0) : 0) #define IS_TWOPORT(dev)(dev-variant-port_num == 2 ? 1 : 0) #define IS_MFCV6_PLUS(dev) (dev-variant-version = 0x60 ? 1 : 0) +#define IS_MFCV7(dev) (dev-variant-version = 0x70 ? 1 : 0) Considering the definition and pattern, wouldn't it be appropriate to call this IS_MFCV7_PLUS? We are still not sure about MFCv8 if it can re-use v7 stuff or not. OK. In that case probably we can restrict the definition to (dev- variant-version == 0x70 ? 1 : 0). Guys, I think that simple ((dev-variant-version 0xF0) == 0x70) would cover every 7.x version. Same could apply to versions 6.x and 5.x. Then instead of using IS_MFCV6_PLUS(dev) one would use IS_MFCV6(dev) || IS_MFCV7(dev). This is a bit longer, but if version 8 will be totally different from v7 then it is much better to use v6||v7 instead of v6_plus. If v8 was known to be different from v7, then we can go without second thoughts about v6 || v7 check. But we cannot confirm that until v8 is released. There is a good possibility of v6_plus macro holding good for v8 also. So as per current understanding, we can restrict v7 macro to only v7.x FW. When v8 is released, we will change the name and usage of this macro if needed. Is that ok? Regards Arun -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v3 4/8] [media] s5p-mfc: Core support for MFC v7
Hi Arun, @@ -684,5 +685,6 @@ void set_work_bit_irqsave(struct s5p_mfc_ctx *ctx); (dev-variant-port_num ? 1 : 0) : 0) : 0) #define IS_TWOPORT(dev)(dev-variant-port_num == 2 ? 1 : 0) #define IS_MFCV6_PLUS(dev) (dev-variant-version = 0x60 ? 1 : 0) +#define IS_MFCV7(dev) (dev-variant-version = 0x70 ? 1 : 0) Considering the definition and pattern, wouldn't it be appropriate to call this IS_MFCV7_PLUS? --- With warm regards, Sachin -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
