linux-toradex.git/drivers/i2c/busses/i2c-highlander.c, branch v4.4.38

i2c: busses: drop owner assignment from platform_drivers

2014-10-20T14:20:37+00:00

A platform_driver does not need to set an owner, it will be populated by the
driver core.

Signed-off-by: Wolfram Sang

i2c: delete non-required instances of include

2014-01-24T17:24:31+00:00

None of these files are actually using any __init type directives
and hence don't need to include .  Most are just a
left over from __devinit and __cpuinit removal, or simply due to
code getting copied from one driver to the next.

Signed-off-by: Paul Gortmaker 
Acked-by: Jean Delvare 
Signed-off-by: Wolfram Sang

i2c: Remove unneeded xxx_set_drvdata(..., NULL) calls

2013-02-21T23:25:50+00:00

There is simply no reason to be manually setting the private driver
data to NULL in the remove/fail to probe cases.  This is just extra
cruft code that can be removed.

A few notes:
* Nothing relies on drvdata being set to NULL.
* The __device_release_driver() function eventually calls
  dev_set_drvdata(dev, NULL) anyway, so there's no need to do it
  twice.
* I verified that there were no cases where xxx_get_drvdata() was
  being called in these drivers and checking for / relying on the NULL
  return value.

This could be cleaned up kernel-wide but for now just take the baby
step and remove from the i2c subsystem.

Reported-by: Wolfram Sang 
Reported-by: Stephen Warren 
Signed-off-by: Doug Anderson 
Reviewed-by: Jean Delvare 
Acked-by: Peter Korsgaard 
Reviewed-by: Mika Westerberg 
Reviewed-by: Marek Vasut 
Signed-off-by: Wolfram Sang

i2c: remove __dev* attributes from subsystem

2012-12-22T19:13:45+00:00

CONFIG_HOTPLUG is going away as an option.  As result the __dev*
markings will be going away.

Remove use of __devinit, __devexit_p, __devinitdata, __devinitconst,
and __devexit.

Signed-off-by: Bill Pemberton 
Acked-by: Peter Korsgaard  (for ocores and mux-gpio)
Acked-by: Havard Skinnemoen  (for i2c-gpio)
Acked-by: Guan Xuetao  (for puf3)
Acked-by: Barry Song  (for sirf)
Reviewed-by: Jean Delvare 
[wsa: Fixed "foo* bar" flaws while we are here]
Signed-off-by: Wolfram Sang

Merge tag 'for-linus' of git://github.com/rustyrussell/linux

2012-01-14T20:32:16+00:00

Autogenerated GPG tag for Rusty D1ADB8F1: 15EE 8D6C AB0E 7F0C F999  BFCB D920 0E6C D1AD B8F1

* tag 'for-linus' of git://github.com/rustyrussell/linux:
  module_param: check that bool parameters really are bool.
  intelfbdrv.c: bailearly is an int module_param
  paride/pcd: fix bool verbose module parameter.
  module_param: make bool parameters really bool (drivers & misc)
  module_param: make bool parameters really bool (arch)
  module_param: make bool parameters really bool (core code)
  kernel/async: remove redundant declaration.
  printk: fix unnecessary module_param_name.
  lirc_parallel: fix module parameter description.
  module_param: avoid bool abuse, add bint for special cases.
  module_param: check type correctness for module_param_array
  modpost: use linker section to generate table.
  modpost: use a table rather than a giant if/else statement.
  modules: sysfs - export: taint, coresize, initsize
  kernel/params: replace DEBUGP with pr_debug
  module: replace DEBUGP with pr_debug
  module: struct module_ref should contains long fields
  module: Fix performance regression on modules with large symbol tables
  module: Add comments describing how the "strmap" logic works

Fix up conflicts in scripts/mod/file2alias.c due to the new linker-
generated table approach to adding __mod_*_device_table entries.  The
ARM sa11x0 mcp bus needed to be converted to that too.

module_param: make bool parameters really bool (drivers & misc)

2012-01-12T23:02:20+00:00

module_param(bool) used to counter-intuitively take an int.  In
fddd5201 (mid-2009) we allowed bool or int/unsigned int using a messy
trick.

It's time to remove the int/unsigned int option.  For this version
it'll simply give a warning, but it'll break next kernel version.

Acked-by: Mauro Carvalho Chehab 
Signed-off-by: Rusty Russell

i2c/busses: Use module_platform_driver()

2012-01-12T19:32:04+00:00

Convert the drivers in drivers/i2c/busses/* to use the
module_platform_driver() macro which makes the code smaller and a bit
simpler.

Cc: Ben Dooks 
Acked-by: Jochen Friedrich 
Acked-by: Peter Korsgaard 
Acked-by: Wolfram Sang 
Cc: Manuel Lauss 
Cc: Barry Song <21cnbao@gmail.com>
Cc: Linus Walleij 
Cc: Yong Zhang 
Cc: Lucas De Marchi 
Cc: Grant Likely 
Cc: Samuel Ortiz 
Signed-off-by: Axel Lin 
Signed-off-by: Jean Delvare

i2c: irq: Remove IRQF_DISABLED

2011-10-29T08:55:45+00:00

Since commit [c58543c8: genirq: Run irq handlers with interrupts disabled],
We run all interrupt handlers with interrupts disabled
and we even check and yell when an interrupt handler
returns with interrupts enabled (see commit [b738a50a:
genirq: Warn when handler enables interrupts]).

So now this flag is a NOOP and can be removed.

Signed-off-by: Yong Zhang 
Acked-by: Wolfram Sang 
Acked-by: Linus Walleij 
Acked-by: Barry Song <21cnbao@gmail.com>
Signed-off-by: Ben Dooks

treewide: transciever/transceiver spelling fixes

2011-06-24T09:20:14+00:00

Just tyops.

Signed-off-by: Joe Perches 
Signed-off-by: Jiri Kosina

i2c-highlander: remover superflous variable

2010-05-19T23:19:01+00:00

When cppcheck found this flaw

[./i2c/busses/i2c-highlander.c:284]: (style) Warning - using char variable in bit operation

it was noted that the 'read'-variable could be simply removed as read_write can
only be 0 or 1 anyhow. So, we remove the flaw and simplify the code.

Reported-by: d binderman 
Signed-off-by: Wolfram Sang 
Acked-by: Paul Mundt 
Acked-by: Jean Delvare 
Cc: Ben Dooks