[PATCH v2] clk: keystone: gate: fix clk_init_data initialization

2014-01-30 Thread Ivan Khoronzhuk
for most of clocks. As result a bunch of unused clocks cannot be disabled. So initialize flags in this structure too. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@ti.com --- Based on git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git keystone/master v1..v2: - updated commit

[PATCH v2] ARM: keystone: dts: drop msmcsram clock node

2014-01-30 Thread Ivan Khoronzhuk
. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@ti.com --- Based on git://git.kernel.org/pub/scm/linux/kernel/git/ssantosh/linux-keystone.git keystone/master v1..v2: - dropped msmcsram clock node, instead disabling - updated comment message arch/arm/boot/dts/keystone-clocks.dtsi | 10 -- 1

[PATCH v3 0/2] Introduce AEMIF driver for Davinci/Keystone archs

2013-12-11 Thread Ivan Khoronzhuk
-aemif: introduce AEMIF driver - memory: ti-aemif: add bindings for AEMIF driver added ti.cs-chipselect property instead of representing chipselect number in cs node name. Ivan Khoronzhuk (2): memory: ti-aemif: introduce AEMIF driver memory: ti-aemif: add bindings for AEMIF driver

[PATCH v3 2/2] memory: ti-aemif: add bindings for AEMIF driver

2013-12-11 Thread Ivan Khoronzhuk
to it, as result we can't encode CS number in reg for AEMIF child devices (NAND/NOR/etc), as it will break bindings compatibility. In this patch, NAND node is used just as an example of child node. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@ti.com --- .../bindings/memory-controllers/ti-aemif.txt

[PATCH v3 1/2] memory: ti-aemif: introduce AEMIF driver

2013-12-11 Thread Ivan Khoronzhuk
selects with 64M byte access per chip select. Synchronous memories such as DDR1 SD RAM, SDR SDRAM and Mobile SDR are not supported. This controller is used on SoCs like Davinci, Keysone2 Signed-off-by: [initial author] Murali Karicheri m-kariche...@ti.com Signed-off-by: Ivan Khoronzhuk ivan.khoronz

Re: [PATCH v2 2/5] Power: reset: add bindings for keystone reset driver

2014-05-05 Thread Ivan Khoronzhuk
On 04/15/2014 02:25 PM, Ivan Khoronzhuk wrote: On 04/14/2014 09:44 PM, Arnd Bergmann wrote: On Monday 14 April 2014 20:41:20 Ivan Khoronzhuk wrote: +Optional properties: + +- ti,soft-reset: Boolean option indicating soft reset. + By default hard reset is used

Re: [PATCH v2] mtd: davinci-nand: disable subpage write for keystone-nand

2014-04-08 Thread Ivan Khoronzhuk
On 03/20/2014 10:14 PM, Santosh Shilimkar wrote: On Thursday 20 March 2014 04:08 PM, Ivan Khoronzhuk wrote: From: Murali Karicheri m-kariche...@ti.com Sub page write doesn't work because of hw issue in controller found on Keystone SOCs. AEMIF controller is also used on DaVinci SOCs which

Re: [PATCH v2 2/5] Power: reset: add bindings for keystone reset driver

2014-04-15 Thread Ivan Khoronzhuk
On 04/14/2014 09:44 PM, Arnd Bergmann wrote: On Monday 14 April 2014 20:41:20 Ivan Khoronzhuk wrote: +Optional properties: + +- ti,soft-reset: Boolean option indicating soft reset. + By default hard reset is used. + +- ti,wdt_list: WDT list that can cause

Re: [PATCH v2] mtd: davinci-nand: disable subpage write for keystone-nand

2014-04-16 Thread Ivan Khoronzhuk
On 04/16/2014 10:08 AM, Brian Norris wrote: On Tue, Apr 08, 2014 at 03:51:34PM +0300, Ivan Khoronzhuk wrote: On 03/20/2014 10:14 PM, Santosh Shilimkar wrote: On Thursday 20 March 2014 04:08 PM, Ivan Khoronzhuk wrote: From: Murali Karicheri m-kariche...@ti.com Sub page write doesn't work

[PATCH v2 4/5] ARM: dts: keystone: update reset node to work with reset driver

2014-04-14 Thread Ivan Khoronzhuk
. Also add range for reset multiplex registers for SoC on the device. These registers are located in Bootcfg memory space and needed to setup behaviour after appropriate watchdog is triggered. Add ti,wdt_list option to declare what watchdog are used to reboot the SoC. Signed-off-by: Ivan Khoronzhuk

[PATCH v2 1/5] Power: reset: keystone-reset: introduce keystone reset driver

2014-04-14 Thread Ivan Khoronzhuk
to use. So add keystone reset driver to handle all this stuff. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@ti.com --- drivers/power/reset/Kconfig | 7 ++ drivers/power/reset/Makefile | 1 + drivers/power/reset/keystone-reset.c | 171 +++ 3

[PATCH v2 3/5] ARM: keystone: remove redundant reset stuff

2014-04-14 Thread Ivan Khoronzhuk
Remove reset stuff in flavour of using keystone reset driver: driver/power/reset/keystone-reset.c Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@ti.com --- arch/arm/mach-keystone/keystone.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/arch/arm/mach

[PATCH v2 0/5] Introduce keystone reset driver

