[PATCH 1/2] drm/bridge: add Silicon Image SiI9234 driver
kbuild test robot
lkp at intel.com
Sat Aug 12 22:43:12 UTC 2017
Hi Maciej,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.13-rc4 next-20170811]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Maciej-Purski/add-Silicon-Image-SiI9234-driver/20170803-200255
base: git://people.freedesktop.org/~airlied/linux.git drm-next
config: i386-randconfig-h0-08130402 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
drivers/gpu//drm/bridge/sii9234.c: In function 'sii9234_mode_valid':
drivers/gpu//drm/bridge/sii9234.c:909:18: warning: unused variable 'ctx' [-Wunused-variable]
struct sii9234 *ctx = bridge_to_sii9234(bridge);
^~~
drivers/gpu//drm/bridge/sii9234.c: In function 'sii9234_mhl_tx_i2c_probe':
>> drivers/gpu//drm/bridge/sii9234.c:971:13: error: 'struct drm_bridge' has no member named 'of_node'
ctx->bridge.of_node = dev->of_node;
^
In file included from include/uapi/linux/stddef.h:1:0,
from include/linux/stddef.h:4,
from include/uapi/linux/posix_types.h:4,
from include/uapi/linux/types.h:13,
from include/linux/types.h:5,
from include/drm/bridge/mhl.h:18,
from drivers/gpu//drm/bridge/sii9234.c:26:
drivers/gpu//drm/bridge/sii9234.c: At top level:
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'strcpy' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:390:2: note: in expansion of macro 'if'
if (p_size == (size_t)-1 && q_size == (size_t)-1)
^~
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'kmemdup' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:380:2: note: in expansion of macro 'if'
if (p_size < size)
^~
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'kmemdup' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:378:2: note: in expansion of macro 'if'
if (__builtin_constant_p(size) && p_size < size)
^~
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:369:2: note: in expansion of macro 'if'
if (p_size < size)
^~
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:367:2: note: in expansion of macro 'if'
if (__builtin_constant_p(size) && p_size < size)
^~
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:358:2: note: in expansion of macro 'if'
if (p_size < size)
^~
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memchr' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:356:2: note: in expansion of macro 'if'
if (__builtin_constant_p(size) && p_size < size)
^~
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:348:2: note: in expansion of macro 'if'
if (p_size < size || q_size < size)
^~
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:345:3: note: in expansion of macro 'if'
if (q_size < size)
^~
include/linux/compiler.h:162:4: warning: '______f' is static but declared in inline function 'memcmp' which is not static
______f = { \
^
include/linux/compiler.h:154:23: note: in expansion of macro '__trace_if'
#define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
^~~~~~~~~~
include/linux/string.h:343:3: note: in expansion of macro 'if'
if (p_size < size)
^~
vim +971 drivers/gpu//drm/bridge/sii9234.c
905
906 static enum drm_mode_status sii9234_mode_valid(struct drm_bridge *bridge,
907 const struct drm_display_mode *mode)
908 {
> 909 struct sii9234 *ctx = bridge_to_sii9234(bridge);
910
911 if (mode->clock > MHL1_MAX_CLK)
912 return MODE_CLOCK_HIGH;
913
914 return MODE_OK;
915 }
916
917 static const struct drm_bridge_funcs sii9234_bridge_funcs = {
918 .mode_valid = sii9234_mode_valid,
919 };
920
921 static int sii9234_mhl_tx_i2c_probe(struct i2c_client *client,
922 const struct i2c_device_id *id)
923 {
924 struct i2c_adapter *adapter = to_i2c_adapter(client->dev.parent);
925 struct sii9234 *ctx;
926 struct device *dev = &client->dev;
927 int ret;
928
929 ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
930 if (!ctx)
931 return -ENOMEM;
932
933 ctx->dev = dev;
934
935 if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA)) {
936 dev_err(dev, "I2C adapter lacks SMBUS feature\n");
937 return -EIO;
938 }
939
940 mutex_init(&ctx->lock);
941
942 ret = sii9234_init_resources(ctx, client);
943 if (ret < 0) {
944 dev_err(&client->dev, "failed to initialize sii9234 resources\n");
945 return ret;
946 }
947 ret = sii9234_hw_on(ctx);
948 if (ret) {
949 dev_err(&client->dev, "failed to enable power\n");
950 goto err_resource;
951 }
952 sii9234_hw_reset(ctx);
953
954 if (!client->irq) {
955 dev_err(dev, "no irq provided\n");
956 return -EINVAL;
957 }
958 irq_set_status_flags(client->irq, IRQ_NOAUTOEN);
959 ret = devm_request_threaded_irq(dev, client->irq, NULL,
960 sii9234_irq_thread,
961 IRQF_TRIGGER_HIGH | IRQF_ONESHOT,
962 "sii9234", ctx);
963 if (ret < 0) {
964 dev_err(dev, "failed to install IRQ handler\n");
965 return ret;
966 }
967
968 i2c_set_clientdata(client, ctx);
969
970 ctx->bridge.funcs = &sii9234_bridge_funcs;
> 971 ctx->bridge.of_node = dev->of_node;
972 drm_bridge_add(&ctx->bridge);
973
974 sii9234_cable_in(ctx);
975
976 return 0;
977
978 err_resource:
979 sii9234_deinit_resources(ctx);
980
981 return ret;
982 }
983
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 25180 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20170813/825b66fb/attachment-0001.gz>
More information about the dri-devel
mailing list