197fb5e8dSThomas Gleixner /* SPDX-License-Identifier: GPL-2.0-only */ 225fdd593SJeykumar Sankaran /* Copyright (c) 2015-2018, The Linux Foundation. All rights reserved. 325fdd593SJeykumar Sankaran */ 425fdd593SJeykumar Sankaran 525fdd593SJeykumar Sankaran #ifndef _DPU_HW_SSPP_H 625fdd593SJeykumar Sankaran #define _DPU_HW_SSPP_H 725fdd593SJeykumar Sankaran 825fdd593SJeykumar Sankaran #include "dpu_hw_catalog.h" 925fdd593SJeykumar Sankaran #include "dpu_hw_mdss.h" 1025fdd593SJeykumar Sankaran #include "dpu_hw_util.h" 1125fdd593SJeykumar Sankaran #include "dpu_formats.h" 1225fdd593SJeykumar Sankaran 13b187794eSDmitry Baryshkov struct dpu_hw_sspp; 1425fdd593SJeykumar Sankaran 1525fdd593SJeykumar Sankaran /** 1625fdd593SJeykumar Sankaran * Flags 1725fdd593SJeykumar Sankaran */ 1825fdd593SJeykumar Sankaran #define DPU_SSPP_FLIP_LR BIT(0) 1925fdd593SJeykumar Sankaran #define DPU_SSPP_FLIP_UD BIT(1) 2025fdd593SJeykumar Sankaran #define DPU_SSPP_SOURCE_ROTATED_90 BIT(2) 2125fdd593SJeykumar Sankaran #define DPU_SSPP_ROT_90 BIT(3) 2225fdd593SJeykumar Sankaran #define DPU_SSPP_SOLID_FILL BIT(4) 2325fdd593SJeykumar Sankaran 2425fdd593SJeykumar Sankaran /** 2525fdd593SJeykumar Sankaran * Component indices 2625fdd593SJeykumar Sankaran */ 2725fdd593SJeykumar Sankaran enum { 2825fdd593SJeykumar Sankaran DPU_SSPP_COMP_0, 2925fdd593SJeykumar Sankaran DPU_SSPP_COMP_1_2, 3025fdd593SJeykumar Sankaran DPU_SSPP_COMP_2, 3125fdd593SJeykumar Sankaran DPU_SSPP_COMP_3, 3225fdd593SJeykumar Sankaran 3325fdd593SJeykumar Sankaran DPU_SSPP_COMP_MAX 3425fdd593SJeykumar Sankaran }; 3525fdd593SJeykumar Sankaran 3625fdd593SJeykumar Sankaran /** 3725fdd593SJeykumar Sankaran * DPU_SSPP_RECT_SOLO - multirect disabled 3825fdd593SJeykumar Sankaran * DPU_SSPP_RECT_0 - rect0 of a multirect pipe 3925fdd593SJeykumar Sankaran * DPU_SSPP_RECT_1 - rect1 of a multirect pipe 4025fdd593SJeykumar Sankaran * 4125fdd593SJeykumar Sankaran * Note: HW supports multirect with either RECT0 or 4225fdd593SJeykumar Sankaran * RECT1. Considering no benefit of such configs over 4325fdd593SJeykumar Sankaran * SOLO mode and to keep the plane management simple, 4425fdd593SJeykumar Sankaran * we dont support single rect multirect configs. 4525fdd593SJeykumar Sankaran */ 4625fdd593SJeykumar Sankaran enum dpu_sspp_multirect_index { 4725fdd593SJeykumar Sankaran DPU_SSPP_RECT_SOLO = 0, 4825fdd593SJeykumar Sankaran DPU_SSPP_RECT_0, 4925fdd593SJeykumar Sankaran DPU_SSPP_RECT_1, 5025fdd593SJeykumar Sankaran }; 5125fdd593SJeykumar Sankaran 5225fdd593SJeykumar Sankaran enum dpu_sspp_multirect_mode { 5325fdd593SJeykumar Sankaran DPU_SSPP_MULTIRECT_NONE = 0, 5425fdd593SJeykumar Sankaran DPU_SSPP_MULTIRECT_PARALLEL, 5525fdd593SJeykumar Sankaran DPU_SSPP_MULTIRECT_TIME_MX, 5625fdd593SJeykumar Sankaran }; 5725fdd593SJeykumar Sankaran 5825fdd593SJeykumar Sankaran enum { 5925fdd593SJeykumar Sankaran DPU_FRAME_LINEAR, 6025fdd593SJeykumar Sankaran DPU_FRAME_TILE_A4X, 6125fdd593SJeykumar Sankaran DPU_FRAME_TILE_A5X, 6225fdd593SJeykumar Sankaran }; 6325fdd593SJeykumar Sankaran 6425fdd593SJeykumar Sankaran enum dpu_hw_filter { 6525fdd593SJeykumar Sankaran DPU_SCALE_FILTER_NEAREST = 0, 6625fdd593SJeykumar Sankaran DPU_SCALE_FILTER_BIL, 6725fdd593SJeykumar Sankaran DPU_SCALE_FILTER_PCMN, 6825fdd593SJeykumar Sankaran DPU_SCALE_FILTER_CA, 6925fdd593SJeykumar Sankaran DPU_SCALE_FILTER_MAX 7025fdd593SJeykumar Sankaran }; 7125fdd593SJeykumar Sankaran 7225fdd593SJeykumar Sankaran enum dpu_hw_filter_alpa { 7325fdd593SJeykumar Sankaran DPU_SCALE_ALPHA_PIXEL_REP, 7425fdd593SJeykumar Sankaran DPU_SCALE_ALPHA_BIL 7525fdd593SJeykumar Sankaran }; 7625fdd593SJeykumar Sankaran 7725fdd593SJeykumar Sankaran enum dpu_hw_filter_yuv { 7825fdd593SJeykumar Sankaran DPU_SCALE_2D_4X4, 7925fdd593SJeykumar Sankaran DPU_SCALE_2D_CIR, 8025fdd593SJeykumar Sankaran DPU_SCALE_1D_SEP, 8125fdd593SJeykumar Sankaran DPU_SCALE_BIL 8225fdd593SJeykumar Sankaran }; 8325fdd593SJeykumar Sankaran 8425fdd593SJeykumar Sankaran struct dpu_hw_sharp_cfg { 8525fdd593SJeykumar Sankaran u32 strength; 8625fdd593SJeykumar Sankaran u32 edge_thr; 8725fdd593SJeykumar Sankaran u32 smooth_thr; 8825fdd593SJeykumar Sankaran u32 noise_thr; 8925fdd593SJeykumar Sankaran }; 9025fdd593SJeykumar Sankaran 9125fdd593SJeykumar Sankaran struct dpu_hw_pixel_ext { 9225fdd593SJeykumar Sankaran /* scaling factors are enabled for this input layer */ 9325fdd593SJeykumar Sankaran uint8_t enable_pxl_ext; 9425fdd593SJeykumar Sankaran 9525fdd593SJeykumar Sankaran int init_phase_x[DPU_MAX_PLANES]; 9625fdd593SJeykumar Sankaran int phase_step_x[DPU_MAX_PLANES]; 9725fdd593SJeykumar Sankaran int init_phase_y[DPU_MAX_PLANES]; 9825fdd593SJeykumar Sankaran int phase_step_y[DPU_MAX_PLANES]; 9925fdd593SJeykumar Sankaran 10025fdd593SJeykumar Sankaran /* 10125fdd593SJeykumar Sankaran * Number of pixels extension in left, right, top and bottom direction 10225fdd593SJeykumar Sankaran * for all color components. This pixel value for each color component 10325fdd593SJeykumar Sankaran * should be sum of fetch + repeat pixels. 10425fdd593SJeykumar Sankaran */ 10525fdd593SJeykumar Sankaran int num_ext_pxls_left[DPU_MAX_PLANES]; 10625fdd593SJeykumar Sankaran int num_ext_pxls_right[DPU_MAX_PLANES]; 10725fdd593SJeykumar Sankaran int num_ext_pxls_top[DPU_MAX_PLANES]; 10825fdd593SJeykumar Sankaran int num_ext_pxls_btm[DPU_MAX_PLANES]; 10925fdd593SJeykumar Sankaran 11025fdd593SJeykumar Sankaran /* 11125fdd593SJeykumar Sankaran * Number of pixels needs to be overfetched in left, right, top and 11225fdd593SJeykumar Sankaran * bottom directions from source image for scaling. 11325fdd593SJeykumar Sankaran */ 11425fdd593SJeykumar Sankaran int left_ftch[DPU_MAX_PLANES]; 11525fdd593SJeykumar Sankaran int right_ftch[DPU_MAX_PLANES]; 11625fdd593SJeykumar Sankaran int top_ftch[DPU_MAX_PLANES]; 11725fdd593SJeykumar Sankaran int btm_ftch[DPU_MAX_PLANES]; 11825fdd593SJeykumar Sankaran 11925fdd593SJeykumar Sankaran /* 12025fdd593SJeykumar Sankaran * Number of pixels needs to be repeated in left, right, top and 12125fdd593SJeykumar Sankaran * bottom directions for scaling. 12225fdd593SJeykumar Sankaran */ 12325fdd593SJeykumar Sankaran int left_rpt[DPU_MAX_PLANES]; 12425fdd593SJeykumar Sankaran int right_rpt[DPU_MAX_PLANES]; 12525fdd593SJeykumar Sankaran int top_rpt[DPU_MAX_PLANES]; 12625fdd593SJeykumar Sankaran int btm_rpt[DPU_MAX_PLANES]; 12725fdd593SJeykumar Sankaran 12825fdd593SJeykumar Sankaran uint32_t roi_w[DPU_MAX_PLANES]; 12925fdd593SJeykumar Sankaran uint32_t roi_h[DPU_MAX_PLANES]; 13025fdd593SJeykumar Sankaran 13125fdd593SJeykumar Sankaran /* 13225fdd593SJeykumar Sankaran * Filter type to be used for scaling in horizontal and vertical 13325fdd593SJeykumar Sankaran * directions 13425fdd593SJeykumar Sankaran */ 13525fdd593SJeykumar Sankaran enum dpu_hw_filter horz_filter[DPU_MAX_PLANES]; 13625fdd593SJeykumar Sankaran enum dpu_hw_filter vert_filter[DPU_MAX_PLANES]; 13725fdd593SJeykumar Sankaran 13825fdd593SJeykumar Sankaran }; 13925fdd593SJeykumar Sankaran 14025fdd593SJeykumar Sankaran /** 1410d06fb90SDmitry Baryshkov * struct dpu_sw_pipe_cfg : software pipe configuration 14225fdd593SJeykumar Sankaran * @src_rect: src ROI, caller takes into account the different operations 14325fdd593SJeykumar Sankaran * such as decimation, flip etc to program this field 14425fdd593SJeykumar Sankaran * @dest_rect: destination ROI. 14525fdd593SJeykumar Sankaran */ 1460d06fb90SDmitry Baryshkov struct dpu_sw_pipe_cfg { 14725fdd593SJeykumar Sankaran struct drm_rect src_rect; 14825fdd593SJeykumar Sankaran struct drm_rect dst_rect; 14925fdd593SJeykumar Sankaran }; 15025fdd593SJeykumar Sankaran 15125fdd593SJeykumar Sankaran /** 15225fdd593SJeykumar Sankaran * struct dpu_hw_pipe_ts_cfg - traffic shaper configuration 15325fdd593SJeykumar Sankaran * @size: size to prefill in bytes, or zero to disable 15425fdd593SJeykumar Sankaran * @time: time to prefill in usec, or zero to disable 15525fdd593SJeykumar Sankaran */ 15625fdd593SJeykumar Sankaran struct dpu_hw_pipe_ts_cfg { 15725fdd593SJeykumar Sankaran u64 size; 15825fdd593SJeykumar Sankaran u64 time; 15925fdd593SJeykumar Sankaran }; 16025fdd593SJeykumar Sankaran 16125fdd593SJeykumar Sankaran /** 1623cfcd130SDmitry Baryshkov * struct dpu_sw_pipe - software pipe description 1633cfcd130SDmitry Baryshkov * @sspp: backing SSPP pipe 1643cfcd130SDmitry Baryshkov * @index: index of the rectangle of SSPP 1653cfcd130SDmitry Baryshkov * @mode: parallel or time multiplex multirect mode 1663cfcd130SDmitry Baryshkov */ 1673cfcd130SDmitry Baryshkov struct dpu_sw_pipe { 1683cfcd130SDmitry Baryshkov struct dpu_hw_sspp *sspp; 1693cfcd130SDmitry Baryshkov enum dpu_sspp_multirect_index multirect_index; 1703cfcd130SDmitry Baryshkov enum dpu_sspp_multirect_mode multirect_mode; 1713cfcd130SDmitry Baryshkov }; 1723cfcd130SDmitry Baryshkov 1733cfcd130SDmitry Baryshkov /** 17425fdd593SJeykumar Sankaran * struct dpu_hw_sspp_ops - interface to the SSPP Hw driver functions 17525fdd593SJeykumar Sankaran * Caller must call the init function to get the pipe context for each pipe 17625fdd593SJeykumar Sankaran * Assumption is these functions will be called after clocks are enabled 17725fdd593SJeykumar Sankaran */ 17825fdd593SJeykumar Sankaran struct dpu_hw_sspp_ops { 17925fdd593SJeykumar Sankaran /** 18025fdd593SJeykumar Sankaran * setup_format - setup pixel format cropping rectangle, flip 18174fd7fdaSDmitry Baryshkov * @pipe: Pointer to software pipe context 18225fdd593SJeykumar Sankaran * @cfg: Pointer to pipe config structure 18325fdd593SJeykumar Sankaran * @flags: Extra flags for format config 18425fdd593SJeykumar Sankaran */ 18574fd7fdaSDmitry Baryshkov void (*setup_format)(struct dpu_sw_pipe *pipe, 186*0e67f514SDmitry Baryshkov const struct msm_format *fmt, u32 flags); 18725fdd593SJeykumar Sankaran 18825fdd593SJeykumar Sankaran /** 18925fdd593SJeykumar Sankaran * setup_rects - setup pipe ROI rectangles 19074fd7fdaSDmitry Baryshkov * @pipe: Pointer to software pipe context 19125fdd593SJeykumar Sankaran * @cfg: Pointer to pipe config structure 19225fdd593SJeykumar Sankaran */ 19374fd7fdaSDmitry Baryshkov void (*setup_rects)(struct dpu_sw_pipe *pipe, 1940d06fb90SDmitry Baryshkov struct dpu_sw_pipe_cfg *cfg); 19525fdd593SJeykumar Sankaran 19625fdd593SJeykumar Sankaran /** 19725fdd593SJeykumar Sankaran * setup_pe - setup pipe pixel extension 19825fdd593SJeykumar Sankaran * @ctx: Pointer to pipe context 19925fdd593SJeykumar Sankaran * @pe_ext: Pointer to pixel ext settings 20025fdd593SJeykumar Sankaran */ 201b187794eSDmitry Baryshkov void (*setup_pe)(struct dpu_hw_sspp *ctx, 20225fdd593SJeykumar Sankaran struct dpu_hw_pixel_ext *pe_ext); 20325fdd593SJeykumar Sankaran 20425fdd593SJeykumar Sankaran /** 20525fdd593SJeykumar Sankaran * setup_sourceaddress - setup pipe source addresses 20674fd7fdaSDmitry Baryshkov * @pipe: Pointer to software pipe context 207dfdc94e4SDmitry Baryshkov * @layout: format layout information for programming buffer to hardware 20825fdd593SJeykumar Sankaran */ 20974fd7fdaSDmitry Baryshkov void (*setup_sourceaddress)(struct dpu_sw_pipe *ctx, 210dfdc94e4SDmitry Baryshkov struct dpu_hw_fmt_layout *layout); 21125fdd593SJeykumar Sankaran 21225fdd593SJeykumar Sankaran /** 21325fdd593SJeykumar Sankaran * setup_csc - setup color space coversion 21425fdd593SJeykumar Sankaran * @ctx: Pointer to pipe context 21525fdd593SJeykumar Sankaran * @data: Pointer to config structure 21625fdd593SJeykumar Sankaran */ 217b187794eSDmitry Baryshkov void (*setup_csc)(struct dpu_hw_sspp *ctx, const struct dpu_csc_cfg *data); 21825fdd593SJeykumar Sankaran 21925fdd593SJeykumar Sankaran /** 22025fdd593SJeykumar Sankaran * setup_solidfill - enable/disable colorfill 22174fd7fdaSDmitry Baryshkov * @pipe: Pointer to software pipe context 22225fdd593SJeykumar Sankaran * @const_color: Fill color value 22325fdd593SJeykumar Sankaran * @flags: Pipe flags 22425fdd593SJeykumar Sankaran */ 22574fd7fdaSDmitry Baryshkov void (*setup_solidfill)(struct dpu_sw_pipe *pipe, u32 color); 22625fdd593SJeykumar Sankaran 22725fdd593SJeykumar Sankaran /** 22825fdd593SJeykumar Sankaran * setup_multirect - setup multirect configuration 22974fd7fdaSDmitry Baryshkov * @pipe: Pointer to software pipe context 23025fdd593SJeykumar Sankaran */ 23125fdd593SJeykumar Sankaran 23274fd7fdaSDmitry Baryshkov void (*setup_multirect)(struct dpu_sw_pipe *pipe); 23325fdd593SJeykumar Sankaran 23425fdd593SJeykumar Sankaran /** 23525fdd593SJeykumar Sankaran * setup_sharpening - setup sharpening 23625fdd593SJeykumar Sankaran * @ctx: Pointer to pipe context 23725fdd593SJeykumar Sankaran * @cfg: Pointer to config structure 23825fdd593SJeykumar Sankaran */ 239b187794eSDmitry Baryshkov void (*setup_sharpening)(struct dpu_hw_sspp *ctx, 24025fdd593SJeykumar Sankaran struct dpu_hw_sharp_cfg *cfg); 24125fdd593SJeykumar Sankaran 24225fdd593SJeykumar Sankaran 24325fdd593SJeykumar Sankaran /** 2445fe0faa6SDmitry Baryshkov * setup_qos_lut - setup QoS LUTs 24525fdd593SJeykumar Sankaran * @ctx: Pointer to pipe context 2465fe0faa6SDmitry Baryshkov * @cfg: LUT configuration 24725fdd593SJeykumar Sankaran */ 2485fe0faa6SDmitry Baryshkov void (*setup_qos_lut)(struct dpu_hw_sspp *ctx, 2495fe0faa6SDmitry Baryshkov struct dpu_hw_qos_cfg *cfg); 25025fdd593SJeykumar Sankaran 25125fdd593SJeykumar Sankaran /** 25225fdd593SJeykumar Sankaran * setup_qos_ctrl - setup QoS control 25325fdd593SJeykumar Sankaran * @ctx: Pointer to pipe context 254a5ebb27bSDmitry Baryshkov * @danger_safe_en: flags controlling enabling of danger/safe QoS/LUT 25525fdd593SJeykumar Sankaran */ 256b187794eSDmitry Baryshkov void (*setup_qos_ctrl)(struct dpu_hw_sspp *ctx, 257a5ebb27bSDmitry Baryshkov bool danger_safe_en); 25825fdd593SJeykumar Sankaran 25925fdd593SJeykumar Sankaran /** 26087e96867SNeil Armstrong * setup_clk_force_ctrl - setup clock force control 26187e96867SNeil Armstrong * @ctx: Pointer to pipe context 26287e96867SNeil Armstrong * @enable: enable clock force if true 26387e96867SNeil Armstrong */ 26487e96867SNeil Armstrong bool (*setup_clk_force_ctrl)(struct dpu_hw_sspp *ctx, 26587e96867SNeil Armstrong bool enable); 26687e96867SNeil Armstrong 26787e96867SNeil Armstrong /** 26825fdd593SJeykumar Sankaran * setup_histogram - setup histograms 26925fdd593SJeykumar Sankaran * @ctx: Pointer to pipe context 27025fdd593SJeykumar Sankaran * @cfg: Pointer to histogram configuration 27125fdd593SJeykumar Sankaran */ 272b187794eSDmitry Baryshkov void (*setup_histogram)(struct dpu_hw_sspp *ctx, 27325fdd593SJeykumar Sankaran void *cfg); 27425fdd593SJeykumar Sankaran 27525fdd593SJeykumar Sankaran /** 27625fdd593SJeykumar Sankaran * setup_scaler - setup scaler 2776edb12d1SDmitry Baryshkov * @scaler3_cfg: Pointer to scaler configuration 2786edb12d1SDmitry Baryshkov * @format: pixel format parameters 27925fdd593SJeykumar Sankaran */ 280b187794eSDmitry Baryshkov void (*setup_scaler)(struct dpu_hw_sspp *ctx, 2816edb12d1SDmitry Baryshkov struct dpu_hw_scaler3_cfg *scaler3_cfg, 282*0e67f514SDmitry Baryshkov const struct msm_format *format); 28325fdd593SJeykumar Sankaran 28425fdd593SJeykumar Sankaran /** 28525fdd593SJeykumar Sankaran * setup_cdp - setup client driven prefetch 28674fd7fdaSDmitry Baryshkov * @pipe: Pointer to software pipe context 28748b3207eSDmitry Baryshkov * @fmt: format used by the sw pipe 28848b3207eSDmitry Baryshkov * @enable: whether the CDP should be enabled for this pipe 28925fdd593SJeykumar Sankaran */ 29074fd7fdaSDmitry Baryshkov void (*setup_cdp)(struct dpu_sw_pipe *pipe, 291*0e67f514SDmitry Baryshkov const struct msm_format *fmt, 29248b3207eSDmitry Baryshkov bool enable); 29325fdd593SJeykumar Sankaran }; 29425fdd593SJeykumar Sankaran 29525fdd593SJeykumar Sankaran /** 296b187794eSDmitry Baryshkov * struct dpu_hw_sspp - pipe description 29725fdd593SJeykumar Sankaran * @base: hardware block base structure 29825fdd593SJeykumar Sankaran * @hw: block hardware details 299babdb815SMarijn Suijten * @ubwc: UBWC configuration data 30025fdd593SJeykumar Sankaran * @idx: pipe index 30125fdd593SJeykumar Sankaran * @cap: pointer to layer_cfg 30225fdd593SJeykumar Sankaran * @ops: pointer to operations possible for this pipe 30325fdd593SJeykumar Sankaran */ 304b187794eSDmitry Baryshkov struct dpu_hw_sspp { 30525fdd593SJeykumar Sankaran struct dpu_hw_blk base; 30625fdd593SJeykumar Sankaran struct dpu_hw_blk_reg_map hw; 307a2e87e9eSDmitry Baryshkov const struct msm_mdss_data *ubwc; 30825fdd593SJeykumar Sankaran 30925fdd593SJeykumar Sankaran /* Pipe */ 31025fdd593SJeykumar Sankaran enum dpu_sspp idx; 31125fdd593SJeykumar Sankaran const struct dpu_sspp_cfg *cap; 31225fdd593SJeykumar Sankaran 31325fdd593SJeykumar Sankaran /* Ops */ 31425fdd593SJeykumar Sankaran struct dpu_hw_sspp_ops ops; 31525fdd593SJeykumar Sankaran }; 31625fdd593SJeykumar Sankaran 3172672e4e7SDmitry Baryshkov struct dpu_kms; 31825fdd593SJeykumar Sankaran /** 319babdb815SMarijn Suijten * dpu_hw_sspp_init() - Initializes the sspp hw driver object. 32025fdd593SJeykumar Sankaran * Should be called once before accessing every pipe. 321a106ed98SDmitry Baryshkov * @dev: Corresponding device for devres management 322babdb815SMarijn Suijten * @cfg: Pipe catalog entry for which driver object is required 32325fdd593SJeykumar Sankaran * @addr: Mapped register io address of MDP 324a2e87e9eSDmitry Baryshkov * @mdss_data: UBWC / MDSS configuration data 32587e96867SNeil Armstrong * @mdss_rev: dpu core's major and minor versions 32625fdd593SJeykumar Sankaran */ 327a106ed98SDmitry Baryshkov struct dpu_hw_sspp *dpu_hw_sspp_init(struct drm_device *dev, 328a106ed98SDmitry Baryshkov const struct dpu_sspp_cfg *cfg, 329a106ed98SDmitry Baryshkov void __iomem *addr, 330a106ed98SDmitry Baryshkov const struct msm_mdss_data *mdss_data, 33187e96867SNeil Armstrong const struct dpu_mdss_version *mdss_rev); 33225fdd593SJeykumar Sankaran 333b187794eSDmitry Baryshkov int _dpu_hw_sspp_init_debugfs(struct dpu_hw_sspp *hw_pipe, struct dpu_kms *kms, 334b187794eSDmitry Baryshkov struct dentry *entry); 3352672e4e7SDmitry Baryshkov 33625fdd593SJeykumar Sankaran #endif /*_DPU_HW_SSPP_H */ 33725fdd593SJeykumar Sankaran 338