2014-04-14 Thread Ivan Khoronzhuk
- re basedon on v3.15-rc1 without changes Ivan Khoronzhuk (5): Power: reset: keystone-reset: introduce keystone reset driver Power: reset: add bindings for keystone reset driver ARM: keystone: remove redundant reset stuff ARM: dts: keystone: update reset node to work with reset driver

[PATCH v2 5/5] ARM: keystone: enable reset driver support

2014-04-14 Thread Ivan Khoronzhuk
Enable reset driver support in order to have opportunity to reboot SoC by watchdog and by software. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@ti.com --- arch/arm/configs/keystone_defconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/configs/keystone_defconfig b/arch/arm

[PATCH v2 2/5] Power: reset: add bindings for keystone reset driver

2014-04-14 Thread Ivan Khoronzhuk
or not. Additionally soft or hard reset can be configured. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@ti.com --- .../bindings/power/reset/keystone-reset.txt| 59 ++ 1 file changed, 59 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/reset/keystone

Re: [PATCH 2/3] mtd: hisilicon: add a new nand controller driver for hisilicon hip04 Soc

2014-06-30 Thread Ivan Khoronzhuk
), HINFC504_DMA_LEN); + + hinfc_write(host, HINFC504_DMA_PARA_DATA_RW_EN + | HINFC504_DMA_PARA_OOB_RW_EN, HINFC504_DMA_PARA); + } else Add {} for else also ./scripts/checkpatch.pl. -- Regards, Ivan Khoronzhuk -- To unsubscribe from this list: send the line

Re: [PATCH v5 1/3] clocksource: timer-keystone: introduce clocksource driver for Keystone

2014-02-06 Thread Ivan Khoronzhuk
On 02/06/2014 02:35 AM, Josh Cartwright wrote: Hey Ivan- On Wed, Feb 05, 2014 at 03:47:38PM +0200, Ivan Khoronzhuk wrote: Add broadcast clock-event device for the Keystone arch. The timer can be configured as a general-purpose 64-bit timer, dual general-purpose 32-bit timers. When configured

Re: [PATCH v5 2/3] clocksource: keystone: add bindings for keystone timer

2014-02-06 Thread Ivan Khoronzhuk
On 02/06/2014 01:36 AM, Rob Herring wrote: On Wed, Feb 5, 2014 at 12:52 PM, Ivan Khoronzhuk ivan.khoronz...@ti.com wrote: On 02/05/2014 07:41 PM, Rob Herring wrote: On Wed, Feb 5, 2014 at 10:18 AM, Ivan Khoronzhuk ivan.khoronz...@ti.com wrote: On 02/05/2014 04:39 PM, Rob Herring wrote

Re: [PATCH v5 1/3] clocksource: timer-keystone: introduce clocksource driver for Keystone

2014-02-06 Thread Ivan Khoronzhuk
On 02/05/2014 10:27 PM, Thomas Gleixner wrote: On Wed, 5 Feb 2014, Ivan Khoronzhuk wrote: + /* here we have to be sure the timer has been disabled */ Sigh. This is not a proper explanation for a barrier, really. You want to explain what it serializes against what. i.e. you want

[Patch v2] nand: davinci: add support for 4K page size nand devices

2014-08-19 Thread Ivan Khoronzhuk
From: Sandeep Paulraj s-paul...@ti.com It is needed for k2l keystone2 EVM which uses NAND flash with 4K page size, hence add support for 4K page size nand devices. Signed-off-by: Sandeep Paulraj s-paul...@ti.com Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@ti.com --- Based on l2-mtd/master

Re: [PATCH 08/10] power/reset: keystone: Register with kernel restart handler

2014-09-29 Thread Ivan Khoronzhuk
On 09/27/2014 03:57 AM, Guenter Roeck wrote: Register with kernel restart handler instead of setting arm_pm_restart directly. Move notifier registration to the end of the probe function to avoid having to implement error handling. Cc: Ivan Khoronzhuk ivan.khoronz...@ti.com Cc: Santosh

[Patch v2 2/2] firmware: dmi-sysfs: add SMBIOS entry point area attribute

2015-01-26 Thread Ivan Khoronzhuk
There are situations when code needs to access SMBIOS entry table area, but cannot use /dev/mem for this. As the table format is consistent only for a version, and can be changed, use binary attribute to give access to raw SMBIOS entry table area. Signed-off-by: Ivan Khoronzhuk ivan.khoronz

[Patch v2 1/2] firmware: dmi_scan: add symbol to get SMBIOS entry area

2015-01-26 Thread Ivan Khoronzhuk
There are situations when code needs to access SMBIOS entry table area. For example, to pass it via sysfs to userspace when it's not allowed to get SMBIOS info via /dev/mem. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- drivers/firmware/dmi_scan.c | 26

[Patch v2 0/2] firmware: dmi-sysfs: add SMBIOS entry point area raw attribute

2015-01-26 Thread Ivan Khoronzhuk
- changed format of get_smbios_entry_area symbol returned pointer on const smbios table firmware: dmi-sysfs: add SMBIOS entry point area attribute - adopted to updated get_smbios_entry_area symbol - removed redundant array to save smbios table Ivan Khoronzhuk

[PATCH 1/2] firmware: dmi_scan: add symbol to get SMBIOS entry area

2015-01-23 Thread Ivan Khoronzhuk
There are situations when code needs to access SMBIOS entry table area. For example, to pass it via sysfs to userspace when it's not allowed to get SMBIOS info via /dev/mem. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- drivers/firmware/dmi_scan.c | 34

