Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[celestica] change to use lm75 standard driver #19

Merged
merged 4 commits into from
Feb 28, 2018
Merged

[celestica] change to use lm75 standard driver #19

merged 4 commits into from
Feb 28, 2018

Conversation

pwisutti
Copy link

- What I did

  • Change the lm75 driver to standard driver for kernel version 4.9.
  • Update init script for creating lm75 devices because the driver's name is changed.

- How I did it

- How to verify it

- Description for the changelog

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

@lguohan
Copy link
Owner

lguohan commented Feb 27, 2018

you can remove the lm75 driver since the kernel image has the driver, you do not need to build it in your module.

lguohan
lguohan requested changes Feb 27, 2018
View reviewed changes
Copy link
Owner

@lguohan lguohan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just remove the lm75 driver from your module.

@pwisutti
Copy link
Author

pwisutti commented Feb 28, 2018 •
edited
Loading

I've already removed lm75 driver from the module and also configured the init script to probe lm75 driver.

lguohan
lguohan reviewed Feb 28, 2018
View reviewed changes
platform/broadcom/sonic-platform-modules-cel/dx010/modules/Makefile Outdated
@@ -1,6 +1 @@
obj-m := dx010_cpld.o mc24lc64t.o emc2305.o dx010_wdt.o leds-dx010.o lm75.o
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you still have lm75.o here?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you make sure your dx010 module can compile as it is?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it should be removed. I've fixed

