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

Add Eight-Bit-Addressing mode to I2CEEBlockDevice. #12446

Merged
merged 11 commits into from
Feb 21, 2020
+82 −34
Merged

Add Eight-Bit-Addressing mode to I2CEEBlockDevice. #12446

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
cd34860
Add Eight-Bit-Adressing mode to I2CEEBlockDevice.
boomer41 Feb 16, 2020
8d1978e
I2CEEBlockDevice: Add paging to eight bit mode
boomer41 Feb 17, 2020
52aed22
Refactor paged_handler to directly give the paged device address.
boomer41 Feb 17, 2020
e850984
Correctly check return codes from bytewise write function of I2C.
boomer41 Feb 17, 2020
267d8cc
Preserve original size as it is modified inside the handler function.
boomer41 Feb 17, 2020
76a177f
Use pageSize-constant instead of hardcoded value
boomer41 Feb 17, 2020
7c589ae
Fix astyle
boomer41 Feb 17, 2020
532f378
Remove overcomplicated code from I2CEEBlockDevice
boomer41 Feb 18, 2020
3741440
Fix typo and clarify documentation
boomer41 Feb 19, 2020
4d0c346
Fix Code style and reorder class members to get besser packing
boomer41 Feb 19, 2020
440fa49
Remove paging and sync from I2CEEBlockDevice::read(), as it is not ne…
boomer41 Feb 19, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 26 additions & 10 deletions components/storage/blockdevice/COMPONENT_I2CEE/I2CEEBlockDevice.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,17 +22,21 @@ using namespace mbed;

I2CEEBlockDevice::I2CEEBlockDevice(
PinName sda, PinName scl, uint8_t addr,
bd_size_t size, bd_size_t block, int freq)
: _i2c_addr(addr), _size(size), _block(block)
bd_size_t size, bd_size_t block, int freq,
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't directly relate to this PR, but this driver should now really have a constructor that takes const i2c_pinmap_t & and passes to I2C to offer the possibility to avoid pulling in pinmap tables.

It would be nice to add this, but I guess as a follow-up PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I think this should be a separate PR as it actually has nothing to do with the 8bit-mode.

bool address_is_eight_bit)
: _i2c_addr(addr), _size(size), _block(block),
_address_is_eight_bit(address_is_eight_bit)
{
_i2c = new (_i2c_buffer) I2C(sda, scl);
_i2c->frequency(freq);
}

I2CEEBlockDevice::I2CEEBlockDevice(
I2C *i2c_obj, uint8_t addr,
bd_size_t size, bd_size_t block)
: _i2c_addr(addr), _size(size), _block(block)
bd_size_t size, bd_size_t block,
bool address_is_eight_bit)
: _i2c_addr(addr), _size(size), _block(block),
_address_is_eight_bit(address_is_eight_bit)
{
_i2c = i2c_obj;
}
Expand Down Expand Up @@ -60,9 +64,15 @@ int I2CEEBlockDevice::read(void *buffer, bd_addr_t addr, bd_size_t size)

_i2c->start();

if (!_i2c->write(_i2c_addr | 0) ||
!_i2c->write((char)(addr >> 8)) ||
!_i2c->write((char)(addr & 0xff))) {
if (!_i2c->write(_i2c_addr | 0)) {
return BD_ERROR_DEVICE_ERROR;
}

if (!_address_is_eight_bit && !_i2c->write((char)(addr >> 8))) {
return BD_ERROR_DEVICE_ERROR;
}

if (!_i2c->write((char)(addr & 0xff))) {
return BD_ERROR_DEVICE_ERROR;
}

Expand Down Expand Up @@ -92,9 +102,15 @@ int I2CEEBlockDevice::program(const void *buffer, bd_addr_t addr, bd_size_t size

_i2c->start();

if (!_i2c->write(_i2c_addr | 0) ||
!_i2c->write((char)(addr >> 8)) ||
!_i2c->write((char)(addr & 0xff))) {
if (!_i2c->write(_i2c_addr | 0)) {
return BD_ERROR_DEVICE_ERROR;
}

if (!_address_is_eight_bit && !_i2c->write((char)(addr >> 8))) {
return BD_ERROR_DEVICE_ERROR;
}

if (!_i2c->write((char)(addr & 0xff))) {
return BD_ERROR_DEVICE_ERROR;
}

Expand Down
40 changes: 25 additions & 15 deletions components/storage/blockdevice/COMPONENT_I2CEE/I2CEEBlockDevice.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,29 +59,37 @@ class I2CEEBlockDevice : public BlockDevice {
public:
/** Constructor to create an I2CEEBlockDevice on I2C pins
*
* @param sda The pin name for the sda line of the I2C bus.
* @param scl The pin name for the scl line of the I2C bus.
* @param addr The 8bit I2C address of the chip, common range 0xa0 - 0xae.
* @param size The size of the device in bytes
* @param block The page size of the device in bytes, defaults to 32bytes
* @param freq The frequency of the I2C bus, defaults to 400K.
* @param sda The pin name for the sda line of the I2C bus.
* @param scl The pin name for the scl line of the I2C bus.
* @param addr The 8bit I2C address of the chip, common range 0xa0 - 0xae.
* @param size The size of the device in bytes
* @param block The page size of the device in bytes, defaults to 32bytes
* @param freq The frequency of the I2C bus, defaults to 400K.
* @param address_is_eight_bit Specifies whether the EEPROM device is using eight bit
* addresses instead of 16 bit addresses. This should not be needed
* unless dealing with very cheap devices.
*/
I2CEEBlockDevice(
PinName sda, PinName scl, uint8_t address,
bd_size_t size, bd_size_t block = 32,
int bus_speed = 400000);
int bus_speed = 400000,
bool address_is_eight_bit = false);

/** Constructor to create an I2CEEBlockDevice on I2C pins
*
* @param i2c The I2C instance pointer
* @param addr The 8bit I2C address of the chip, common range 0xa0 - 0xae.
* @param size The size of the device in bytes
* @param block The page size of the device in bytes, defaults to 32bytes
* @param freq The frequency of the I2C bus, defaults to 400K.
*/
*
* @param i2c The I2C instance pointer
* @param addr The 8bit I2C address of the chip, common range 0xa0 - 0xae.
* @param size The size of the device in bytes
* @param block The page size of the device in bytes, defaults to 32bytes
* @param freq The frequency of the I2C bus, defaults to 400K.
* @param address_is_eight_bit Specifies whether the EEPROM device is using eight bit
* addresses instead of 16 bit addresses. This should not be needed
* unless dealing with very cheap devices.
*/
I2CEEBlockDevice(
mbed::I2C *i2c_obj, uint8_t address,
bd_size_t size, bd_size_t block = 32);
bd_size_t size, bd_size_t block = 32,
bool address_is_eight_bit = false);

/** Destructor of I2CEEBlockDevice
*/
Expand Down Expand Up @@ -169,6 +177,8 @@ class I2CEEBlockDevice : public BlockDevice {
uint32_t _size;
uint32_t _block;

bool _address_is_eight_bit;
boomer41 marked this conversation as resolved.
Show resolved Hide resolved

int _sync();
};

Expand Down