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
.
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
-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
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
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
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
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
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
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
.
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
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
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
- 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
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
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
), 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
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
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
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
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
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
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
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
- 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
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
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
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
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
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
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
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
, 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
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
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
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
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
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
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
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
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
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
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
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
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
+ 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
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
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
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
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
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
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
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
, 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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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.
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
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
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
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
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
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
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
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);
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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>
---
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
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
cpdma_desc_pool_destroy(ctlr->pool);
- spin_unlock_irqrestore(>lock, flags);
return ret;
}
EXPORT_SYMBOL_GPL(cpdma_ctlr_destroy);
--
Regards,
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
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
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
201 - 300 of 1224 matches
Mail list logo