@lguohan lguohan merged commit e9f28e1 into lguohan:stretch Feb 28, 2018
lguohan pushed a commit that referenced this pull request Mar 3, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
1e69ee0
lguohan pushed a commit that referenced this pull request Mar 14, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
fad487a
lguohan pushed a commit that referenced this pull request Mar 19, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
2999dcf
lguohan pushed a commit that referenced this pull request Apr 4, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
7417dab
lguohan pushed a commit that referenced this pull request Apr 6, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
de4e044
lguohan pushed a commit that referenced this pull request Apr 14, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
9097215
lguohan pushed a commit that referenced this pull request May 2, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
9eabda4
lguohan pushed a commit that referenced this pull request May 3, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
6610ab3
lguohan pushed a commit that referenced this pull request Jul 10, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
ebe07ed
lguohan pushed a commit that referenced this pull request Jul 13, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
e2401b4
lguohan pushed a commit that referenced this pull request Jul 24, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
7b0eb0e
lguohan pushed a commit that referenced this pull request Jul 30, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
a876018
lguohan pushed a commit that referenced this pull request Aug 5, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
3e62a86
lguohan pushed a commit that referenced this pull request Aug 11, 2018
@pwisutti @lguohan
[celestica] change to use lm75 standard driver (#19)
025dad0
lguohan pushed a commit that referenced this pull request Nov 4, 2019
@yxieca
[201811] advance sub module heads (sonic-net#3135)
050787e 
Submodule src/sonic-platform-common 42119e1..5d7954e:
  > [ChassisBase] Make reboot cause constant strings human-readable (#35)
  > Add .gitignore file (#28)
  > [sonic_platform_base] Add sonic_sfp and sonic_eeprom to sonic_platform_base (#27)
  > Enhance new platform API (#19)
  > fix typo in platform API base class (#25)

Submodule src/sonic-swss 9cf7b01..1e99c93:
  > Set timer only when interval changes. Not in each firing of the timer. (sonic-net#945)

Submodule src/sonic-utilities ec1e93f..24958f1:
  > [fast reboot] stop removing opennsl module before reboot (sonic-net#560)

Submodule src/sonic-swss-common b472f6e..d6140fa:
  > timerfd:read failure - Record in logs as error. (sonic-net#295)
  > do not abort when read timerfd return 0 and errno = 0 (sonic-net#291)
  > Add an assert to logger, which will log a message and abort. (sonic-net#286)

Signed-off-by: Ying Xie <[email protected]>
lguohan pushed a commit that referenced this pull request Oct 8, 2020
@jleveque
[sonic-utilities] Build and install as a Python wheel package (sonic-…
3987cbd 
…net#5409)

We are moving toward building all Python packages for SONiC as wheel packages rather than Debian packages. This will also allow us to more easily transition to Python 3.

Python files are now packaged in "sonic-utilities" Pyhton wheel. Data files are now packaged in "sonic-utilities-data" Debian package.

**- How I did it**
- Build and install sonic-utilities as a Python package
- Remove explicit installation of wheel dependencies, as these will now get installed implicitly by pip when installing sonic-utilities as a wheel
- Build and install new sonic-utilities-data package to install data files required by sonic-utilities applications
- Update all references to sonic-utilities scripts/entrypoints to either reference the new /usr/local/bin/ location or remove absolute path entirely where applicable

Submodule updates:

* src/sonic-utilities aa27dd9...2244d7b (5):
  > Support building sonic-utilities as a Python wheel package instead of a Debian package (sonic-net#1122)
  > [consutil] Display remote device name in show command (sonic-net#1120)
  > [vrf] fix check state_db error when vrf moving (sonic-net#1119)
  > [consutil] Fix issue where the ConfigDBConnector's reference is missing (sonic-net#1117)
  > Update to make config load/reload backward compatible. (sonic-net#1115)

* src/sonic-ztp dd025bc...911d622 (1):
  > Update paths to reflect new sonic-utilities install location, /usr/local/bin/ (#19)
lguohan pushed a commit that referenced this pull request Dec 19, 2020
@dflynn-Nokia
[sonic-mgmt-common] Update submodule (sonic-net#6129)
98a8753 
This update brings in the following commits.

86c1108 Enable arm architecture to build in addition to amd64 (#37)
4acb2c3 fix bugs and enhance Transformer (#35)
49e5a22 ygot related enhancements and fixes (#34)
51224de Fix ietf yang search path for cvl schema builds (#32)
3c6cdb3 CVL Changes #8: 'must' and 'when' expression evaluation (#31)
dabf231 CVL Changes #7: 'leafref' evaluation (#28)
6f9535f CVL Changes #6: Customized Xpath Engine integration (#27)
5e2466b DB-Layer fixes/enhancements (#26)
9a27302 CVL Changes #4: Implementation of new CVL APIs (#22)
dbf1093 Translib support for authorization, yang versioning and Delete flag (#21)
80f369e CVL Changes #5: YParser enhancement (#23)
904ce18 CVL Changes #3: Multi-db instance support (#20)
9d24a34 CVL Changes #2:  YValidator infra changes for evaluating xpath expression (#19)
f3fc40f CVL Changes #1: Initial CVL code reorganization and common infra changes (#18)
4922601 Bulk and RPC API support in translib (#16)
1d730df RFC7895 yang module library implementation (#15)
lguohan pushed a commit that referenced this pull request May 30, 2022
@KostiantynYarovyiBf @VadymYashchenko
@dmytroxIntel @vboykox @bratashX @mykolaxgerasymenko
[BFN] Updated platform plugins (sonic-net#9540)
4037867 
* [BFN] Updated platform APIs impl

Signed-off-by: Andriy Kokhan <[email protected]>

* Extended BFN platform SFP APIs implementation

* Update sfp.py

* [BFN] Extended SFP platform plugin implementation

Signed-off-by: Andriy Kokhan <[email protected]>

* [BFN] Extended Fans platform plugin implementation

* [BFN] divided classes Fan and  FanDrawer into 2 files

* Signed-off-by: Vadym Yashchenko <[email protected]>

What I did
	Add get_model() function
	Add get_low_critical_threshold() function
	Change __get(...) function.
How I did it
	Differnece from previous implementation of __get(...) function is return real value or -9999.9 if value is not provided by thrift API

* Add get_presence() function and revised __get() function

Signed-off-by: Vadym Yashchenko <[email protected]>

* [BFN] Updated PSU platform APIs impl

Signed-off-by: Dmytro Lytvynenko <[email protected]>

* Added BFN PSU cache (#9)

Signed-off-by: Andriy Kokhan <[email protected]>

* [BFN]  Fans and Fantray platform APIs update (#7)

* [BFN] Updated SFP platform APIs (#10)

Signed-off-by: Volodymyr Boyko <[email protected]>

* [BFN] Updated platform API for thermal (#8)

* Signed-off-by: Vadym Yashchenko <[email protected]>

* Revert "[BFN]  Fans and Fantray platform APIs update (#7)" (#11)

This reverts commit c62a733.

* Add support health monitor system (#15)

Signed-off-by: Petro Bratash <[email protected]>

* Update chassis.py

* [BFN] Updated FANs and FAN Tray platform API (#14)

* Fix fix_alignment (#17)

Signed-off-by: Petro Bratash <[email protected]>

* [BFN] Improvement show environment (#16)

* Added PSU temperature skip into platform.json (#18)

Signed-off-by: Andriy Kokhan <[email protected]>

* Do not skip psud on Newport

Signed-off-by: Andriy Kokhan <[email protected]>

* [BFN] fix fan status from Not OK to Ok (#19)

* [BFN] Updated SFP platform plugin (#13)

Signed-off-by: Volodymyr Boyko <[email protected]>

* [DPB] Fix typo for Ethernet0 2x200G[100G,40G] breakout mode (#21)

Signed-off-by: Mykola Gerasymenko <[email protected]>

* [barefoot] Tmp fix vendor_rev (#22)

Signed-off-by: Volodymyr Boyko <[email protected]>

* Fixed python issues in sonic_platform/fan_drawer.py

Signed-off-by: Andriy Kokhan <[email protected]>

* Updated fan_drawer.py

* Fixing trailing white spaces in fan_drawer.py

* [BFN] Fix thrift for SFPs API

Signed-off-by: Volodymyr Boyko <[email protected]>

* In platform.json, replaced 'false' with '0' to workaround ast.literal_eval() issue

Signed-off-by: Andriy Kokhan <[email protected]>

* [Newport] Thermal manager  (#23)

* Signed-off-by: Vadym Yashchenko <[email protected]>

* Revert "In platform.json, replaced 'false' with '0' to workaround ast.literal_eval() issue"

This reverts commit 1e73127.

* Removed 'controllable' options from platform.json to fix factory default config generation

Signed-off-by: Andriy Kokhan <[email protected]>

* Update thermal_manager.py

* Migrated SFP plugin to sonic_xcvr API (#30)

Signed-off-by: Andriy Kokhan <[email protected]>

Co-authored-by: KostiantynYarovyiBf <[email protected]>
Co-authored-by: Vadym Yashchenko <[email protected]>
Co-authored-by: Dmytro Lytvynenko <[email protected]>
Co-authored-by: Volodymyr Boiko <[email protected]>
Co-authored-by: Petro Bratash <[email protected]>
Co-authored-by: Mykola Gerasymenko <[email protected]>
lguohan pushed a commit that referenced this pull request Nov 21, 2022
@judyjoseph
Update submodule (sonic-net#12635)
1320319 
sonic-host-services submodule update with following changes

6eac2d3 Merge pull request #19 from judyjoseph/macsec_feature_enable
31c6108 Sync has_per_asic_scope attribute to config_db in all namespaces for multi-asic
185547f Add UT to improve coverage, for namespace config update
7c3aca0 macsec_supported info is part of DEVICE_RUNTIME_METADATA itself.
lguohan pushed a commit that referenced this pull request Jan 3, 2023
@kellyyeh
Advance dhcprelay submodule head (sonic-net#12717)
947561c 
Added below commits:
9b30690 jcaiMR Fri Dec 16 fix handleSwssNotification crash in dhcp6relay (#28)
047afb7 jcaiMR Wed Dec 14 14:08:58 2022 +0800 Fix multiple vlan issue (#27)
ff6bec3 Vivek Thu Dec 8 09:44:15 2022 -0800 Made the Error log informative (#22)
2fbe729 jcaiMR Wed Nov 30 14:41:53 2022 +0800 disable cfg dynamic change (#25)
13d0805 Liu Shilong Wed Nov 30 10:54:11 2022 +0800 Use github code scanning instead of LGTM (#26)
1e846f6 kellyyeh Wed Nov 23 14:36:02 2022 -0800 Fix packet range check for relay-reply packets (#21)
4d19e13 kellyyeh Thu Nov 17 10:04:53 2022 -0800 Add unittest infrastructure (#5)
7f4fdab jcaiMR Fri Nov 11 14:47:51 2022 +0800 fix packet range check issue (#20)
257ecdf kellyyeh Thu Nov 3 11:34:11 2022 -0700 Add client packet UDP header length check (#19)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lguohan lguohan lguohan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pwisutti @lguohan