[PATCH 0/2] firmware: dmi-sysfs: add SMBIOS

2015-01-23 Thread Ivan Khoronzhuk
entry. So this patch series adds SMBIOS area to dmi sysfs in order to allow utils in question to work correctly with dmi sysfs. Ivan Khoronzhuk (2): firmware: dmi_scan: add symbol to get SMBIOS entry area firmware: dmi-sysfs: add SMBIOS entry point area attribute drivers/firmware/dmi-sysfs.c

[PATCH 2/2] firmware: dmi-sysfs: add SMBIOS entry point area attribute

2015-01-23 Thread Ivan Khoronzhuk
There are situations when code needs to access SMBIOS entry table area, but cannot use /dev/mem for this. As the table format is consistent only for a version, and can be changed, use binary attribute to give access to raw SMBIOS entry table area. Signed-off-by: Ivan Khoronzhuk ivan.khoronz

[Patch v4] firmware: dmi-sysfs: add SMBIOS entry point area attribute

2015-02-04 Thread Ivan Khoronzhuk
entry. So this patch adds SMBIOS area to dmi-sysfs in order to allow utils in question to work correctly with dmi sysfs interface. Reviewed-by: Ard Biesheuvel ard.biesheu...@linaro.org Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- v1: https://lkml.org/lkml/2015/1/23/643 v2: https

Re: [Patch v3] firmware: dmi-sysfs: add SMBIOS entry point area raw attribute

2015-02-03 Thread Ivan Khoronzhuk
On 02/03/2015 12:49 PM, Matt Fleming wrote: On Wed, 28 Jan, at 05:56:25PM, Ivan Khoronzhuk wrote: diff --git a/drivers/firmware/dmi-sysfs.c b/drivers/firmware/dmi-sysfs.c index e0f1cb3..61b6a38 100644 --- a/drivers/firmware/dmi-sysfs.c +++ b/drivers/firmware/dmi-sysfs.c @@ -29,6 +29,8

Re: [Patch v3] firmware: dmi-sysfs: add SMBIOS entry point area raw attribute

2015-02-03 Thread Ivan Khoronzhuk
Hi, Mark On 02/03/2015 04:58 PM, Mark Salter wrote: On Wed, 2015-01-28 at 14:39 +0200, Ivan Khoronzhuk wrote: Some utils, like dmidecode and smbios, needs to access SMBIOS entry table area in order to get information like SMBIOS version, size, etc. Currently it's done via /dev/mem

Re: [Patch] firmware: dmi_scan: fix dmi scan to handle End of Table structure

2015-02-04 Thread Ivan Khoronzhuk
On 02/04/2015 04:04 PM, Ard Biesheuvel wrote: On 4 February 2015 at 13:57, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: The dmi-sysfs should create End of Table entry, that is type 127. But after adding initial SMBIOS v3 support the 127-0 entry is not handled any more, as result it's

[Patch] firmware: dmi_scan: fix dmi scan to handle End of Table structure

2015-02-04 Thread Ivan Khoronzhuk
, it can be handled in newly introduced libdmifs by adding end entry virtually, but it's brake backward compatibility and don't correspond to SMBIOS DMI table size read from SMBIOS entry point structure. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- drivers/firmware/dmi_scan.c | 6

[Patch v2] firmware: dmi_scan: fix dmi scan to handle End of Table structure

2015-02-04 Thread Ivan Khoronzhuk
end-of-table check after it's handled by decode. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- v2..v1: Move end of table check after it's handled instead of removing Correct commit drivers/firmware/dmi_scan.c | 13 +++-- 1 file changed, 7 insertions(+), 6

Re: [Patch v4] firmware: dmi-sysfs: add SMBIOS entry point area attribute

2015-02-11 Thread Ivan Khoronzhuk
On 02/11/2015 04:43 PM, Matt Fleming wrote: On Wed, 11 Feb, at 02:17:03PM, Matt Fleming wrote: On Tue, 10 Feb, at 11:51:44AM, Ivan Khoronzhuk wrote: If you are Ok with this patch, could you please pickup it? Applied, thanks Ivan! Btw this patch doesn't apply cleanly, the reject looks like

[Patch v2 0/3] firmware: dmi_scan: add some SMBIOSv3 corrections

2015-02-18 Thread Ivan Khoronzhuk
for SMBIOS3 - use shifted dmi_ver var for all versions - display docrev version for 32 bit 3+ versions like 3.2.x - don't display docrev for versions 3 firmware: dmi_scan: use direct access to static vars - move as the last patch and leave only corrections Ivan

[Patch v2 1/3] firmware: dmi_scan: fix dmi_len type

2015-02-18 Thread Ivan Khoronzhuk
cannot imagine dmi structure count more than 65535 and it can require changing type of vars that work with it. So I didn't correct it. Acked-by: Ard Biesheuvel a...@linaro.org Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- drivers/firmware/dmi_scan.c | 4 ++-- 1 file changed, 2

[Patch v2 3/3] firmware: dmi_scan: use direct access to static vars

2015-02-18 Thread Ivan Khoronzhuk
There is no reason to pass static vars to function that can use only them. The dmi_table() can use only dmi_len and dmi_num static vars, so use them directly. In this case we can freely change their type in one place and slightly decrease redundancy. Signed-off-by: Ivan Khoronzhuk ivan.khoronz

Re: [Patch v2] firmware: dmi_scan: fix dmi scan to handle End of Table structure

