help to update kernel patch for ag9064 with LPC by hans-tseng · Pull Request #1447 · sonic-net/sonic-buildimage

hans-tseng · 2018-03-02T08:51:50Z

- What I did
please help to add the kernel patch on ag9064 platform which supports the LCP and ipmi tools
- How I did it
add the patch file "config-delta-ag9064.patch" in to kernel patch
- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: hans <hans.tseng@deltaww.com>

lguohan · 2018-03-02T21:11:21Z

platform/broadcom/sonic-platform-modules-delta/ag9064/config-delta-ag9064.patch

@@ -0,0 +1,44 @@
+diff --git a/debian/build/build_amd64_none_amd64/.config b/debian/build/build_amd64_none_amd64/.config


this should going to sonic-linux-kernel repo.

I will push it to kernel repo

lguohan · 2018-03-03T05:25:39Z

platform/broadcom/sonic-platform-modules-delta/debian/files

@@ -1 +1,2 @@
 platform-modules-ag9032v1_1.1_amd64.deb main extra
+platform-modules-ag9064_1.1_amd64.deb main extra


this is debian generated file, please remove this file completed.

I will remove it

lguohan · 2018-03-03T18:33:06Z

platform/broadcom/sonic-platform-modules-delta/ag9064/modules/hbm.h

+struct mei_msg_hdr;
+struct mei_cl;
+
+/**


this file is in the 3.16.0-5 kernel code why do we include it here?

i will check and remove it

lguohan · 2018-03-03T18:33:37Z

platform/broadcom/sonic-platform-modules-delta/ag9064/modules/hw.h

+ * This program is distributed in the hope it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+ * more details.


this file is also in 3.16.0-5 kernel source tree.

I will check and remove it

lguohan · 2018-03-03T18:35:52Z

platform/broadcom/sonic-platform-modules-delta/ag9064/modules/i2c-mei_main.c

+#include <linux/ioport.h>
+#include <linux/init.h>
+#include <linux/i2c.h>
+#include <linux/acpi.h>


what is this mei-i2c driver used for? can you provide context here? why do we need it on this platform?

We will add the comment on it

lguohan

as comments

hans-tseng · 2018-03-05T01:22:24Z

Hi, sir
I will close this pull request and re-update again as comments

be51ebc Add IPv6 key item support to request parser (sonic-net#1449) 76e2251 When teamd feature state is disabled the Netdevice created by teamd were (sonic-net#1450) 6aa97ce Use .clear() after std::move() (sonic-net#1444) d5757db Add libzmq to README dependencies (sonic-net#1447) c7b262e Add libzmq to Makefiles (sonic-net#1443) 0b2e59a [drop counters] Clarify log messages for initial counter setup (sonic-net#1445) 003cf24 [dvs] Refactor and add buffer pool wm test (sonic-net#1446) 2f5d2d9 [acl] Remove Ethertype from L3V6 qualifiers (sonic-net#1433) f7b974f Fix issue: bufferorch only pass the first attribute to sai when setting attribute (sonic-net#1442) Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>

be51ebc Add IPv6 key item support to request parser (#1449) 76e2251 When teamd feature state is disabled the Netdevice created by teamd were (#1450) 6aa97ce Use .clear() after std::move() (#1444) d5757db Add libzmq to README dependencies (#1447) c7b262e Add libzmq to Makefiles (#1443) 0b2e59a [drop counters] Clarify log messages for initial counter setup (#1445) 003cf24 [dvs] Refactor and add buffer pool wm test (#1446) 2f5d2d9 [acl] Remove Ethertype from L3V6 qualifiers (#1433) f7b974f Fix issue: bufferorch only pass the first attribute to sai when setting attribute (#1442) Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>

Refactor neighbor_advertiser script (#1447) Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>

be51ebc Add IPv6 key item support to request parser (sonic-net#1449) 76e2251 When teamd feature state is disabled the Netdevice created by teamd were (sonic-net#1450) 6aa97ce Use .clear() after std::move() (sonic-net#1444) d5757db Add libzmq to README dependencies (sonic-net#1447) c7b262e Add libzmq to Makefiles (sonic-net#1443) 0b2e59a [drop counters] Clarify log messages for initial counter setup (sonic-net#1445) 003cf24 [dvs] Refactor and add buffer pool wm test (sonic-net#1446) 2f5d2d9 [acl] Remove Ethertype from L3V6 qualifiers (sonic-net#1433) f7b974f Fix issue: bufferorch only pass the first attribute to sai when setting attribute (sonic-net#1442) Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>

…03 (sonic-net#1447) ```<br>* c3ce9e119 - (HEAD -> 202503) Merge branch '202412' of https://github.com/Azure/sonic-buildimage-msft into 202503 (2025-08-02) [Sonic Automation] * e82f230 - (origin/202412, 202412) Revert "Add GNMI client cert cname check support. (sonic-net#18709)" (sonic-net#1438) (2025-08-01) [Feng-msft]<br>```

hanstseng added 3 commits March 1, 2018 14:50

add the ag9064 platform

3b6fc7b

Signed-off-by: hans <hans.tseng@deltaww.com>

remove the unnecessary file

b2dcd1d

Signed-off-by: hans <hans.tseng@deltaww.com>

please help to add the kernel patch for ag9064 for ipmi

cb46c37

Signed-off-by: hans <hans.tseng@deltaww.com>

lguohan reviewed Mar 2, 2018

View reviewed changes

lguohan reviewed Mar 3, 2018

View reviewed changes

lguohan requested changes Mar 3, 2018

View reviewed changes

hans-tseng closed this Mar 5, 2018

abdosi mentioned this pull request Sep 29, 2020

[Submodule update] sonic swss #5489

Merged

3 tasks

abdosi added a commit that referenced this pull request Feb 20, 2021

[submodule update] sonic-utilities

fa1934f

Refactor neighbor_advertiser script (#1447) Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>

theasianpianist added a commit to theasianpianist/sonic-buildimage that referenced this pull request Feb 5, 2022

Add libzmq to README dependencies (sonic-net#1447)

d5757db

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

help to update kernel patch for ag9064 with LPC#1447

help to update kernel patch for ag9064 with LPC#1447
hans-tseng wants to merge 3 commits intosonic-net:masterfrom
hans-tseng:master

hans-tseng commented Mar 2, 2018

Uh oh!

lguohan Mar 2, 2018

Uh oh!

hans-tseng Mar 5, 2018

Uh oh!

lguohan Mar 3, 2018

Uh oh!

hans-tseng Mar 5, 2018

Uh oh!

lguohan Mar 3, 2018 •

edited

Loading

Uh oh!

hans-tseng Mar 5, 2018

Uh oh!

lguohan Mar 3, 2018

Uh oh!

hans-tseng Mar 5, 2018

Uh oh!

lguohan Mar 3, 2018

Uh oh!

hans-tseng Mar 5, 2018

Uh oh!

lguohan left a comment

Uh oh!

hans-tseng commented Mar 5, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		@@ -0,0 +1,44 @@
		diff --git a/debian/build/build_amd64_none_amd64/.config b/debian/build/build_amd64_none_amd64/.config

		@@ -1 +1,2 @@
		platform-modules-ag9032v1_1.1_amd64.deb main extra
		platform-modules-ag9064_1.1_amd64.deb main extra

Conversation

hans-tseng commented Mar 2, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

lguohan Mar 3, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

lguohan left a comment

Choose a reason for hiding this comment

Uh oh!

hans-tseng commented Mar 5, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

lguohan Mar 3, 2018 •

edited

Loading