Re: [PATCH 7/9] phy: remove .owner field for drivers using module_platform_driver

Hi Kishon, On Wed, 17 Sep 2014, Kishon Vijay Abraham I wrote: Hi, On Friday 15 August 2014 06:10 PM, Peter Griffin wrote: This patch removes the superflous .owner field for drivers which use the module_platform_driver or platform_driver_register api, as this is overriden

[PATCH 0/9] phy: various misc cleanups to phy drivers

. 3) Remove OOM site specific failure messages. These are unecessary and make the kernel bigger. 4) Add missing kzalloc error check, which could avoid NULL ptr dereference. regards, Peter. Peter Griffin (9): phy: phy-omap-control: Remove unncessary site specific OOM messages phy: phy-mvebu

[PATCH 1/9] phy: phy-omap-control: Remove unncessary site specific OOM messages

will print a error message, and that any error path which maybe found which doesn't would be considered a bug in kzalloc. Signed-off-by: Peter Griffin peter.grif...@linaro.org --- drivers/phy/phy-omap-control.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/phy/phy

[PATCH 9/9] phy: phy-spear1340-miphy: Use module_platform_driver to register driver.

Using the module_platform_driver macro to register the driver as this gets rid of a lot of the boilerplate code. Also remove .owner field as this gets overridden in __platform_driver_register. Signed-off-by: Peter Griffin peter.grif...@linaro.org --- drivers/phy/phy-spear1340-miphy.c | 13

[PATCH 7/9] phy: remove .owner field for drivers using module_platform_driver

This patch removes the superflous .owner field for drivers which use the module_platform_driver or platform_driver_register api, as this is overriden in __platform_driver_register. Signed-off-by: Peter Griffin peter.grif...@linaro.org --- drivers/phy/phy-bcm-kona-usb2.c | 1 - drivers/phy

[PATCH 4/9] phy: phy-ti-pipe3: Remove unncessary site specific OOM messages

will print a error message, and that any error path which maybe found which doesn't would be considered a bug in kzalloc. Signed-off-by: Peter Griffin peter.grif...@linaro.org --- drivers/phy/phy-ti-pipe3.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/phy/phy-ti

[PATCH 8/9] phy: phy-spear1310-miphy: Use module_platform_driver to register driver.

Use the module_platform_driver macro to register the driver as this gets rid of a lot of the boilerplate code. Also remove .owner field as this gets overridden in __platform_driver_register. Signed-off-by: Peter Griffin peter.grif...@linaro.org --- drivers/phy/phy-spear1310-miphy.c | 13

[PATCH 5/9] phy: phy-spear1310-miphy: Remove unncessary site specific OOM messages

will print a error message, and that any error path which maybe found which doesn't would be considered a bug in kzalloc. Signed-off-by: Peter Griffin peter.grif...@linaro.org --- drivers/phy/phy-spear1310-miphy.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/phy/phy

[PATCH 3/9] phy: phy-omap-usb2: Remove unncessary site specific OOM messages

will print a error message, and that any error path which maybe found which doesn't would be considered a bug in kzalloc. Signed-off-by: Peter Griffin peter.grif...@linaro.org --- drivers/phy/phy-omap-usb2.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/phy/phy

[PATCH 6/9] phy: phy-spear1340-miphy: Remove unncessary site specific OOM messages

will print a error message, and that any error path which maybe found which doesn't would be considered a bug in kzalloc. Signed-off-by: Peter Griffin peter.grif...@linaro.org --- drivers/phy/phy-spear1340-miphy.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/phy/phy

[PATCH 2/9] phy: phy-mvebu-sata: Add missing error check for devm_kzalloc

Currently this driver is missing a check on the return value of devm_kzalloc, which would cause a NULL pointer dereference in a OOM situation. This patch adds the aformentioned missing check. Signed-off-by: Peter Griffin peter.grif...@linaro.org --- drivers/phy/phy-mvebu-sata.c | 2 ++ 1 file