2015-02-18 Thread Ivan Khoronzhuk
Hi, Matt Could you please pick up this patch. On 02/05/2015 12:36 PM, Ard Biesheuvel wrote: On 4 February 2015 at 14:22, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: The dmi-sysfs should create End of Table entry, that is type 127. But after adding initial SMBIOS v3 support the 127-0

[Patch v2 2/3] firmware: dmi_scan: use full dmi version for SMBIOS3

2015-02-18 Thread Ivan Khoronzhuk
don't know the docrev. In other cases display version like it was. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- drivers/firmware/dmi_scan.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/drivers/firmware/dmi_scan.c b/drivers/firmware

Re: [Patch v2] firmware: dmi_scan: fix dmi scan to handle End of Table structure

2015-02-18 Thread Ivan Khoronzhuk
Ok, I'll correct commit msg like: But after adding initial SMBIOS v3 support (fc43026278b23b3515cf8f909ec29df94b3ae1a2) the 127-0 entry is nothandled any more, as result it's not created in sysfs. On 02/18/2015 03:04 PM, Matt Fleming wrote: On Wed, 04 Feb, at 04:22:05PM, Ivan Khoronzhuk

Re: [Patch v2] firmware: dmi_scan: fix dmi scan to handle End of Table structure

2015-02-18 Thread Ivan Khoronzhuk
On 02/18/2015 03:38 PM, Matt Fleming wrote: On 18 February 2015 at 13:20, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: Ok, I'll correct commit msg like: But after adding initial SMBIOS v3 support (fc43026278b23b3515cf8f909ec29df94b3ae1a2) the 127-0 entry is nothandled any more, as result

[Patch v3] firmware: dmi_scan: fix dmi scan to handle End of Table structure

2015-02-18 Thread Ivan Khoronzhuk
the size of whole DMI table must correspond to sum of all DMI entry sizes. So move the end-of-table check after it's handled by dmi_table. Reviewed-by: Ard Biesheuvel a...@linaro.org Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- v3..v2: Include regression commit in commit

Re: [Patch v2 0/2] firmware: dmi-sysfs: add SMBIOS entry point area raw attribute

2015-01-27 Thread Ivan Khoronzhuk
On 01/27/2015 06:23 PM, Grant Likely wrote: On Mon, 26 Jan 2015 15:28:36 +0200 , Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: Some utils, like dmidecode and smbios, needs to access SMBIOS entry table area in order to get information like SMBIOS version, size, etc. Currently it's done

[Patch v3] firmware: dmi-sysfs: add SMBIOS entry point area raw attribute

2015-01-28 Thread Ivan Khoronzhuk
entry. So this patch adds SMBIOS area to dmi-sysfs in order to allow utils in question to work correctly with dmi sysfs interface. Reviewed-by: Ard Biesheuvel ard.biesheu...@linaro.org Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- v1: https://lkml.org/lkml/2015/1/23/643 v2: https

Re: [Patch v3] firmware: dmi-sysfs: add SMBIOS entry point area raw attribute

2015-01-28 Thread Ivan Khoronzhuk
+ linux-...@vger.kernel.org + linux-...@vger.kernel.org On 01/28/2015 02:39 PM, Ivan Khoronzhuk wrote: Some utils, like dmidecode and smbios, needs to access SMBIOS entry table area in order to get information like SMBIOS version, size, etc. Currently it's done via /dev/mem. But for situation

Re: [PATCH] firmware: efi: remove unneeded guid unparse

2015-01-29 Thread Ivan Khoronzhuk
Oh, sorry. Just didn't note that it's picked up from linaro list. On 01/29/2015 04:47 PM, Matt Fleming wrote: On Thu, 2015-01-29 at 16:35 +0200, Ivan Khoronzhuk wrote: There is no reason to translate guid number to string here. So remove it in order to not do unneeded work. Signed-off

[PATCH] firmware: efi: remove unneeded guid unparse

2015-01-29 Thread Ivan Khoronzhuk
There is no reason to translate guid number to string here. So remove it in order to not do unneeded work. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- drivers/firmware/efi/efi.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/firmware/efi/efi.c b/drivers

Re: [PATCH 1/2] firmware: dmi_scan: add symbol to get SMBIOS entry area

2015-01-26 Thread Ivan Khoronzhuk
On 01/26/2015 10:44 AM, Ard Biesheuvel wrote: On 23 January 2015 at 20:21, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: There are situations when code needs to access SMBIOS entry table area. For example, to pass it via sysfs to userspace when it's not allowed to get SMBIOS info via /dev

Re: [Patch v4] firmware: dmi-sysfs: add SMBIOS entry point area attribute

2015-02-10 Thread Ivan Khoronzhuk
Hi Matt, On 02/04/2015 07:06 PM, Ivan Khoronzhuk wrote: Some utils, like dmidecode and smbios, need to access SMBIOS entry table area in order to get information like SMBIOS version, size, etc. Currently it's done via /dev/mem. But for situation when /dev/mem usage is disabled, the utils have

[Patch 2/3] firmware: dmi_scan: fix dmi_len type

2015-02-11 Thread Ivan Khoronzhuk
cannot imagine dmi structure count more than 65535 and it can require changing type of vars that work with it. So I didn't correct it. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- drivers/firmware/dmi_scan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Patch 1/3] firmware: dmi_scan: use direct access to static vars

2015-02-11 Thread Ivan Khoronzhuk
There is no reason to pass static vars to function that can use only them. The dmi_table() can use only dmi_len and dmi_num static vars, so use them directly. In this case we can freely change their type in one place and slightly decrease redundancy. Signed-off-by: Ivan Khoronzhuk ivan.khoronz

Re: [Patch 3/3] firmware: dmi_scan: use full dmi version for SMBIOS3

2015-02-11 Thread Ivan Khoronzhuk
On 02/11/2015 11:55 AM, Ard Biesheuvel wrote: On 11 February 2015 at 17:46, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: New SMBIOS3 spec adds additional field for versioning - docrev. The docrev identifies the revision of a specification implemented in the table structures, so display

[Patch 3/3] firmware: dmi_scan: use full dmi version for SMBIOS3

2015-02-11 Thread Ivan Khoronzhuk
, and it's obvious that 0x000208 is less than 0x030201 for example. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@linaro.org --- drivers/firmware/dmi_scan.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/firmware/dmi_scan.c b/drivers/firmware/dmi_scan.c

[Patch 0/3] firmware: dmi_scan: add some SMBIOSv3 corrections

2015-02-11 Thread Ivan Khoronzhuk
This series adds corrections to dmi_scan: - extends version to be like 3.4.5 format - fix dmi_len to be 32 bit wide Ivan Khoronzhuk (3): firmware: dmi_scan: use direct access to static vars firmware: dmi_scan: fix dmi_len type firmware: dmi_scan: use full dmi version for SMBIOS3 drivers

Re: [Patch 2/3] firmware: dmi_scan: fix dmi_len type

2015-02-11 Thread Ivan Khoronzhuk
On 02/11/2015 11:53 AM, Ard Biesheuvel wrote: On 11 February 2015 at 17:46, Ivan Khoronzhuk ivan.khoronz...@linaro.org wrote: According to SMBIOSv3 specification the length of DMI table can be up to 32bits wide. So use appropriate type to avoid overflow. It's obvious that dmi_num

Re: [Patch 2/3] firmware: dmi_scan: fix dmi_len type

2015-02-14 Thread Ivan Khoronzhuk
On 02/13/2015 06:12 PM, Matt Fleming wrote: On Wed, 11 Feb, at 06:12:59PM, Ard Biesheuvel wrote: Good point. Actually, I don't really see the need for patch #1, even if I agree that it would have been better to write it like you have in the first place. But leaving the dmi_len as u16 is

[Patch] firmware: dmi_scan: split dmisubsystem from dmi-sysfs

2015-03-16 Thread Ivan Khoronzhuk
by Jean Delvare. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@globallogic.com --- This patch is logical continuation of [dmidecode] [Patch v4] firmware: dmi-sysfs: add SMBIOS entry point area attribute https://lkml.org/lkml/2015/2/4/475 Pay attention that this includes /sys/firmware/dmi

[Patch 2/3] firmware: dmi_scan: add SBMIOS entry and DMI tables

2015-04-02 Thread Ivan Khoronzhuk
Delvare. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@globallogic.com --- .../ABI/testing/sysfs-firmware-dmi-tables | 22 ++ drivers/firmware/dmi-sysfs.c | 11 ++- drivers/firmware/dmi_scan.c| 80 ++ include/linux/dmi.h

[Patch 0/3] firmware: dmi_scan: add SBMIOS entry point and DMI tables

2015-04-02 Thread Ivan Khoronzhuk
export dmi_kobj if CONFIG_DMI is not set - improve read callback for table attributes Ivan Khoronzhuk (3): firmware: dmi_scan: rename dmi_table to dmi_decode_table firmware: dmi_scan: add SBMIOS entry and DMI tables Documentation: ABI: sysfs-firmware-dmi: add -entries suffix to file

[Patch 3/3] Documentation: ABI: sysfs-firmware-dmi: add -entries suffix to file name

2015-04-02 Thread Ivan Khoronzhuk
kobject. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@globallogic.com --- .../ABI/testing/{sysfs-firmware-dmi = sysfs-firmware-dmi-entries} | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) rename Documentation/ABI/testing/{sysfs-firmware-dmi = sysfs-firmware-dmi-entries} (99%) diff

[Patch 1/3] firmware: dmi_scan: rename dmi_table to dmi_decode_table

2015-04-02 Thread Ivan Khoronzhuk
The dmi_table function looks like data instance, but it does DMI table decode. This patch renames it to dmi_decode_table name as more appropriate. That allows us to use dmi_table name for correct purposes. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@globallogic.com --- drivers/firmware

[Patch v2 1/3] firmware: dmi_scan: rename dmi_table to dmi_decode_table

2015-04-20 Thread Ivan Khoronzhuk
The dmi_table function looks like data instance, but it does DMI table decode. This patch renames it to dmi_decode_table name as more appropriate. That allows us to use dmi_table name for correct purposes. Reviewed-by: Jean Delvare jdelv...@suse.de Signed-off-by: Ivan Khoronzhuk ivan.khoronz

[Patch v2 2/3] firmware: dmi_scan: add SBMIOS entry and DMI tables

2015-04-20 Thread Ivan Khoronzhuk
Delvare. Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@globallogic.com --- .../ABI/testing/sysfs-firmware-dmi-tables | 22 ++ drivers/firmware/dmi-sysfs.c | 17 +++-- drivers/firmware/dmi_scan.c| 82 ++ include/linux

[Patch v2 3/3] Documentation: ABI: sysfs-firmware-dmi: add -entries suffix to file name

2015-04-20 Thread Ivan Khoronzhuk
kobject. Reviewed-by: Jean Delvare jdelv...@suse.de Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@globallogic.com --- .../ABI/testing/{sysfs-firmware-dmi = sysfs-firmware-dmi-entries} | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) rename Documentation/ABI/testing/{sysfs-firmware-dmi

[Patch v2 0/3] firmware: dmi_scan: add SBMIOS entry point and DMI tables

2015-04-20 Thread Ivan Khoronzhuk
Ivan Khoronzhuk (3): firmware: dmi_scan: rename dmi_table to dmi_decode_table firmware: dmi_scan: add SBMIOS entry and DMI tables Documentation: ABI: sysfs-firmware-dmi: add -entries suffix to file name ...sfs-firmware-dmi = sysfs-firmware-dmi-entries} | 2 +- .../ABI/testing/sysfs

[Patch v3 1/3] firmware: dmi_scan: rename dmi_table to dmi_decode_table

2015-04-21 Thread Ivan Khoronzhuk
The dmi_table function looks like data instance, but it does DMI table decode. This patch renames it to dmi_decode_table name as more appropriate. That allows us to use dmi_table name for correct purposes. Reviewed-by: Jean Delvare jdelv...@suse.de Signed-off-by: Ivan Khoronzhuk ivan.khoronz

[Patch v3 2/3] firmware: dmi_scan: add SBMIOS entry and DMI tables

2015-04-21 Thread Ivan Khoronzhuk
Delvare. Tested-by: Roy Franz roy.fr...@linaro.org Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@globallogic.com --- .../ABI/testing/sysfs-firmware-dmi-tables | 22 ++ drivers/firmware/dmi-sysfs.c | 17 +++-- drivers/firmware/dmi_scan.c| 78

[Patch v3 0/3] firmware: dmi_scan: add SBMIOS entry point and DMI tables

2015-04-21 Thread Ivan Khoronzhuk
path in dmi-sysfs - don't use globally dmi_table var - use DMI in attribute name - correct error path in dmi_init - leave dmi_kobj even in case of error - include linux/kobject.h in header Ivan Khoronzhuk (3): firmware: dmi_scan: rename dmi_table

[Patch v3 3/3] Documentation: ABI: sysfs-firmware-dmi: add -entries suffix to file name

2015-04-21 Thread Ivan Khoronzhuk
kobject. Reviewed-by: Jean Delvare jdelv...@suse.de Signed-off-by: Ivan Khoronzhuk ivan.khoronz...@globallogic.com --- .../ABI/testing/{sysfs-firmware-dmi = sysfs-firmware-dmi-entries} | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) rename Documentation/ABI/testing/{sysfs-firmware-dmi

Re: [Patch v2] nand: davinci: add support for 4K page size nand devices

2015-07-20 Thread Ivan Khoronzhuk
On 20.07.15 20:32, Brian Norris wrote: On Tue, Aug 19, 2014 at 03:31:54PM +0300, Ivan Khoronzhuk wrote: From: Sandeep Paulraj s-paul...@ti.com It is needed for k2l keystone2 EVM which uses NAND flash with 4K page size, hence add support for 4K page size nand devices. Signed-off-by: Sandeep

[PATCH] net: ethernet: ti: cpsw: remove rx_descs property

2016-06-02 Thread Ivan Khoronzhuk
x pool can be used in full. It will not have any impact on performance, as anyway, the "redundant" descriptors were unused. Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- Based on master Documentation/devicetree/bindings/net/cpsw.txt | 3 --- arch/arm/boot/dts/am33xx.

[PATCH] net: ethernet: ti: cpsw: remove unused priv lock

2016-06-02 Thread Ivan Khoronzhuk
There is no reason in this lock. At least for now. Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- Based on master drivers/net/ethernet/ti/cpsw.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c index 9

Re: [PATCH] net: ethernet: ti: cpsw: remove rx_descs property

2016-06-03 Thread Ivan Khoronzhuk
On 03.06.16 19:50, Grygorii Strashko wrote: On 06/03/2016 01:43 AM, Ivan Khoronzhuk wrote: There is no reason to hold s/w dependent parameter in device tree. Even more, there is no reason in this parameter because davinici_cpdma driver splits pool of descriptors equally between tx and rx

Re: [PATCH] net: ethernet: ti: cpsw: remove rx_descs property

2016-06-03 Thread Ivan Khoronzhuk
On 03.06.16 22:13, Grygorii Strashko wrote: On 06/03/2016 09:25 PM, Ivan Khoronzhuk wrote: On 03.06.16 19:50, Grygorii Strashko wrote: On 06/03/2016 01:43 AM, Ivan Khoronzhuk wrote: There is no reason to hold s/w dependent parameter in device tree. Even more, there is no reason

Re: [PATCH v2 0/2] net: ethernet: ti: cpsw: delete rx_descs property

2016-06-08 Thread Ivan Khoronzhuk
On 09.06.16 02:11, Schuyler Patton wrote: On 06/08/2016 09:06 AM, Ivan Khoronzhuk wrote: On 08.06.16 17:01, Ivan Khoronzhuk wrote: Hi Schuyer, On 07.06.16 18:26, Schuyler Patton wrote: Hi, On 06/07/2016 08:59 AM, Ivan Khoronzhuk wrote: There is no reason in rx_descs property because

[PATCH v2 0/2] net: ethernet: ti: cpsw: delete rx_descs property

2016-06-07 Thread Ivan Khoronzhuk
patches - return number of rx buffers from cpdma driver Ivan Khoronzhuk (2): net: ethernet: ti: cpsw: remove rx_descs property Documentation: DT: cpsw: remove rx_descs property Documentation/devicetree/bindings/net/cpsw.txt | 1 - arch/arm/boot/dts/am33xx.dtsi | 1 - arch/arm

[PATCH v2 1/2] net: ethernet: ti: cpsw: remove rx_descs property

2016-06-07 Thread Ivan Khoronzhuk
as anyway, the "redundant" descriptors were unused. Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- drivers/net/ethernet/ti/cpsw.c | 13 +++-- drivers/net/ethernet/ti/cpsw.h | 1 - drivers/net/ethernet/ti/davinci_cpdma.c | 6 ++ drivers/net/e

[PATCH v2 2/2] Documentation: DT: cpsw: remove rx_descs property

2016-06-07 Thread Ivan Khoronzhuk
There is no reason to hold s/w dependent parameter in device tree. Even more, there is no reason in this parameter because davinici_cpdma driver splits pool of descriptors equally between tx and rx channels anyway. Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- Documen

Re: [PATCH v2 1/2] net: ethernet: ti: cpsw: remove rx_descs property

2016-06-11 Thread Ivan Khoronzhuk
On 11.06.16 08:50, David Miller wrote: From: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> Date: Tue, 7 Jun 2016 16:59:35 +0300 if (!cpsw_common_res_usage_state(priv)) { + int buf_num; struct cpsw_priv *priv_sl0 = cpsw_get_slave_priv(priv, 0);

Re: [PATCH] net: ethernet: ti: cpsw: use destroy ctlr to destroy channels

2016-06-11 Thread Ivan Khoronzhuk
On 11.06.16 12:24, Ivan Khoronzhuk wrote: On 11.06.16 04:17, David Miller wrote: From: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> Date: Sat, 11 Jun 2016 01:11:54 +0300 Based on master master... of what? master of git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/lin

[PATCH] net: ethernet: ti: cpsw: use destroy ctlr to destroy channels

2016-06-10 Thread Ivan Khoronzhuk
There is no reason to destroy channels that are destroyed while cpdma_ctlr destroy. In this case no need to remember how much channels where created and destroy them by one, as cpdma_ctlr destroys all of them. Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- Based on

Re: [PATCH] net: ethernet: ti: cpsw: use destroy ctlr to destroy channels

2016-06-11 Thread Ivan Khoronzhuk
On 11.06.16 04:17, David Miller wrote: From: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> Date: Sat, 11 Jun 2016 01:11:54 +0300 Based on master master... of what? master of git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git but it's applicable on linux-next maste

Re: [PATCH v2 0/2] net: ethernet: ti: cpsw: delete rx_descs property

2016-06-14 Thread Ivan Khoronzhuk
On 13.06.16 11:22, Mugunthan V N wrote: On Saturday 11 June 2016 04:34 AM, Schuyler Patton wrote: On 06/08/2016 07:03 PM, Ivan Khoronzhuk wrote: On 09.06.16 02:11, Schuyler Patton wrote: On 06/08/2016 09:06 AM, Ivan Khoronzhuk wrote: On 08.06.16 17:01, Ivan Khoronzhuk wrote: Hi

Re: [PATCH v2 0/2] net: ethernet: ti: cpsw: delete rx_descs property

2016-06-14 Thread Ivan Khoronzhuk
On 13.06.16 18:19, Andrew F. Davis wrote: On 06/13/2016 03:22 AM, Mugunthan V N wrote: On Saturday 11 June 2016 04:34 AM, Schuyler Patton wrote: On 06/08/2016 07:03 PM, Ivan Khoronzhuk wrote: On 09.06.16 02:11, Schuyler Patton wrote: On 06/08/2016 09:06 AM, Ivan Khoronzhuk wrote

Re: [PATCH v2 0/2] net: ethernet: ti: cpsw: delete rx_descs property

2016-06-14 Thread Ivan Khoronzhuk
On 11.06.16 02:04, Schuyler Patton wrote: On 06/08/2016 07:03 PM, Ivan Khoronzhuk wrote: On 09.06.16 02:11, Schuyler Patton wrote: On 06/08/2016 09:06 AM, Ivan Khoronzhuk wrote: On 08.06.16 17:01, Ivan Khoronzhuk wrote: Hi Schuyer, On 07.06.16 18:26, Schuyler Patton wrote: Hi

Re: [PATCH v2 0/2] net: ethernet: ti: cpsw: delete rx_descs property

2016-06-08 Thread Ivan Khoronzhuk
Hi Schuyer, On 07.06.16 18:26, Schuyler Patton wrote: Hi, On 06/07/2016 08:59 AM, Ivan Khoronzhuk wrote: There is no reason in rx_descs property because davinici_cpdma driver splits pool of descriptors equally between tx and rx channels. So, this patch series makes driver to use available

Re: [PATCH v2 0/2] net: ethernet: ti: cpsw: delete rx_descs property

2016-06-08 Thread Ivan Khoronzhuk
On 08.06.16 17:01, Ivan Khoronzhuk wrote: Hi Schuyer, On 07.06.16 18:26, Schuyler Patton wrote: Hi, On 06/07/2016 08:59 AM, Ivan Khoronzhuk wrote: There is no reason in rx_descs property because davinici_cpdma driver splits pool of descriptors equally between tx and rx channels. So

[PATCH] net: ethernet: ti: cpsw: fix rx-usecs interrupt pacing consistency

2016-06-02 Thread Ivan Khoronzhuk
o be 100us - down/up interface - read new unexpected rx-usecs Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- Based on ti-linux-4.4.y drivers/net/ethernet/ti/cpsw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers

Re: [PATCH] net: ethernet: ti: cpsw: fix rx-usecs interrupt pacing consistency

2016-06-02 Thread Ivan Khoronzhuk
On 02.06.16 16:14, Ivan Khoronzhuk wrote: The rx-usecs shouldn't be changed while interface down/up. Currently, for instance, if it's set to 100us, after interface down/up it's 500us. It's a hidden bug that can lead to lavish interrupt pacing time increasing while "down/up" up to

Re: [PATCH 04/15] drivers: net: cpsw: ethtool: fix accessing to suspended device

2016-06-16 Thread Ivan Khoronzhuk
On 16.06.16 15:48, Ivan Khoronzhuk wrote: On 15.06.16 19:45, Grygorii Strashko wrote: On 06/15/2016 07:14 PM, Florian Fainelli wrote: On 06/15/2016 04:55 AM, Grygorii Strashko wrote: The CPSW might be suspended by RPM if all ethX interfaces are down, but it still could be accesible

[PATCH v3 0/2] net: ethernet: ti: cpsw: delete rx_descs property

2016-06-17 Thread Ivan Khoronzhuk
master Since v2: - add declaration of buf_num in correct order Since v1: - separate device tree and driver patches - return number of rx buffers from cpdma driver Ivan Khoronzhuk (2): net: ethernet: ti: cpsw: remove rx_descs property Documentation: DT: cpsw: remove rx_descs property

[PATCH v3 1/2] net: ethernet: ti: cpsw: remove rx_descs property

2016-06-17 Thread Ivan Khoronzhuk
as anyway, the "redundant" descriptors were unused. Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- drivers/net/ethernet/ti/cpsw.c | 13 +++-- drivers/net/ethernet/ti/cpsw.h | 1 - drivers/net/ethernet/ti/davinci_cpdma.c | 6 ++ drivers/net/e

[PATCH v3 2/2] Documentation: DT: cpsw: remove rx_descs property

2016-06-17 Thread Ivan Khoronzhuk
There is no reason to hold s/w dependent parameter in device tree. Even more, there is no reason in this parameter because davinici_cpdma driver splits pool of descriptors equally between tx and rx channels anyway. Acked-by: Rob Herring <r...@kernel.org> Signed-off-by: Ivan Khor

Re: [PATCH] net: ethernet: ti: cpdma: switch to use genalloc

2016-06-23 Thread Ivan Khoronzhuk
genalloc and add desc_num property for each channel for limitation of max number of allowed descriptors for each CPDMA channel. This patch do not affect on net throuput. Cc: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> Signed-off-by: Grygorii Strashko <grygorii.stras...@ti.com> ---

Re: [PATCH 04/15] drivers: net: cpsw: ethtool: fix accessing to suspended device

2016-06-16 Thread Ivan Khoronzhuk
going to add two additional ehttool ops containing RPM sensitive part and not, better to split code on those, if possible...and hiding put/get in begin op can call rpm when it's not needed at all.. -- Regards, Ivan Khoronzhuk

Re: [REGRESSION] firmware: dmi_scan: add SBMIOS entry and DMI tables

2016-03-28 Thread Ivan Khoronzhuk
If I should report this issue somewhere else, please tell me too, and I’ll do my best to follow up there. Thanks, Paul [1] https://www.coreboot.org [2] https://ticket.coreboot.org/issues/33 [3] https://www.coreboot.org/pipermail/coreboot/2015-October/080568.html -- Regards, Ivan Khoronzhuk

Re: [PATCH 1/3] net: ethernet: ti: cpdma: fix lockup in cpdma_ctlr_destroy()

2016-07-23 Thread Ivan Khoronzhuk
cpdma_desc_pool_destroy(ctlr->pool); - spin_unlock_irqrestore(>lock, flags); return ret; } EXPORT_SYMBOL_GPL(cpdma_ctlr_destroy); -- Regards, Ivan Khoronzhuk

[PATCH v3 11/13] net: ethernet: ti: cpsw: move platform data and slaves info to cpsw_common

2016-08-09 Thread Ivan Khoronzhuk
These data are common for net devs in dual_emac mode. No need to hold it for every priv instance, so move them under cpsw_common. Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- drivers/net/ethernet/ti/cpsw.c | 265 + 1 file change

[PATCH v3 13/13] net: ethernet: ti: cpsw: move ale, cpts and drivers params under cpsw_common

2016-08-09 Thread Ivan Khoronzhuk
The ale, cpts, version, rx_packet_max, bus_freq, interrupt pacing parameters are common per net device that uses the same h/w. So, move them to common driver structure. Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- drivers/net/ethernet/ti/cpsw.c

[PATCH v3 07/13] net: ethernet: ti: cpsw: replace pdev on dev

2016-08-09 Thread Ivan Khoronzhuk
No need to hold pdev link when only dev is needed. This allows to simplify a bunch of cpsw->pdev->dev now and farther. Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org> --- drivers/net/ethernet/ti/cpsw.c | 65 ++ 1 file changed, 3

<    1   2   3   4   5   6   7   8   9   10   >